Feedback on FloatingMenuSkin

There are many improvements that could be made to this skin since it makes many assumptions about TWiki paths like:

• assuming that if you specify Index as a menu item that it links to WebIndex
• assuming that all TWiki webs, like Sandbox all have WebHome as their home page.

1. Support user defined multi layer menus (not just the TWiki web menu)
2. Allow font specifications (color, size, type, etc.) to be specified

1. to minimize net traffic by putting 'static' content into files instead of hard coding all of the JavaScript in the template file. To do this and allow dynamic content, PHP was needed.
2. easier to perform some of the decisions in PHP rather than JavaScript. Maybe someone that is more fluent in JS could write a modification to this plugin that would be 100% JS.

-- TaitCyrus - 01 Sep 2003

Floating menues are interesting, they save screen space and give a familiar look to GUI apps.

Please note that a PHP-enabled TWiki site could be a major security hole since any user could insert PHP code in a TWiki topic and gain access to everything a PHP programmer has.

-- PeterThoeny - 02 Sep 2003

Hmmm. Good point! I guess I should move the php into the cgi-bin directory assuming that the pub directory is configured to not allow php to be run from there (since a person could 'attach' php files).

-- TaitCyrus - 08 Sep 2003