Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 102911 - [KeyBindings] assist: Not correctly sized when window is narrow or short
Summary: [KeyBindings] assist: Not correctly sized when window is narrow or short
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.1   Edit
Hardware: All All
: P3 trivial (vote)
Target Milestone: 3.2 M1   Edit
Assignee: Douglas Pollock CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-06 14:43 EDT by Michael Van Meekeren CLA
Modified: 2005-08-09 11:54 EDT (History)
2 users (show)

See Also:


Attachments
small popup (9.68 KB, image/png)
2005-07-06 14:44 EDT, Michael Van Meekeren CLA
no flags Details
other dropdowns (112.78 KB, image/png)
2005-07-07 16:31 EDT, Michael Van Meekeren CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Van Meekeren CLA 2005-07-06 14:43:58 EDT
see attached image.  If i move the shell to the top of the screen and make it
narrow the key assist is a small shell, but on left right, bottom it works fine.
Comment 1 Michael Van Meekeren CLA 2005-07-06 14:44:46 EDT
Created attachment 24393 [details]
small popup
Comment 2 Douglas Pollock CLA 2005-07-06 15:42:20 EDT
If the window itself is small, then the key assist will open small.  This is
intentional.  It is intended to minimize the amount of the workbench window it
can occupy.

The only situation I can see this being a problem is if the key assist were to
appear in a small RCP application ... like an instant messaging client.  The
menu item (Help > Key Assist...) is already limited to IDE applications.  I will
move the key binding to IDE as well.  This means that an RCP application will
only really get the assist showing if they bind it themselves, use multi-stroke
bindings or the user creates their own binding to it.  I believe these three
cases are fairly indicative of a larger window.

I'm pretty sure this isn't exactly what you guys are hoping for, but this seems
like a pretty extreme case.  Without concrete use cases as to why this would be
a problem, it would be hard to adjust the sizing algorithm.
Comment 3 Michael Van Meekeren CLA 2005-07-06 16:05:22 EDT
I'd suggest that additionally it should have a minimum size of 1 or 2 rows so
that it never opens in a completely useless state as in the image.
Comment 4 Douglas Pollock CLA 2005-07-07 10:21:18 EDT
Can you show me an image of a usable workbench window where one line does not
appear in the key assist dialog?
Comment 5 Michael Van Meekeren CLA 2005-07-07 16:31:54 EDT
Created attachment 24441 [details]
other dropdowns

I guess my comment is that if your drop down is not going to be resizable (as
many others are) then it should never be a size that looks goofy and is not
usable.
Comment 6 Nick Edgar CLA 2005-07-12 10:09:25 EDT
As part of this change, the showKeyAssist command was moved up to
org.eclipse.ui.ide.  Just wondering why?
Comment 7 Douglas Pollock CLA 2005-07-12 11:30:09 EDT
The show key assist command was not moved, only the key binding.  Please see 
comment #2. 
Comment 8 Douglas Pollock CLA 2005-08-09 11:54:11 EDT
Verified that the show key assist command exists in "org.eclipse.ui" and that 
the binding and menu item appear in "org.eclipse.ui.ide"