Bug 160543 - Quickfind sidebar: Search depends on F&R settings
Summary: Quickfind sidebar: Search depends on F&R settings
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
24.8.0.0 alpha0+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:25.2.0
Keywords:
Depends on:
Blocks: Sidebar-Find
  Show dependency treegraph
 
Reported: 2024-04-05 13:54 UTC by Heiko Tietze
Modified: 2024-08-23 20:24 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
options and number of matches label (391.91 KB, image/png)
2024-06-14 23:36 UTC, Jim Raykowski
Details
with similarity option (30.93 KB, image/png)
2024-06-15 00:42 UTC, Jim Raykowski
Details
with find bar 'Match Case' and F&R icon button and additional icon button for more search options (45.89 KB, image/png)
2024-06-17 18:11 UTC, Jim Raykowski
Details
Demo of search options dialog approach (1.21 MB, video/x-matroska)
2024-06-22 22:36 UTC, Jim Raykowski
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Heiko Tietze 2024-04-05 13:54:21 UTC
Follow-up to bug 95405: The quickfind sidebar (QFS) has no means to change the search and always takes the options from the F&R dialog. We should at least add "[ ] Whole words only" and "[ ] Similarity" and make the QFS independent from the F&R search.
Comment 1 V Stuart Foote 2024-04-05 18:04:18 UTC
QFS should have *no* dependency on the Find & Replace dialog, rather should come from config for the Findbar.

Also, note there is a results "Search key found xx times" showing in the Find bar--that detail really belongs in the SB QFS content panel, not requiring the Find bar to be shown.
Comment 2 V Stuart Foote 2024-04-05 18:06:30 UTC
=-testing-=
TB78 - 20240405

Version: 24.8.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 30c6e51fc9cb0fa864e1755271343d847fcced25
CPU threads: 8; OS: Windows 10.0 Build 19045; UI render: Skia/Vulkan; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: CL threaded
Comment 3 Jim Raykowski 2024-06-14 23:36:52 UTC
Created attachment 194742 [details]
options and number of matches label

(In reply to Heiko Tietze from comment #0)
> Follow-up to bug 95405: The quickfind sidebar (QFS) has no means to change
> the search and always takes the options from the F&R dialog. We should at
> least add "[ ] Whole words only" and "[ ] Similarity" and make the QFS
> independent from the F&R search.
Ah! I didn't remember your comment about the Similarity option when doing this. Should Match Case be replaced by it?

(In reply to V Stuart Foote from comment #1)
> Also, note there is a results "Search key found xx times" showing in the
> Find bar--that detail really belongs in the SB QFS content panel, not
> requiring the Find bar to be shown.
The number of finds is also indicated in the status bar selected word and character count although for unknown reason the status bar selected word count is different.
Comment 4 Jim Raykowski 2024-06-15 00:42:37 UTC
Created attachment 194743 [details]
with similarity option
Comment 5 Heiko Tietze 2024-06-17 07:26:01 UTC
Not a fan of too many options everywhere. Perhaps we can hide it with an expander. Or provide these options in a popup menu. Small toolbar buttons could work too (thinking of VSCode), though add space to the minimum width (btw. the two checkboxes can become much larger when translated).

Similarity is kind of orthogonal to the other options- if fully exclusive consider to disable checkboxes.
Comment 6 V Stuart Foote 2024-06-17 16:02:54 UTC
I think the QFS needs to be its own SB content panel, while 'Find & Replace' dialog gets its own work-up as a new fully populated content panel. 

Quick Find should *continue to track* against the Find Bar behavior. 

Meaning cutting back scope of some of the panel formatting of the results. E.g. 'Whole Words' is not available to the Find bar, nor in the Navigator's 'Repeat search' mode. Find Bar options are currently limited to "find in values" and "match case", with choice to show the "Search results message". 

Doing more complex find/replace actions on the SB framework needs a new *dedicated* panel with its own design and integration with the F&R dialog config(s). 

And when there is additional F&R SB content-- the 'Find' Deck title could change if needed to hold both content panels.
Comment 7 Jim Raykowski 2024-06-17 18:11:36 UTC
Created attachment 194785 [details]
with find bar 'Match Case' and F&R icon button and additional icon button for more search options

Maybe like what is shown in the attached?
Comment 8 V Stuart Foote 2024-06-17 20:55:32 UTC
(In reply to Jim Raykowski from comment #7)
> Created attachment 194785 [details]
> with find bar 'Match Case' and F&R icon button and additional icon button
> for more search options
> 
> Maybe like what is shown in the attached?

Sure, meaning no SB deck or content panel for the F&R dialog actions. Just the theme'd F&R icon for opening the full dialog.

Of course that would mean there is still no GUI representation of a "Find All" result (other than Calc's) with just the "Search results message" showing feedback in the dialog.

And other than Calc, wouldn't a SB content panel for graphical F&R search results be of some utility? Use the same framework as QFS is fine, just don't mix the search modes/configurations.
Comment 9 Jim Raykowski 2024-06-22 22:36:16 UTC
Created attachment 194913 [details]
Demo of search options dialog approach

Here's a demo of using a dialog for search options. Also shown in the demo is a change to make the QFS use the F&R attributes options settings. For unknown reason 'Match case' and 'Whole words only' settings from the QFS search options dialog are detected by the F&R dialog but not the Similarity settings.
Comment 10 V Stuart Foote 2024-06-23 01:03:49 UTC
(In reply to Jim Raykowski from comment #9)
> Created attachment 194913 [details]
> Demo of search options dialog approach
> 
> Here's a demo of using a dialog for search options. Also shown in the demo
> is a change to make the QFS use the F&R attributes options settings. For
> unknown reason 'Match case' and 'Whole words only' settings from the QFS
> search options dialog are detected by the F&R dialog but not the Similarity
> settings.

Thanks Jim. It is looking promising.

Have we decided then that the Find results deck is going to be common to the QFS *AND* the F&R dialog?  The Results panel is shaping up nicely to handle both, question is if the F&R settings should impact the FindBar/QFS--or not.

Seems a cleaner break can be made if not, with the FindBar/QFS offering limited options and the F&R dialog standing alone (even if its results end up delivered via the Find deck).
Comment 11 Jim Raykowski 2024-06-23 18:24:35 UTC
(In reply to Jim Raykowski from comment #9)
> ... Also shown in the demo
> is a change to make the QFS use the F&R attributes options settings.

This should have been: 

... Also shown in the demo is a change to make the QFS *NOT* use the F&R  attributes and format settings.
Comment 12 Jim Raykowski 2024-06-24 05:20:31 UTC
PS9 makes F&R dialog search settings not affect QFS searches and QFS search settings not affect searches made using the F&R dialog.
Comment 13 Commit Notification 2024-06-24 20:49:17 UTC
Jim Raykowski committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/601cf05456073c082ac6bbe43b501eb81256711b

tdf#160543 Quickfind sidebar enhancements

It will be available in 25.2.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 14 Buovjaga 2024-08-21 05:01:33 UTC
Should this be closed as fixed?