search

wzin / liquid-galaxy

Automatically exported from code.google.com/p/liquid-galaxy
Apache License 2.0
0 stars 0 forks source link

Legacy Touchscreen does not show/hide new groups #39

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
To reproduce:
1. Create a new touchscreen BookmarkGroup with a slug called something other 
than "earth", "moon", or "mars".
2. Try selecting its icon on the legacy touchscreen interface.
3. Curse when it does not appear and Javascript errors come out instead.

The toggleExpand() and noneExpand() JavaScript functions handle showing and 
hiding the touchscreen bookmark menus.  But they're woefully unable to handle 
menus for groups other than "earth", "moon", "mars", and "layers".

http://code.google.com/p/liquid-galaxy/source/browse/display_control/templates/t
ouchscreen_legacy.html?repo=lg-cms#29

http://code.google.com/p/liquid-galaxy/source/browse/display_control/static/touc
hscreen_legacy/javascript.js?repo=lg-cms#165

Please refactor this functionality any elegant and scalable way you see fit.

Also try adding a functional test so Selenium may confirm this works.

Original issue reported on code.google.com by AxisOfEn...@gmail.com on 27 Jun 2013 at 6:17

GoogleCodeExporter commented 9 years ago

Original comment by AxisOfEn...@gmail.com on 27 Jun 2013 at 6:21