Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The vertical tabs now appear empty. The list has no values. Any ideas?
Comment | File | Size | Author |
---|---|---|---|
#9 | 1004246-9-styleguide-maintenance-page.patch | 2.11 KB | jessehs |
Comments
Comment #1
agentrickardStill failing. This patch cleans things up some, but there is a fatal JS error that prevents the tab labels from rendering.
Even swapping the generated form out with working core form code failed. Only the first item appears, and the links fail entirely.
JS error (from Firebug):
I don't know enough about Drupal's JS -- this function is 3 lines long -- to debug further.
Comment #2
xumepadismal CreditAttribution: xumepadismal commentedHave the same issue. It seems that problem is the "Maintenance page" style example.
theme('maintenance_page')
inserts entire HTML with<script>
s and<link>
s to page and breaks all layout. Don't sure the best solution for this. Maybe iframe?Anyway, I've commented lines 216-220 in 'styleguide.styleguide.inc' for now to have possibility to test verticals tabs and it works for me.
Comment #3
agentrickardNice sleuthing!
Comment #4
dozymoe CreditAttribution: dozymoe commentedHere's the patch.
Can't do maintenance page preview per installed themes, would only display the default theme. It has something todo with the location of the file maintenance-page.tpl.php, which is stored in the theme registry.
Hacking around, I tried:
But the preview page also listed the default theme css files.
drupal_static_reset('drupal_add_css');
just removed all of css added by modules and themes :Pedit: er, that code was in the page callback.
Comment #5
dozymoe CreditAttribution: dozymoe commentedEh, forgot the set the flag.
Comment #6
agentrickardYou have to use l() and url() or else it breaks when running in a subdirectory.
Comment #7
dead_armClean up this comment (no emoticons, proper punctuation, capitalize HTML)
Page callback functions shouldn't start with underscores.
We should consider putting the width/height into CSS...
Comment #8
agentrickardFor a one-off iframe, is it really worth it? This code already runs through and alter hook if people need to change the behavior...
Comment #9
jessehsHere's a new patch, rerolled, addressing some concerns. Working fieldgroups/javascript seems like a pretty major issue IMHO.
Comment #10
gmclelland CreditAttribution: gmclelland commentedThe patch in #9 applies and worked for me. Before the patch all javascript on the page was broken for me.
Comment #11
tancConfirmed patch in #9 fixed the broken javascript and allowed the vertical tabs to render correctly.
Comment #13
BWPanda CreditAttribution: BWPanda commentedThe patch (applied to the latest dev version) fixes the vertical tabs so that they display correctly, but doesn't display the maintenance page correctly - the page linked to from the Maintenance Page section, and the page displayed in the iframe, are copies of the current styleguide page, not the maintenance page...
Comment #14
loopduplicateAdding related issue: #2505137: Error on drupal.js while using Style Guide
Comment #15
loopduplicateComment #16
loopduplicateHi All,
Regarding #13, I'm not able to reproduce "the Maintenance Page section, and the page displayed in the iframe, are copies of the current styleguide page". The Maintenance page section has admin/appearance/styleguide/maintenance-page for the link and the iframe. It is not the main styleguide page. If this is an issue, we need to file a separate report. The bug in this issue is the vertical tabs stopped working and there is a JS error on the page. That is fixed with #9.
I'm moving this back to RTBC. Please feel free to move it back if you think I'm wrong.
Regards,
Jeff
Comment #17
loopduplicate