Bug 530767 - Find/Replace dialog should be wider and should arrange its buttons in one row
Summary: Find/Replace dialog should be wider and should arrange its buttons in one row
Status: CLOSED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.7.1   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: Documentation
Depends on:
Blocks:
 
Reported: 2018-02-06 04:42 EST by Lars Vogel CLA
Modified: 2021-06-24 05:21 EDT (History)
9 users (show)

See Also:


Attachments
Screenshot (33.19 KB, image/png)
2018-02-06 04:42 EST, Lars Vogel CLA
no flags Details
Old Button position (410.27 KB, image/png)
2018-02-21 15:02 EST, Lars Vogel CLA
no flags Details
New buttons (174.26 KB, image/png)
2018-02-21 15:03 EST, Lars Vogel CLA
no flags Details
Screenshot (120.92 KB, image/png)
2018-02-22 04:28 EST, Lars Vogel CLA
no flags Details
New Layout with a left alignment (18.34 KB, image/png)
2019-09-03 08:28 EDT, Jonas Hungershausen CLA
no flags Details
Both layouts on Mac (297.17 KB, image/png)
2019-09-17 05:21 EDT, Karsten Thoms CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2018-02-06 04:42:23 EST
Created attachment 272552 [details]
Screenshot

As a long term goal, I would like to change the Find/Relace dialog to be inline, e.g., not blocking.

To make it already a bit better (less blocking), I suggest to make it wider and put all buttons (except close) into one row.

