-
Notifications
You must be signed in to change notification settings - Fork 13
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
Sync scrolling in markdown preview? #27
Comments
cool. working it right now. so there's a bit of bookkeeping between the viewers and editors... it's already smelling manager-like, but the first pass will be a |
Still nothing to show for it. One thing: i did have to hoist the I had forgotten how much i dislike syncing to referent frames... the codemirror stuff sounds tantalizingly close in a number of places with things like Meanwhile, scrolling the The last thing, and something i didn't realize about markdownit, is that, for example, Will update if i make more progress! |
Thanks for keeping me in the loop! It sounds a little gnarly work to be honest, glad someone is tackling it ;) |
Elevator Pitch
Add bi-directional scrolling between an Editor and a Markdown Preview. Allow disabling this behavior.
Motivation
Yesterday during the collaborative editing demo during the community call, it was somewhat painful watching people editing markdown and have weird scrolling happen in an otherwise great experience.
As we can't get that data out marked upstream, this seems like the best place to land it. It's been a tick since I've looked at the code (binder's busted, btw), and I'd make this a plugin, but it feels like a it should be a core feature to this "ecosystem".
The other whammy is the default 300ms delay which is way too long with multiple editors, but that's for another day...
Design Ideas
The upstream demo contains the primary content for basic synced scrolling. Doing it right right is harder, but this would give us someplace to start.
As both the Editor and Markdown Preview documents have empty toolbars (by default), my inclination would be to enable this by default and offer a toggleable in Main Menu item in Edit or something.
Future Work
With more knowledge of the tokens between two documents, we could do even prettier LSP-style things, like jump-to-token and synced highlighting.
The text was updated successfully, but these errors were encountered: