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

Links are clickable #2382

Merged
merged 5 commits into from
Sep 15, 2018
Merged

Links are clickable #2382

merged 5 commits into from
Sep 15, 2018

Conversation

daiyam
Copy link
Contributor

@daiyam daiyam commented Sep 8, 2018

This changes allows to click on links in the editor (like in VS Code).

link-clickable

}
onMouseDown(e) {
const { target, metaKey } = e
if (!metaKey) {
Copy link
Member

Choose a reason for hiding this comment

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

On Macintosh keyboards, this is the ⌘ Command key. On Windows keyboards, this is the Windows key (⊞ Windows).

I don't know how it looks on MacOS but on Windows and Linux, the meta key is used to open the main menu therefore when you hit meta key + click link, the link does open in the browser but the menu is also displayed which is quite annoying.

I think you will have to change this key to something else. Also you can't set it to ctrl on windows and linux as it's the key for multi-cursor in codemirror. Shift perhaps?

@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Sep 9, 2018
@daiyam
Copy link
Contributor Author

daiyam commented Sep 9, 2018

@ZeroX-DG oops, my bad. I've fixed it.

Copy link
Member

@ZeroX-DG ZeroX-DG left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

image

@ZeroX-DG ZeroX-DG added next release (v0.11.10) and removed awaiting review ❇️ Pull request is awaiting a review. labels Sep 9, 2018
@Rokt33r Rokt33r merged commit 80af8dc into BoostIO:master Sep 15, 2018
@daiyam daiyam deleted the link-clickable branch September 15, 2018 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants