fnagel/jquery-ui — Issue #175: Discussion about additional wrapperElement required
Author: thg2k
Created: 2011-10-14 02:55:25+00:00
State: open

Hi fnagel, I'm using your selectmenu widget on a production website.

After 4 months I came back to your repository to merge latest changes, but the current version (ok it says "dev version" in the header but still..) completely borked my web site, so I had to revert to the previous version from around june 2011.

My main problem now is that you added this "wrapperElement" option which defaults to "

", but there is no easy way to remove it.
I have a lot of selectmenus around my website and I don't need to add additional complexity to the markup so I wanted to disable it, but the way your code currently handles it makes it difficult to remove it.

So my question in the end is.. do you have any real testcase why this option is needed? If I want to wrap I can just wrap my , then when I call selectmenu() it will .insertAfter() in the right place so the result is the same... plus the menu is already wrapper inside as , so I really don't see any need to add an additional

.

Also second question, do you think it's time to maintain a stable branch? With the current state I cannot afford to merge it in my production site if I have to spend several hours debugging problems each time :(