search

josmas / openwonderland

Automatically exported from code.google.com/p/openwonderland
GNU General Public License v2.0
3 stars 5 forks source link

hud visibility management inconsistencies #174

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
This is about huds that may be enabled via
a menu item on the window menu.

There is inconsistent behavior when
checking or unchecking the window menu item:

UserListHud:
Minimizing the user list hud unchecks the
checkbox of the menu item.
When you maxime the hud by clicking on its icon
the hud appears, but the menu item remains unchecked.
When you instead check the menu item you end up with both
the icon and the visible hud.

TextChatHud:
Minimizing the text chat hud leaves the
corresponding menu item checked.
Maximizing the hud makes the hud visible
and leaves the icon visible as well.

I think the HUDComponentManager classes are responsible
for this behavior.

The expected behavior is up for discussion at
http://groups.google.com/group/openwonderland/browse_thread/thread/ddfee5f7c8b73
2ff

Original issue reported on code.google.com by objectle...@gmail.com on 28 Feb 2011 at 9:05

GoogleCodeExporter commented 9 years ago 
Attached are modifications in order to achieve the following behavior:
When a HUD is minimized the corresponding menu item should be unchecked.
Checking a HUD menu item makes it appear at the last position incuding
maximizing a minimized HUD.

Note: We realized this behavior is the responsibility of the HUDPlugIns
and not of the HUD management classes in the core.

I tried to fix this for all HUDs in the core that are reachable via a MenuItem.
Please let me know if I missed something:

project wonderland: UserList, Audio Status, TextChat, Gestures, Shortcuts

project wonderland-modules: Quick Reference

Original comment by objectle...@gmail.com on 6 Mar 2011 at 10:06

Attachments:

GoogleCodeExporter commented 9 years ago 
Here is a cumulative patch for PaletteClientPlugin
that leaves the execution of the action listener execute in the AWT EventThread.

Original comment by objectle...@gmail.com on 7 Mar 2011 at 8:38

Attachments:

GoogleCodeExporter commented 9 years ago 
this also fixes issue #91

Original comment by objectle...@gmail.com on 8 Mar 2011 at 9:15

GoogleCodeExporter commented 9 years ago 
Issue 91 has been merged into this issue.

Original comment by jonathan...@gmail.com on 21 Mar 2011 at 4:45

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r4667.

Original comment by jonathan...@gmail.com on 21 Mar 2011 at 4:49