Bug 160544 - Quickfind sidebar: option to expose the sidebar Quick Find ctrl+F (comment 20)
Summary: Quickfind sidebar: option to expose the sidebar Quick Find ctrl+F (comment 20)
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
24.8.0.0 alpha0+
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on: 85905
Blocks: Sidebar-Find
  Show dependency treegraph
 
Reported: 2024-04-05 13:58 UTC by Heiko Tietze
Modified: 2024-07-26 22:14 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Heiko Tietze 2024-04-05 13:58:43 UTC
Follow-up to bug 95405: The sidebar is supposed to be an improvement to the classic toolbar. We should default the quicksearch to this control - and have the legacy toolbar as an alternative option. 

Could be Tools > Options > Writer > View: Default Quick Find (o) Sidebar ( ) Toolbar
Comment 1 V Stuart Foote 2024-04-05 16:02:12 UTC
The new Quick find deck is not a replacement for the "Find and Replace" dialog, and it is only a supplement to current <Ctrl>+F Find bar, and "Repeat find" from the Navigator (it reused common code).

The quick find was tracked with proposal of bug 95405, but thus far does not address the full feature set of the <Ctrl>+G dialog which would require an entirely new content panel.

See no advantage ATM to forcing <Ctrl>+F to the Sidebar deck.
Comment 2 Heiko Tietze 2024-04-05 16:04:10 UTC
Talking about quickfind (ctrl+F) only. Ctrl+G or H should remain at the F&R dialog.
Comment 3 V Stuart Foote 2024-04-05 16:10:11 UTC
(In reply to Heiko Tietze from comment #2)
> ... Ctrl+G or H should remain at the F&R dialog.

Yes, <Ctrl>+H (sorry my typo).  <Ctrl>+G of course is the "Go to..." page dialog.

NTL don't think the SB Quick find deck is ready for default use--especially with out a corresponding F&R content panel worked up.
Comment 4 Cor Nouws 2024-04-05 20:09:34 UTC
The quickfind works as follows:
- Ctrl+F <find foo> ESC
=> all as before.

Will that also be the case with Find in de sidebar? Will ESC bring the sidebar to the state of before hitting Ctrl+F?
Comment 5 Heiko Tietze 2024-04-08 07:13:09 UTC
I'd just keep the sidebar open, but Ctrl+F5 or F11 both toggle the sidebar.
Comment 6 Cor Nouws 2024-04-09 21:33:03 UTC
(In reply to Heiko Tietze from comment #5)
> I'd just keep the sidebar open, but Ctrl+F5 or F11 both toggle the sidebar.

So that looks as if it would interrupt possibly work (based on some selection, collapsed panels..) in the sidebar?
Comment 7 Roman Kuznetsov 2024-04-17 07:43:17 UTC
set to NEW as a valid enhancement
Comment 8 Cor Nouws 2024-04-17 10:21:06 UTC
(In reply to Cor Nouws from comment #6)
> (In reply to Heiko Tietze from comment #5)
> > I'd just keep the sidebar open, but Ctrl+F5 or F11 both toggle the sidebar.
> 
> So that looks as if it would interrupt possibly work (based on some
> selection, collapsed panels..) in the sidebar?
IMO this needs to be sorted out before this gets implemented.
Comment 9 Heiko Tietze 2024-04-17 12:08:34 UTC
(In reply to Cor Nouws from comment #8)
> (In reply to Cor Nouws from comment #6)
> > (In reply to Heiko Tietze from comment #5)
> > > I'd just keep the sidebar open, but Ctrl+F5 or F11 both toggle the sidebar.
> > 
> > So that looks as if it would interrupt possibly work (based on some
> > selection, collapsed panels..) in the sidebar?
> IMO this needs to be sorted out before this gets implemented.
I'm fine with toggle as well as to keep the sidebar open (command just opens).
Comment 10 Cor Nouws 2024-04-17 16:03:27 UTC
(In reply to Heiko Tietze from comment #9)
> I'm fine with toggle as well as to keep the sidebar open (command just
> opens).
You mean that you accept that the status/situation etc of the Sidebar changes when you use Ctrl+F for search?
If so: I am not, but could leave with an expert option that allows me to keep the current behavior :)
Comment 11 Cor Nouws 2024-04-17 16:05:25 UTC
(In reply to Cor Nouws from comment #10)
> If so: I am not, but could leave with an expert option that allows me to
> keep the current behavior :)
Sight, should be:  If so: "I do not, but could live with ..."
Comment 12 Cor Nouws 2024-05-02 10:06:08 UTC
I think it's worth to discuss this again before someone puts his/her hands on this. See previous comments
Comment 13 V Stuart Foote 2024-05-02 11:51:42 UTC
Too much remains to be done on the SB content panels for a replacement of the Find bar (<Ctrl>+F) TB, with even more needed to supplement/replace the <Ctrl>+H "Find and Replace..." dialog.

Also, as a normal TB the Find bar can be undocked, supporting workflows where that is useful. 

While the *entire& SB would have to be undocked (absent the work needed for bug 85905) or the quick find replicated (like the Navigator) using SB framework. 

Just not an appealing UI.
Comment 14 Heiko Tietze 2024-05-02 11:55:18 UTC
(In reply to V Stuart Foote from comment #13)
> Too much remains to be done on the SB content panels...
What exactly is the blocker?

> Just not an appealing UI.
Maybe but a much superior workflow.
Comment 15 V Stuart Foote 2024-05-02 13:43:43 UTC
Hmm, lets count them up:

Functional shortfalls
1. always performs a 'find all', with no 'next' 'previous'
2. always performs transliterated string search, no 'Match Case'
3. no button link to the full 'Find and Replace...' dialog
4. no results infobar 'Search key found # times'

UI short falls
1. the find returns the matches embedded in a wall of text, with brackets around the match rather than highlights
2. the string enclosing the match is unpredictable, should use a ICU break iterator to start and end result string on sentence, or a clause break
3. for matches too numerous to fit, truncate the Content deck list box to single line--no wrap, each match centered in the content panel
4. mouse click navigation between the matches, just cursor U,D button movement; content panel needs same Navigate buttons as the Navigator
5. when matches span more than one page, the on page visual highlights alone don't give sufficient feedback--need match details (maybe Para, Page, Document) as a content panel.
Comment 16 Cor Nouws 2024-05-05 07:00:08 UTC
(In reply to Heiko Tietze from comment #14)
> Maybe but a much superior workflow.

I still miss reflection on what IMO will be a horrible outcome of the work as it is envisioned now: after using Ctrl+F for search, the status/situation etc of the Sidebar is changed, meaning the user would need to manually re-establish that state, while currently Ctrl+F .. <find> .. ESC leaves all other UI elements as is.
Comment 17 V Stuart Foote 2024-05-05 12:31:00 UTC
(In reply to Cor Nouws from comment #16)

>... after using Ctrl+F for search, the status/situation
> etc of the Sidebar is changed, meaning the user would need to manually
> re-establish that state, while currently Ctrl+F .. <find> .. ESC leaves all
> other UI elements as is.

Not sure about that, we have shortcut/accelerators for each of the SB decks, <Alt>1-9, or <F11> for the Stylist--or a mouse click selection on the SB deck buttons. So can just pop back to the deck needed, simply leaving the Find deck. No more involved than using the <Esc> now from the Find bar.
Comment 18 Cor Nouws 2024-05-05 21:03:41 UTC
(In reply to V Stuart Foote from comment #17)
> Not sure about that, we have shortcut/accelerators for each of the SB decks,
> <Alt>1-9, or <F11> for the Stylist--or a mouse click selection on the SB
> deck buttons. So can just pop back to the deck needed, simply leaving the
> Find deck. No more involved than using the <Esc> now from the Find bar.
Almost as simple, since the focus/selection in the pane changes (at least for me).
Comment 19 Eyal Rozenberg 2024-05-08 22:18:27 UTC
(In reply to Heiko Tietze from comment #0)
> Follow-up to bug 95405: The sidebar is supposed to be an improvement to the
> classic toolbar. We should default the quicksearch to this control - and
> have the legacy toolbar as an alternative option. 

When we have a decent search sidebar, then let's talk about that. 

Fix some of its bugs (I'm going to file a couple after my first single use), add some of the Find dialog functionality, etc... 

So, for now, definite no-no from me.

Even in principle I'm not sure I would support this, because the sidebar takes a huge chunk of your viewport away, while the find bar is just a vertical bar, and at the bottom where it's less noticeable than a top toolbar.
Comment 20 Heiko Tietze 2024-05-17 07:27:19 UTC
We discussed the topic in the design meeting.

Most participants shared the view of an unfinished state but agreed on the option. It should consequently remain on the toolbar.
Comment 21 Eyal Rozenberg 2024-05-17 10:36:17 UTC
(In reply to Heiko Tietze from comment #20)
> Most participants shared the view of an unfinished state but agreed on the
> option.

While I can't badmouth opt-in optional functionality, I quite doubt the usefulness of this. I don't see the "superior workflow" Heiko mentions.

Anyway, adding a couple of glaring bugs before the resolution of which this option absolutely doesn't make sense.
Comment 22 Heiko Tietze 2024-05-17 12:56:49 UTC
(In reply to Eyal Rozenberg from comment #21)
> Anyway, adding a couple of glaring bugs before the resolution of which this
> option absolutely doesn't make sense.
Thanks for digging BZ but better add these as See Also.