Skip to content
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

Allow links in hover decorations to be inside a <code> tag #29082

Merged
merged 1 commit into from Aug 24, 2017

Conversation

eamodio
Copy link
Contributor

@eamodio eamodio commented Jun 20, 2017

Related to #29076

Allows link actions when links are embedded inside another tag -- e.g. <code> tags

For example in GitLens I want to add the following command links:
image

And I'd like to keep the <code> tag styling on them.

@mjbvz @jrieken I've repurposed this PR to just allow the above.

@mention-bot
Copy link

@eamodio, thanks for your PR! By analyzing the history of the files in this pull request, we identified @egamma and @jrieken to be potential reviewers.

@eamodio eamodio changed the title Allow command: links in hover decorations Allow links in hover decorations to be inside a <code> tag Jul 29, 2017
@eamodio
Copy link
Contributor Author

eamodio commented Aug 15, 2017

Any chance this could make it in for August? I'd love to be able to keep the GitLens hover styling, but also allow command links.

@eamodio
Copy link
Contributor Author

eamodio commented Aug 24, 2017

@jrieken Any chance for the august milestone for this? 🤞

@jrieken
Copy link
Member

jrieken commented Aug 24, 2017

I am good with it. Wake up @mjbvz

@mjbvz mjbvz merged commit d7da141 into microsoft:master Aug 24, 2017
@mjbvz
Copy link
Contributor

mjbvz commented Aug 24, 2017

Thanks @eamodio! Should be in next insiders build

@mjbvz mjbvz added this to the August 2017 milestone Aug 24, 2017
@eamodio
Copy link
Contributor Author

eamodio commented Aug 24, 2017

Awesome thank you!

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants