-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Menu is broken when opening anchor links #885
Comments
The important part is opening the link in a new tab. That's the whole point of it being a separate URL, no? That one can link to a tab from elsewhere? tabs.mov |
Oh oh. Okay. So it's not that the functionality doesn't work, it's that the
ui doesn't update on new tab.
My guess is its the new header linking support. I'll take a look
…On Mon., Dec. 6, 2021, 10:51 a.m. Daniel Beck, ***@***.***> wrote:
The important part is opening the link in a new tab.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#885 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAA24B2RKJIMPM6D4MZFWTLUPUAZTANCNFSM5JOV34QA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Had to revert. I liked the new solution, but it caused rendering issues. |
halkeye
added a commit
that referenced
this issue
Dec 8, 2021
halkeye
added a commit
that referenced
this issue
Dec 10, 2021
Fix #885 - Properly handle tabs on page rehydrate
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Clicking "dependencies" on https://plugins.jenkins.io/matrix-auth/ changes the URL to https://plugins.jenkins.io/matrix-auth/#dependencies.
Opening a link to "https://plugins.jenkins.io/matrix-auth/#dependencies" takes you there directly, but the menu doesn't change accordingly:
The text was updated successfully, but these errors were encountered: