Author: kim
Description:
screenshot of functionality
https://gerrit.wikimedia.org/r/#/c/87322/
This enables users to first select some text in the textarea, and
click the checkbox labeled "Search/replace within text selection"
when searching / replacing text.
When this checkbox is checked, the text searched/replaced will a subset
of the original text determined by the user's text selection.
Version: unspecified
Severity: enhancement
Attached:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Added "search and replace within selection" functionality | mediawiki/extensions/WikiEditor | master | +31 -16 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T65665 WikiEditor: Search and replace (tracking) | |||
| Duplicate | None | T56961 Added "search and replace within selection" functionality |
mr.heat wrote:
Don't get me wrong: I like the feature. But I think the checkbox is odd.
What happens if nothing is selected? The checkbox must be hidden or at least disabled (grayed out). I guess nothing is selected 99% of the time and therefor the checkbox is useless 99% of the time.
What happens if I click "Find next" and then decide to enable the "within selection" checkbox? "Find next" changes the selection.
As I said I have a similar feature in my [[dewiki:User:TMg/autoFormatter]]: If something is selected it uses the selection. If nothing is selected it uses the full textarea. From what I heard from my users this works as expected.
A simple way to make this visible to the user is to change the title of the popup from "Search and replace" to "Search and replace in selection". Maybe change the button from "Replace all" to "Replace all in selection".
Related:
kim wrote:
(In reply to comment #1)
Don't get me wrong: I like the feature. But I think the checkbox is odd.
What happens if nothing is selected? The checkbox must be hidden or at least
disabled (grayed out). I guess nothing is selected 99% of the time and
therefor
the checkbox is useless 99% of the time.What happens if I click "Find next" and then decide to enable the "within
selection" checkbox? "Find next" changes the selection.
This works as expected as i only fetch the selection upon opening the dialog, changes to the selection when the dialog is open does not have any effect.
As I said I have a similar feature in my [[dewiki:User:TMg/autoFormatter]]:
If
something is selected it uses the selection. If nothing is selected it uses
the
full textarea. From what I heard from my users this works as expected.A simple way to make this visible to the user is to change the title of the
popup from "Search and replace" to "Search and replace in selection". Maybe
change the button from "Replace all" to "Replace all in selection".Related:
- Bug 40797 (duplicate, but I'm not sure which one to close)
- Bug 23942 (can't have "search and replace" and "insert internal link
brackets
without showing a popup" buttons the same time)
- Bug 23992 ("search and replace" broken in Opera 12)
As your request, i removed the checkbox and added a new title to the dialog if
there is a selection of thext when clicking on the search and replace functionality.
Did however not change any button texts, as this seems to cause very long button names, where the dialog is quite small. And it strictly isn't necessary when this is shown in the title.
mr.heat wrote:
(In reply to comment #2)
i only fetch the selection upon opening the dialog
My question is: Isn't this confusing? I can see that the selection changes in the background. When I close the dialog the changed selection stays changed. My original selection is gone. This makes it look like the function behaves different when I close and reopen the dialog and when I keep it open. Same selection seen in the background but different behavior.
Please do the following test:
Same test with "Replace all".
kim wrote:
Well, i did your test. and it works fine.
Feel free to test it out yourself
kim wrote:
I dont reckon this as confusing, i mean if your doing a find operation, then you need to set a new selection for the found text occurrence. But i dont think this functionality was designed specifically for the find functionality. more for the replace functionality.
Change 87322 had a related patch set uploaded by Paladox:
Added "search and replace within selection" functionality
Change 87322 had a related patch set uploaded (by Paladox):
Added "search and replace within selection" functionality