diff options
author | Tor Norbye <tnorbye@google.com> | 2012-06-06 12:27:20 -0700 |
---|---|---|
committer | Tor Norbye <tnorbye@google.com> | 2012-06-06 12:27:20 -0700 |
commit | c5d16e5c345333414a416a8fb63fc74bd5e8e40a (patch) | |
tree | 846d77af3d69517cf9cb61e7737d0be627083a7a /eclipse/dictionary.txt | |
parent | 455038d1740083aa5e2b5b0ec852cd81bcc9ce1e (diff) | |
download | sdk-c5d16e5c345333414a416a8fb63fc74bd5e8e40a.zip sdk-c5d16e5c345333414a416a8fb63fc74bd5e8e40a.tar.gz sdk-c5d16e5c345333414a416a8fb63fc74bd5e8e40a.tar.bz2 |
Workaround for Eclipse 4.x focus issue
In Eclipse 4, the system calls setFocus on the outline view *after* a
mouse click inside the outline has been processed. This ends up
calling setFocus on the treeviewer. If you clicked in the property
sheet to for example start editing a text property, this means the
focus loss will cause the property editor to abort editing again. The
net result is that you always need to click twice to start editing
properties if the Outline view does not already contain focus.
This changeset adds a workaround for this. It has two parts.
First, simply make the OutlinePage's setFocus() method do nothing if
it's called during a mouse click in the property sheet (which we can
track with a mouse listener).
The second part is trickier. The outline view itself (which is
provided by Eclipse and we have no hooks into it) will be called
first, and it calls setFocus() on its first child, which is a
composite, so it ends up searching down for the first focusable
component. The workaround is to add our own widget as the first
child, and make it declare that it's focusable but make the setFocus
call a no-op. Second, make the widget invisible since we don't want
anything but the tree viewer in the outline.
Change-Id: Ib89943c410bce599723f59a4f202f339adab6327
Diffstat (limited to 'eclipse/dictionary.txt')
-rw-r--r-- | eclipse/dictionary.txt | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/eclipse/dictionary.txt b/eclipse/dictionary.txt index dbb51e5..f0f43de 100644 --- a/eclipse/dictionary.txt +++ b/eclipse/dictionary.txt @@ -186,6 +186,7 @@ ninepatch nodpi num ok +op os palette param |