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

fix: open link on Mod-Click #130

Merged
merged 14 commits into from
Jul 20, 2022
Merged

fix: open link on Mod-Click #130

merged 14 commits into from
Jul 20, 2022

Conversation

dancormier
Copy link
Contributor

fixes #62

TODO

  • Tests!

@dancormier dancormier added bug Something isn't working mode - rich text Affects the editor's rich text (wysiwyg) mode labels May 19, 2022
@dancormier dancormier requested a review from b-kelly May 19, 2022 19:56
@netlify
Copy link

netlify bot commented May 19, 2022

Deploy Preview for nifty-lalande-39c157 ready!

Name Link
🔨 Latest commit 59b59a2
🔍 Latest deploy log https://app.netlify.com/sites/nifty-lalande-39c157/deploys/628cfbe906bdfe0008827af3
😎 Deploy Preview https://deploy-preview-130--nifty-lalande-39c157.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Collaborator

@b-kelly b-kelly left a comment

Choose a reason for hiding this comment

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

Blocking merge, waiting for tests and changes to link-editor plugin

* Returns the mark at cursor if it is of type `link`
* @param state The current editor state
*/
function findLinkAtCursor(state: EditorState): Mark {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we can chuck this functionality into the link-tooltip plugin and avoid having to add another plugin. I'd wait on this for the time being though, as I'm nearly finished with a total overhaul of that file and changing it would cause some nasty merge conflicts.

@b-kelly b-kelly self-requested a review July 20, 2022 15:59
@b-kelly b-kelly marked this pull request as ready for review July 20, 2022 15:59
@b-kelly b-kelly merged commit ea8da4b into main Jul 20, 2022
@b-kelly b-kelly deleted the dcormier/mod-click-link-open-62 branch July 20, 2022 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working mode - rich text Affects the editor's rich text (wysiwyg) mode
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ctrl+clicking a hyperlink should open the link
2 participants