Skip to content

web: Close modal on route navigation#21622

Open
GirlBossRush wants to merge 2 commits intomainfrom
fix-modal-close-on-routing
Open

web: Close modal on route navigation#21622
GirlBossRush wants to merge 2 commits intomainfrom
fix-modal-close-on-routing

Conversation

@GirlBossRush
Copy link
Copy Markdown
Contributor

@GirlBossRush GirlBossRush commented Apr 15, 2026

Details

This PR fixes a small issue where modals do not close after the router navigates to another page, such as when clicking an enterprise only link within a modal.

@GirlBossRush GirlBossRush requested a review from a team as a code owner April 15, 2026 14:39
@GirlBossRush GirlBossRush changed the base branch from json-fixes to main April 15, 2026 14:43
@GirlBossRush GirlBossRush added the area:frontend Features or issues related to the browser, TypeScript, Node.js, etc label Apr 15, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 15, 2026

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit 9501f46
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/69dfa3095c381c000880a3b4
😎 Deploy Preview https://deploy-preview-21622--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kensternberg-authentik kensternberg-authentik changed the base branch from main to json-fixes April 15, 2026 17:43
Copy link
Copy Markdown
Contributor

@kensternberg-authentik kensternberg-authentik left a comment

Choose a reason for hiding this comment

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

Elegant and excellent.

@github-project-automation github-project-automation bot moved this from Todo to In Progress in authentik Core Apr 15, 2026
@GirlBossRush GirlBossRush changed the base branch from json-fixes to main April 15, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Features or issues related to the browser, TypeScript, Node.js, etc

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants