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

Markdown block shortcuts #558

Merged
merged 1 commit into from
Nov 25, 2021
Merged

Markdown block shortcuts #558

merged 1 commit into from
Nov 25, 2021

Conversation

pulyaevskiy
Copy link
Contributor

@cgestes I couldn't find your version in your PR (#546) for some reason, so there was no commits to cherry-pick which would be authored by you.

This implementation is basically the same as in the Gist I shared a while back, just adds some tests and docs.

Does not include the cursor position fix for Zefyr (will be submitted separately).

@pulyaevskiy pulyaevskiy changed the base branch from master to 1.0-dev November 24, 2021 22:06
@pulyaevskiy pulyaevskiy merged commit b641aa1 into 1.0-dev Nov 25, 2021
@pulyaevskiy pulyaevskiy deleted the md-block-shortcuts branch November 25, 2021 01:39
@cgestes
Copy link
Collaborator

cgestes commented Nov 25, 2021

I am pretty I wrote more test than that, you probably can import them. I am also pretty sure what you posted in the gist was broken (that was my understanding back then).

future will tell :)

@pulyaevskiy
Copy link
Contributor Author

I just added the position diff fix and tested the whole thing locally, worked as expected. Let me know if you find any issues with it.

@cgestes
Copy link
Collaborator

cgestes commented Nov 26, 2021

seems fine. pushed the tests that didn't seem to overlap with yours. Only one very specific (and strange one) is failing.
I cant remember how it was triggered.

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.

2 participants