Skip to content

Fix broken "open in new tab" link in UIKit #1202

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

Merged
merged 1 commit into from
May 9, 2020

Conversation

sghoweri
Copy link
Contributor

@sghoweri sghoweri commented May 9, 2020

Just noticed this was broken while reviewing a separate PR... this PR should fix the "Open In New Tab" link in Pattern Lab's context menu so it now opens the correct HTML file vs a broken "undefined" page.

Before (Broken)
CleanShot 2020-05-09 at 14 20 23

After (Working)
CleanShot 2020-05-09 at 14 21 01

@sghoweri
Copy link
Contributor Author

sghoweri commented May 9, 2020

@bmuenzenmeyer or @JosefBredereck any chance one of you guys could do a super quick review of this UIKit hotfix?

@JosefBredereck JosefBredereck self-requested a review May 9, 2020 18:26
@JosefBredereck
Copy link
Contributor

Yes, it is, I already implemented it in #1143 so I know it will work. Also, you added in addition an check if the URL is available, which I really like. I will remove it from #1143 because it is some lines above this one.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 73.388% when pulling f41ae05 on hotfix/fix-uikit-open-in-new-tab into ee0f2ec on dev.

@sghoweri
Copy link
Contributor Author

sghoweri commented May 9, 2020

Yes, it is, I already implemented it in #1143 so I know it will work. Also, you added in addition an check if the URL is available, which I really like. I will remove it from #1143 because it is some lines above this one.

Ah, I totally missed that this fix was already in #1143 (good catch!). Thanks @JosefBredereck!

@sghoweri sghoweri merged commit 2f36604 into dev May 9, 2020
@sghoweri sghoweri deleted the hotfix/fix-uikit-open-in-new-tab branch May 9, 2020 18:33
@sghoweri
Copy link
Contributor Author

sghoweri commented May 9, 2020

Released in v5.10.1

@sghoweri sghoweri added this to the v5.10.1 milestone May 9, 2020
antonia-rose pushed a commit to quelltexterin/nemo-uikit-workshop that referenced this pull request Apr 12, 2023
antonia-rose pushed a commit to quelltexterin/nemo-uikit-workshop that referenced this pull request Apr 12, 2023
…-open-in-new-tab

Fix broken "open in new tab" link in UIKit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants