If you disable javascript support in your browser and load the node add page you can see a
<div class="vertical-tabs clear-block"> </div>
See the attached screenshot. I have not verified if this is also in D7.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | vertical_tabs-475850.patch | 1.24 KB | les lim |
| vertical_tabs_no_js_bug.png | 3.98 KB | hass |
Comments
Comment #1
dave reidCan you verify this happens in the latest version?
Comment #2
hass commentedYes, it is still shown in the latest release.
Comment #3
les limYes, this is still there. The module uses Form API to create the Vertical Tabs container and JS to move the fieldsets into it, but if javascript is unavailable, we'll still be left with an empty DIV that has a border but no tabs.
One solution is to hide the Vertical Tabs container in the default CSS and show it with JS when the interface is being processed. Patch attached.
Comment #4
dave reidTested and works just fine. Committed to CVS (http://drupal.org/cvs?commit=285296). Thanks!
Comment #5
les limHold on, is this the right approach? What if there's a case in which a themer wants to use CSS to hide vertical tabs for some reason? Using javascript to forcibly show the interface will always override that intent. This may be worth further thought.
Comment #6
dave reidBest way is to backport the D7 JavaScript code to manipulate the elements inside the and show those only. For now, this is the best approach. If you have a patch, please open a new issue.