Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

fix: Can't open external browser in Markdown Preview with external link containing '#'#3233

Merged
Rokt33r merged 2 commits into
BoostIO:masterfrom
hikerpig:bugfix/openExternalLink
Oct 10, 2019
Merged

fix: Can't open external browser in Markdown Preview with external link containing '#'#3233
Rokt33r merged 2 commits into
BoostIO:masterfrom
hikerpig:bugfix/openExternalLink

Conversation

@hikerpig

@hikerpig hikerpig commented Sep 2, 2019

Copy link
Copy Markdown
Contributor

Description

Current implementation might mistake external link with '#' for internal link. This PR fixes this.

Proper behavior should be:

- [Open in new browser](http://google.com/#viewport)

- [scrolls to internal heading](#Description)

Issue fixed

Type of changes

  • 🔘 Bug fix (Change that fixed an issue)

Checklist:

@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Sep 4, 2019

@ZeroX-DG ZeroX-DG left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 🎉

@ZeroX-DG ZeroX-DG added approved 👍 Pull request has been approved by sufficient reviewers. and removed awaiting review ❇️ Pull request is awaiting a review. labels Sep 15, 2019
@Rokt33r Rokt33r removed the approved 👍 Pull request has been approved by sufficient reviewers. label Oct 10, 2019
@Rokt33r Rokt33r added this to the v0.13.0 milestone Oct 10, 2019
@Rokt33r Rokt33r merged commit 3eabf95 into BoostIO:master Oct 10, 2019
@Flexo013

Copy link
Copy Markdown
Contributor

Might be related to #3287

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

4 participants