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

Bug 362421

Summary: [QuickAccess] Regression: "Quick access" panel lacks major usability features from 3.x stream
Product: [Eclipse Project] Platform Reporter: Andrey Loskutov <loskutov>
Component: UIAssignee: Dean Roberts <dean.t.roberts>
Status: VERIFIED FIXED QA Contact: Remy Suen <remy.suen>
Severity: major    
Priority: P3 CC: dean.t.roberts, pwebster, remy.suen
Version: 4.2Flags: pwebster: review+
Target Milestone: 4.2 M4   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 363582    
Bug Blocks: 320673    
Attachments:
Description Flags
Remember size after re-size
none
Screen shot showing default QuickAssist size
none
Screenshot of quick access on 3.7
none
Screenshot of quick access on 4.2 none

Description Andrey Loskutov CLA 2011-10-30 03:35:13 EDT
Build Identifier: Build id: I20111028-1100

AS IS:
1) Start to type into "Quick access" area. A table with possible matches will appear. It is usually too narrow, so you have to resize it to see the content. Unfortunately, the new size is not remembered, so you have to do it each and every time.

2) Start to type into "Quick access" area.  A table with possible matches will appear. It is opened ALWAYS as a small box on the right side of the window. User can NOT move it to any other location. On large screens, it should appear *centered*, or at least allow to move it and remember the position after moving for the next time.

3) There is no "Previous choices" in the "Quick access" area. In 3.8, after pressing <Ctrl+3>, "Previous choices" was shown in the "Quick access" area, letting user quickly select most recently used view/command. Now there is NO such feature anymore. 

4) Type <Ctrl+Space> into "Quick access" area (this was my natural attempt to workaround issue 3). You would now expect kind of content assist, showing "Previous choices". Nope. Nothing happens. Even worse, if at the same time a Java editor is active, it will show you *Java editor* content assist. This is simply a bug.

TO BE:
Please give either the old "Quick assist" view back or provide all of its features on the new one.

Reproducible: Always
Comment 1 Paul Webster CLA 2011-11-03 14:33:35 EDT
#4 is covered by another bug, and #2 it must be opened under the quick access search field.

But it should be large enough, we should save the size if it is resized, and we definitely want the Previous Choices back.

PW
Comment 2 Andrey Loskutov CLA 2011-11-03 15:13:34 EDT
(In reply to comment #1)
> But it should be large enough, we should save the size if it is resized

I'm just wondering what is the reason to include this as a *permanent* search field on the *main* toolbar? It just doesn't fit there, or at least not as a *permanent* text field. You can NOT make it larger as now, as then you will waste even more toolbar space.

Why not to show/hide this field, like Opera/Firefox does it for the in-page text search field (in Opera: F3, in Firefox: Ctrl+F)? This way you also will get more place for additional option checkboxes etc.

BTW, if somebody decided that Ctrl+3 popup was not good enough for some reasons, could one please explain me, how the new solution (including all it's new bugs and missing features) is better then the old one? Why one need to re-invent the wheel again and spend time and resources for it, if it was already there, *working just fine*? I can't really believe that there were no other more important and/or broken things in Eclipse to work on.
Comment 3 Paul Webster CLA 2011-11-03 16:06:44 EDT
(In reply to comment #2)
> You can NOT make it larger as now, as then you will
> waste even more toolbar space.

My reference to large enough was in response to your comment about when the dropdown list opened, it was too small.  The dropdown list should be reasonably large to begin with, and if enlarged by the user that should be remembered.

Not making the search box itself larger.

PW
Comment 4 Dean Roberts CLA 2011-11-11 10:08:40 EST
Created attachment 206850 [details]
Remember size after re-size

Attaching a patch that will remember the size after a re-size.  The size is persisted across workbench invocations.

I also modified some multiple monitor code to get better behaviour for the resized window on multiple monitor setups.

When applying this patch, you should make sure the patch for bug 363582 is applied first, to avoid patch mismatch problems.
Comment 5 Dean Roberts CLA 2011-11-11 10:13:43 EST
Created attachment 206851 [details]
Screen shot showing default QuickAssist size

Item #1 also mentioned that the original window size is too narrow.  I've attached a screen shot of the original window size on my system.  At least on my system, I find the initial size acceptable.  Are you experiencing something different?
Comment 6 Dean Roberts CLA 2011-11-11 10:18:58 EST
For item #3, I see previous choices when I invoke the QuickAccess either via Ctrl+3 or typing directly in the search bar.

The previous choices will appear initially, but will disappear once you start typing, if they don't match the text you have typed so far.  I believe this is desired and expected behaviour.
Comment 7 Andrey Loskutov CLA 2011-11-11 14:15:17 EST
Created attachment 206867 [details]
Screenshot of quick access on 3.7
Comment 8 Andrey Loskutov CLA 2011-11-11 14:22:24 EST
Created attachment 206870 [details]
Screenshot of quick access on 4.2

You have right, don't know how I've tested before, but now I see the "Previous choices" too.

Regarding the appearance / size - just compare two screenshots made on the same PC. The quick access from 3.7 is more useful/ergonomic, as it is *much bigger* and appears *centered* on the wide screen. 

NO ONE expects to see a "choice dialog" in the top right corner. Have you ever asked why all this "error/info/question/go to" dialogs are centered? That's simple, because user's attention is there.
Beside this, current (4.2) state is simply inconsistent to the rest of the "go to" dialogs. Open "Open resource / Type" dialogs. Do they appear in the upper right corner and have such small size? Why not?
Comment 10 Dean Roberts CLA 2011-11-28 08:36:02 EST
#1 is fixed by attached (and released) patch
#2 works as intended
#3 works for me ... IE pervious choices are displayed
#4 is covered by another bug (as per comment 2)

So closing this defect as fixed
Comment 11 Dean Roberts CLA 2011-12-06 13:08:37 EST
Verified on I20111205-2330