To open the dialog press Ctrl+F in a text editor.
Comment 1 Lars Vogel CLA 2018-02-06 04:42:51 EST
Jonas, can you give it a try? IIRC Repo is eclipse.platform.text
Comment 2 Eclipse Genie CLA 2018-02-21 12:22:10 EST
New Gerrit change created: https://git.eclipse.org/r/117889
Comment 3 Dani Megert CLA 2018-02-21 12:27:01 EST
(In reply to Lars Vogel from comment #0)
> Created attachment 272552 [details]
> Screenshot
> 
> As a long term goal, I would like to change the Find/Relace dialog to be
> inline, e.g., not blocking.

What do you mean by "blocking"? Making it wider even hides more stuff from the underlying editor, so, not sure why that should be better.
Comment 4 Lars Vogel CLA 2018-02-21 15:02:30 EST
Created attachment 272799 [details]
Old Button position
Comment 5 Lars Vogel CLA 2018-02-21 15:03:04 EST
Created attachment 272800 [details]
New buttons
Comment 6 Lars Vogel CLA 2018-02-21 15:07:09 EST
Thanks Jonas, to me it looks better and by beeing wider instead of longer, I can see more code. 

Can you try to move more options into the lines? This should give us also more space.

@Dani, we are currently still experimenting.
Comment 7 Lars Vogel CLA 2018-02-21 15:13:46 EST
Jonas, as an experiment you could also try to extend PopupDialog instead of Dialog and see if that saves more space.
Comment 8 Dani Megert CLA 2018-02-22 04:23:51 EST
(In reply to Lars Vogel from comment #6)
> @Dani, we are currently still experimenting.

OK, but what do you mean by blocking? The dialog is not modal.
Comment 9 Lars Vogel CLA 2018-02-22 04:28:17 EST
Created attachment 272808 [details]
Screenshot

> OK, but what do you mean by blocking? The dialog is not modal.

It is blocking the users view by opening a dialog. IntelliJ does it much nicer, but having the search inline in the text view (see screenshot).
Comment 10 Jonas Hungershausen CLA 2018-02-23 09:40:43 EST
(In reply to Lars Vogel from comment #7)
> Jonas, as an experiment you could also try to extend PopupDialog instead of
> Dialog and see if that saves more space.

I tried that, but there are *a lot* of methods missing in PopupDialog that are used and overridden in the class itself. I couldn't figure out what the alternatives were. 

> Can you try to move more options into the lines? This should give us also more  > space.

What do you mean by this? More of the checkboxes like the Options section?
Comment 11 Lars Vogel CLA 2019-09-03 08:24:57 EDT
Matthias, WDYT? I think the new layout looks way better.
Comment 12 Jonas Hungershausen CLA 2019-09-03 08:28:26 EDT
Created attachment 279756 [details]
New Layout with a left alignment
Comment 13 Pierre-Yves Bigourdan CLA 2019-09-13 12:21:47 EDT
The new layout looks better in my opinion as well, however, completely inlining the search would indeed be the ideal end goal.
Comment 14 Karsten Thoms CLA 2019-09-17 05:21:47 EDT
Created attachment 279903 [details]
Both layouts on Mac

I'm undecided. The new look is more clearer, but consumes more valuable space. I tend slightly for the new layout.
Comment 15 Julian Honnen CLA 2019-09-17 06:40:42 EDT
Random thought: How about removing the close button and moving Find etc. down into the button bar?

Besides the shell's close button and Esc, it seems redundant.
Comment 16 Matthias Becker CLA 2019-09-17 07:00:42 EDT
(In reply to Julian Honnen from comment #15)
> Random thought: How about removing the close button and moving Find etc.
> down into the button bar?
> 
> Besides the shell's close button and Esc, it seems redundant.

Mh. Sound's like something we should try. Why not.
Comment 17 Karsten Thoms CLA 2019-09-17 07:05:13 EDT
(In reply to Julian Honnen from comment #15)
> Random thought: How about removing the close button and moving Find etc.
> down into the button bar?
> 
> Besides the shell's close button and Esc, it seems redundant.

I think it is quite common to have a Close/Finish/... button to close dialogs. I agree that normal users know the close icon and ESC. But do you know any dialog that behaves such way already?
Is there some guideline for dialogs?
Comment 18 Julian Honnen CLA 2019-09-17 07:19:52 EDT
(In reply to Karsten Thoms from comment #17)
> I think it is quite common to have a Close/Finish/... button to close
> dialogs. I agree that normal users know the close icon and ESC. But do you
> know any dialog that behaves such way already?
> Is there some guideline for dialogs?

It's irregular, sure. But I also can't think of any other non-modal, on top dialog. So we might have a case for breaking the rules ;)
Comment 19 Till Brychcy CLA 2019-09-17 07:45:51 EDT
Hmm I haven't tried, but it looks like this massively increases the dialog's minimum width. 

That would be bad for me, because I like to place the replace dialog on the right side of the editor, but still want so see as much outline as possible.

Can you make it "floating", so if the make the dialog narrow, the current layout appears, but if you make it wide, your suggested layout appears?

I don't like IntelliJ's solution because it wastes a lot of vertical space (which is scarcer than horizontal space with today's wide screen monitors)
Comment 20 Karsten Thoms CLA 2019-09-18 05:14:16 EDT
(In reply to Till Brychcy from comment #19)
> Can you make it "floating", so if the make the dialog narrow, the current
> layout appears, but if you make it wide, your suggested layout appears?

I like the idea of that responsiveness. But might not be obvious that the dialog could be enlarged, and who would do so? And "just" to have a nicer layout of buttons? At least it must be remembered then which dialog size the user set.
Comment 21 Julian Honnen CLA 2019-09-18 05:38:14 EDT
+1 for a dynamic layout

(In reply to Karsten Thoms from comment #20)
> I like the idea of that responsiveness. But might not be obvious that the
> dialog could be enlarged, and who would do so? And "just" to have a nicer
> layout of buttons? At least it must be remembered then which dialog size the
> user set.

The dialog already remembers its size and position.
Comment 22 Alexander Kurtakov CLA 2019-11-14 03:10:40 EST
What is the status here?
Comment 23 Dani Megert CLA 2019-11-14 12:19:12 EST
I will have to see how the result looks. I am against a too wide Find/Replace dialog.
Comment 24 Mickael Istria CLA 2021-06-24 05:21:11 EDT
This change isn't perceive as a consensual improvement, and is seen as a regression for some. That added-value is not obvious. I don't think it's worth pushing it forward.
(FWIW, I also prefer a narrow dialog for find/replace so I can move it over the project explorer and see the whole editor; a wider dialog would most likely hide a part of the editor)

> As a long term goal, I would like to change the Find/Relace dialog to be inline, e.g., not blocking.

That's a totally different topic and not a reason to just change the dialog here.

> +1 for a dynamic layout

Although it's not related, that's not what this report and the attached patch are about. So I'm going to close those and let anyone willing to influence (or even code) in favor of such responsive dialog layout do it in a separate ticket.