Bug 147566 - Isolate Quick search (Ctrl+F) bar from search formatting applied as result from a prior Find&Replace (Shft+Ctrl+F action)
Summary: Isolate Quick search (Ctrl+F) bar from search formatting applied as result fr...
Status: RESOLVED DUPLICATE of bug 115665
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
7.1.8.1 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Find-Search Find&Replace-Styles
  Show dependency treegraph
 
Reported: 2022-02-21 07:55 UTC by Anduril
Modified: 2024-11-02 17:31 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
searchslot (9.84 KB, image/png)
2022-02-21 07:55 UTC, Anduril
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Anduril 2022-02-21 07:55:12 UTC
Created attachment 178424 [details]
searchslot

Actual behaviour
If the user applies a formatting to the search in the search and replace window (Cmd Shift F) and then later on makes just a search (in the same or in another document) with the search slot (Cmd F) (screenshot attached) the formatting applied to the search is retained, but the user is not warned resulting in unwanted results.

Expected behaviour
When user applies a formatting to the search in the search and replace window and then later on makes just a search (in the same or in another document) with the search slot, the user should be warned that formatting (which would be a plus) has been applied and be able to un-apply it.

Reproduce
Follow the steps in Actual behaviour
Comment 1 Heiko Tietze 2022-03-14 09:18:53 UTC
I understand this as a bug since no F&R settings should be taken into the quick find search.

Steps to reproduce:
* Insert dummy text like "He heard quiet steps behind him. He heard quiet steps behind him."
* change the font color of one item like "He" in Dark Red 2.
* Search per F&R for the term with the format font color = dark red 2
  -> only one "He" will be found
* close and open the quick find, search for "He"
  -> only the one "He" will be found that is colored

Stuart, Cor: I remember discussions about this in the past. Anything I forgot to mention? Is it maybe a duplicate?
Comment 2 V Stuart Foote 2022-03-14 15:01:00 UTC
Hmm, confirm the issue. Format settings made in the <Ctrl>+H 'Find & Replace' dialog bleed over to subsequent searches from the <Ctrl>+F 'Find bar'

Not sure that work on bug 62601 (and dupes) were really correctly resolved.

But in addition to affecting the <Ctrl>+F 'Find bar' it also now impacts the Navigator's new 'Repeat search' mode. Which I guess should be expected--since the Navigator's instance is same framework as the Find bar's.

Intent of bug 62601 was to keep the Find bar simple and light weight (effectively make use of transformations/regex to return generalized results) while Find & Replace would allow greater control.

Would expect the Navigator's repeat search should take on the greater control of settings from F&R dialog, as it does.

What is askew is that the Find bar's generalized search is responding to settings made in the F&R dialog. But secondly that there is no indication that settings from F&R are active. 

Given common framework (Find toolbar, and Navigator deck) We may not be able to make the Find bar truly a generalized search, but should then at least provide an indicator of F&R settings being applied.

@Jim, any thoughts?
Comment 3 Jim Raykowski 2022-03-15 02:32:30 UTC
https://gerrit.libreoffice.org/c/core/+/131569

(In reply to V Stuart Foote from comment #2)
> Hmm, confirm the issue. Format settings made in the <Ctrl>+H 'Find &
> Replace' dialog bleed over to subsequent searches from the <Ctrl>+F 'Find
> bar'
The patch makes for only non formatted search from Find bar.
> But in addition to affecting the <Ctrl>+F 'Find bar' it also now impacts the
> Navigator's new 'Repeat search' mode. 
The patch makes 'Repeat search' follow the formatting of the previous search, so done after a formatted search from the F&R dialog it will also do a formatted search. When done after a search from the Find bar, made always a non formatted search with this patch, it will also do a non formatted search.
Comment 4 Anduril 2022-03-16 15:00:37 UTC
Hi everybody,
could a warning that formatting is on such as 
- a background color (orange?) in the find bar plus 
- a check followed by 
  - text summing up what formatting is being applied be a solution that keeps current improvements, but makes applied changes to the search field visible?

Users would then be able to uncheck the check and the background color would go back to white for ex.
Comment 5 Anduril 2022-03-16 15:01:03 UTC Comment hidden (obsolete)
Comment 6 Anduril 2022-03-19 18:04:04 UTC
Alternatively the search and replace window and the find tab could work independently: the former would retain the applied changes, while the latter would only perform simple searches, without inheriting the formatting of the former.
Comment 7 Jim Raykowski 2022-03-19 21:56:40 UTC
(In reply to Anduril from comment #6)
> Alternatively the search and replace window and the find tab could work
> independently: the former would retain the applied changes, while the latter
> would only perform simple searches, without inheriting the formatting of the
> former.

That's exactly what the patch does :)

I've been trying to figure out why it fails a uitest on the remote build but not local :(
Comment 8 Mora Jolley 2022-07-20 07:06:54 UTC Comment hidden (spam)
Comment 9 peterloyed 2022-11-15 06:38:36 UTC Comment hidden (spam)
Comment 10 Kathy Barrera 2023-06-02 07:20:19 UTC Comment hidden (spam)
Comment 11 Daniele 2023-06-02 14:01:46 UTC
(In reply to Jim Raykowski from comment #7)

Thanks Jim Raykowski for explaining. This solution is just perfect!
Comment 12 samy 2023-06-06 17:24:45 UTC Comment hidden (spam)
Comment 13 robitony 2023-12-06 04:35:05 UTC Comment hidden (spam)
Comment 14 ahmedabrahim 2024-10-02 14:27:30 UTC Comment hidden (spam)
Comment 15 Cor Nouws 2024-10-09 09:14:23 UTC
(must be a duplicate..)
Comment 16 V Stuart Foote 2024-10-09 12:01:06 UTC
(In reply to Cor Nouws from comment #15)
> (must be a duplicate..)

Issue of OP was => WF, i.e. provide no "warning" of a bleed-over of find configs. 

Instead behavior between the search widgets seems correctly isolated now as suggested in comment 6, and to finish what was bug 62601 and agreed by UX-design around bug 72080, with Jim's commit https://gerrit.libreoffice.org/c/core/+/131569

I would say this can be resolved fixed for a 25.2 release. The 'Find bar' quick search, and the 'Find and Replace' dialog now isolated, but one or the other controlling the SB Navigator "Repeat search" behavior.

Some consideration probably now needed as to search configs of the *new* Sidebar Find deck--which currently defaults to a simple string search for its hits.

@Jim, Ok to close? Or were there remaining glitches re your comment 7?
Comment 17 V Stuart Foote 2024-10-09 12:09:23 UTC
(In reply to V Stuart Foote from comment #16)
> UX-design around bug 72080, with Jim's commit
> https://gerrit.libreoffice.org/c/core/+/131569

That shows abandoned, actual patch was submitted against bug 115665 "Format in the CTRL+H search affects the CTRL+F search"

https://gerrit.libreoffice.org/c/core/+/169606

*** This bug has been marked as a duplicate of bug 115665 ***
Comment 18 V Stuart Foote 2024-10-09 12:15:27 UTC
(In reply to V Stuart Foote from comment #17)
>... against bug 115665 "Format
> in the CTRL+H search affects the CTRL+F search"
> 
> https://gerrit.libreoffice.org/c/core/+/169606
> 

and https://gerrit.libreoffice.org/c/core/+/168834
Comment 19 ahmedabrahim 2024-11-02 17:31:03 UTC Comment hidden (spam)