Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 345033

Summary: [client] Improve the find feature in the editor
Product: [ECD] Orion Reporter: Felipe Heidrich <eclipse.felipe>
Component: EditorAssignee: libing wang <libingw>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: bokowski, libingw, mamacdon, susan
Version: 0.2Keywords: helpwanted
Target Milestone: 0.3   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Felipe Heidrich CLA 2011-05-06 16:35:55 EDT
I think the find (ctrl+f) could be better. Personally I would like to see mmost of the features we have in the Java Editor in Eclipse available to the Orion Editor.


- Do not close find dialog automatically when the user start the search
- Find / Replace (Bug 344071) 
- Backward and Forward search
- Case sensitive on /off
- Whole word on / off
- Wrap search on / off
Comment 1 Mark Macdonald CLA 2011-05-06 17:02:55 EDT
> - Do not close find dialog automatically when the user start the search
> - Case sensitive on /off
> - Whole word on / off
> - Wrap search on / off
I agree, and I think we need a proper Find dialog where you can set these options. We should also have a "Regex on/off" option, since it is awkward to type in a JS-style /regex/flags literal.

> - Backward and Forward search
We do actually support this (Ctrl+K and Ctrl+Shift+K) but you'd never find it unless you knew the hotkeys.
Comment 2 Boris Bokowski CLA 2011-05-06 21:16:10 EDT
If we're implementing this, we should try to avoid a dialog and use a more subtle UI like in Firefox or Chrome.
Comment 3 Felipe Heidrich CLA 2011-05-08 21:34:05 EDT
I agree with Mark and Boris commints, should add

>> - Backward and Forward search
>We do actually support this (Ctrl+K and Ctrl+Shift+K) but you'd never find it
unless you knew the hotkeys.

You can (and should) keep the shortcut  (Ctrl+K and Ctrl+Shift+K), Backward and Forward buttons in the dialog can apply only for the default button of the dialog.


Boris:
Yes, remove the dialog that is being used now. Just don't be too
subtle. I understand that is personal test, but in my opnion
--our current incremental search is too subtle (hard to read/see)
--the way safari implemented their search I think it is a great example how to be subtle and high usability.
Comment 4 libing wang CLA 2011-10-20 16:40:24 EDT
already fixed.