Skip to content

Fix dropdown menus hidden by FOUC CSS specificity regression (#835) - #836

Merged
jjroelofs merged 1 commit into
8.xfrom
Drale1/8.x/#835-Dropdown-menus-do-not-open-after-update-to-8.1.3
Jul 29, 2026
Merged

Fix dropdown menus hidden by FOUC CSS specificity regression (#835)#836
jjroelofs merged 1 commit into
8.xfrom
Drale1/8.x/#835-Dropdown-menus-do-not-open-after-update-to-8.1.3

Conversation

@drale1

@drale1 drale1 commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Linked issues

#835

https://www.drupal.org/project/dxpr_theme/issues/3610960

Solution

The 8.1.3 FOUC rule .dxpr-theme-header ul.dropdown-menu { display: none; } has higher specificity than Bootstrap’s .dropdown-menu.show, so open dropdowns stay hidden.

Updated the inline FOUC style in templates/html.html.twig to .dxpr-theme-header ul.dropdown-menu:not(.show) { display: none; }, so closed menus still avoid FOUC while open menus can display.

Checklist

  • I have read the CONTRIBUTING.md document.
  • My commit messages follow the contributing standards and style of this project.
  • My code follows the coding standards and style of this project.
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Need to run update.php after code changes
  • Requires a change to end-user documentation.
  • Requires a change to developer documentation.
  • Requires a change to QA tests.
  • Requires a new QA test.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

@drale1
drale1 requested a review from jjroelofs July 20, 2026 09:27
@jjroelofs
jjroelofs merged commit c5a67c3 into 8.x Jul 29, 2026
5 checks passed
@jjroelofs
jjroelofs deleted the Drale1/8.x/#835-Dropdown-menus-do-not-open-after-update-to-8.1.3 branch July 29, 2026 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants