simgunz added inline comments.

INLINE COMMENTS

> davidhurka wrote in toolaction.cpp:71
> Why remove this tooltip?
> 
> To make ToolAction (or my inferior ToggleActionMenu) more universal to use it 
> for a Change Colors feature, it would be nice if tooltips are not needed 
> anymore.

That tooltip does not appear anywhere in the UI in practice, at least I have 
not manage to make it appear. Have you?

Now I am using ToolAction to show other things, so the tooltips, if needed, 
need to be set outside. Still a TODO.

Where is ToggleActionMenu defined?

REPOSITORY
  R223 Okular

REVISION DETAIL
  https://phabricator.kde.org/D15580

To: simgunz, #okular
Cc: aacid, anthonyfieroni, davidhurka, knambiar, ngraham, tobiasdeiminger, 
okular-devel, fbampaloukas, joaonetto, tfella, darcyshen

Reply via email to