Use a searchbar similar to the one in okular
ClosedPublic

Authored by ognarb on Mar 8 2019, 9:32 PM.

Diff Detail

Repository
R875 Falkon
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ognarb created this revision.Mar 8 2019, 9:32 PM
Restricted Application added a project: Falkon. · View Herald TranscriptMar 8 2019, 9:32 PM
Restricted Application added a subscriber: falkon. · View Herald Transcript
ognarb requested review of this revision.Mar 8 2019, 9:32 PM
ognarb edited the summary of this revision. (Show Details)Mar 8 2019, 9:32 PM
ognarb added a reviewer: Falkon.
ognarb added a reviewer: VDG.Mar 8 2019, 9:43 PM
ognarb updated this revision to Diff 53486.Mar 8 2019, 10:25 PM

Use find instead of search

SGOrava added a subscriber: SGOrava.Mar 9 2019, 7:20 AM

Hello,

My crying:
I usually search for short phrases or few words when having extra big (screen sized) seach field is meaningles.
The advantage of current approach is that all control buttons are close together (I wuld call it "compact design")

But I also have to see a useful side of this patch > tooltips.
Current searchbar buttons lack tooltips.

For the current patch:
It lack icon for "Match case" button, in kate there is nice icon used for this.

Hello,

My crying:
I usually search for short phrases or few words when having extra big (screen sized) seach field is meaningles.
The advantage of current approach is that all control buttons are close together (I wuld call it "compact design")

I don't have any preference, but it should be consistent between okular, khtml, konqueror and ktexteditor. Any opinions from VDG?

ognarb retitled this revision from Use a searchbar similar to the one from okular to Use a searchbar similar to the one in okular.Mar 9 2019, 11:24 AM
loh.tar added a subscriber: loh.tar.Mar 9 2019, 1:34 PM

Just my 2Cents

The advantage of current approach is that all control buttons are close together (I wuld call it "compact design")

I also dislike when the edit field and some related buttons are to wide separated

  • I dislike ToolTips, they are most of the time annoying, WhatsThis is not so easy to find but on the far end better
  • In this case are the meaning of the buttons pretty clear, I see no need for some extra explanation
  • It looks like that your patch bring the missed clear-field button, great!
drosca accepted this revision.Mar 14 2019, 8:56 AM
drosca added a subscriber: drosca.

I like it, thanks!

This revision is now accepted and ready to land.Mar 14 2019, 8:56 AM

Just my 2Cents

The advantage of current approach is that all control buttons are close together (I wuld call it "compact design")

I also dislike when the edit field and some related buttons are to wide separated

  • I dislike ToolTips, they are most of the time annoying, WhatsThis is not so easy to find but on the far end better

System settings -> Desktop Behvior -> Workspace -> Uncheck "Display informational tooltips on mouse hover" Enjoy ;)

  • In this case are the meaning of the buttons pretty clear, I see no need for some extra explanation

It's an accessibility feature.

  • It looks like that your patch bring the missed clear-field button, great!

:D

I like it, thanks!

Thanks ;)

This revision was automatically updated to reflect the committed changes.