Skip to content

fix(docs): dialog navbar issues - #1817

Merged
chakrihacker merged 4 commits into
feat/member-mgmt-docsfrom
fix/navbar-bottom-hairline
Sep 4, 2026
Merged

fix(docs): dialog navbar issues#1817
chakrihacker merged 4 commits into
feat/member-mgmt-docsfrom
fix/navbar-bottom-hairline

Conversation

@chakrihacker

@chakrihacker chakrihacker commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Description

Hides the navbar signup/login buttons (#adu-root) when a Universal Components dialog is open. When a UC dialog opens, it overlaps the top navigation area, causing the navbar CTA buttons to render on top of or beneath the dialog in a visually broken state. This fix uses a CSS body:has() parent selector to detect an open dialog and fade out #adu-root with a short opacity transition, then defers visibility: hidden until after the fade completes so the buttons are removed from tab order cleanly.

References

Screenshot 2026-09-02 at 6 47 47 PM

Testing

  1. Open any page in the docs site that renders a Universal Components dialog (e.g. Member Management).
  2. Trigger the dialog to open — confirm the navbar CTA buttons fade out and are no longer visible or focusable.
  3. Close the dialog — confirm the buttons fade back in.
  4. Verify no layout shift or z-index conflicts on light and dark mode.

Checklist

  • I've read and followed CONTRIBUTING.md.
  • I've tested the site build for this change locally.
  • I've made appropriate docs updates for any code or config changes.
  • I've coordinated with the Product Docs and/or Docs Management team about non-trivial changes.

@chakrihacker
chakrihacker requested a review from a team as a code owner September 2, 2026 13:15
Comment thread main/ui/css/styles.css
margin-right: 0 !important;
}

#adu-root {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chakrihacker Can we check if this working fine else where?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't affect at other places

@chakrihacker
chakrihacker merged commit d2c9c25 into feat/member-mgmt-docs Sep 4, 2026
2 checks passed
@chakrihacker
chakrihacker deleted the fix/navbar-bottom-hairline branch September 4, 2026 11:33
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.

3 participants