Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

window focus listener not installed #169

Open
GoogleCodeExporter opened this issue May 4, 2015 · 1 comment
Open

window focus listener not installed #169

GoogleCodeExporter opened this issue May 4, 2015 · 1 comment

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. UnifiedToolBar toolBar =  UnifiedToolBar();
2. myComponent.add(toolBar);
3. start app

What is the expected output? What do you see instead?
WindowFocusListener should be added to toolBar's parent, causing toolBar to be 
painted in focused state upon start up.

 What do you see instead?
toolBar is painted in disabled or inactive state upon start up. in fact, it 
appears the WindowFocusListener never is added to toolBar's parent (repainting 
upon focus change is visibly choppy/delayed)

What version of the product are you using? On what operating system?
0.9.5 library, Mac OS X 10.5.8,  java 1.6.0_17

Please provide any additional information below.
Implementing this on my own, I found success when instead of using a 
weakReference to the WindowFocusListener when creating an AncestorListener, I 
used the WindowFocusListener (see WindowUtils.java)

I'm suspect of this bug, considering it would be obvious to anyone using your 
library and would have already been a tracked bug... can anyone reproduce this? 
I'm also don't understand why the weak reference is used.

finally, Thanks a lot for this great open source project, I have learned a lot 
by scouring your source code and it has helped a great deal on the project I 
was unfortunately far into before I discovered this project.

Thanks,
Seth

Original issue reported on code.google.com by [email protected] on 21 Aug 2010 at 11:47

@GoogleCodeExporter
Copy link
Author

As you suggest, the focus painting doesn't seem to work. I am attaching a patch 
I used to fix it in our environment.

Original comment by [email protected] on 17 Nov 2010 at 6:52

Attachments:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant