[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 --- Comment #6 from Gerrit Notification Bot gerritad...@wikimedia.org --- Change 87322 had a related patch set uploaded by Paladox: Added search and replace within selection functionality https://gerrit.wikimedia.org/r/87322 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 Gerrit Notification Bot gerritad...@wikimedia.org changed: What|Removed |Added Status|NEW |PATCH_TO_REVIEW -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 Derk-Jan Hartman hartman.w...@gmail.com changed: What|Removed |Added Blocks||63665 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 --- Comment #4 from k...@heldig.org --- Well, i did your test. and it works fine. * I pasted 12-12-12 in the texarea * I selected the first 12-12 * I searched for 12 and replaced with 3. * I hit find next and replace, causing it to change to 3-12-12 * I hit find next and replace, causing it to change to 3-3-12 * I hit find next and replace, causing it to give me the messsage: no more occurences found * I then reset the textbox and did the same with replace all, which gave the result 3-3-12 Feel free to test it out yourself -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 --- Comment #5 from k...@heldig.org --- 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. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 --- Comment #2 from k...@heldig.org --- (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. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 --- Comment #3 from TMg mr.h...@gmx.de --- (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: * Paste the text 12-12-12 in the textarea. * Select the first 12-12-. * Search for 12 and replace it with 3. * Hit Find next and Replace. Should become 3-12-12. * Repeat. Should become 3-3-12. * Repeat. Should stay (!) 3-3-12. Same test with Replace all. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 TMg mr.h...@gmx.de changed: What|Removed |Added CC||mr.h...@gmx.de --- Comment #1 from TMg mr.h...@gmx.de --- 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: * 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) -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 54961] Added search and replace within selection functionality
https://bugzilla.wikimedia.org/show_bug.cgi?id=54961 Andre Klapper aklap...@wikimedia.org changed: What|Removed |Added Priority|Unprioritized |Lowest -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l