-
Notifications
You must be signed in to change notification settings - Fork 3
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 cursor when first opening on viewer #17
Sync cursor when first opening on viewer #17
Conversation
Perfect & fantastic Gives me an oftopic thought though: The way the cursor line is at the very bottom of the page feels pretty disorienting to me when opening But what I'm getting to with this is that I wish the cursor sync could have a similar option to Vim's scrolloff, if you know what I mean But of course it's more complicated because lines aren't a fixed size like in terminal |
Yes definitely. I have also had this thought but haven't followed it to think about how to implement that, haha. We use JavaScript's EDIT maybe also this. Can't look into it in detail right now. |
Yeah, that calls for a new issue |
Agree! Feel free to open one :) |
Ok, I think we explained the feat quite well here so I just quoted us with the links and all. |
This requires a small change with |
I'll adjust this after we've merged the PR on main Vivify :) |
da7b6af
to
bc41658
Compare
Hello @tuurep this should work now with the version of Vivify that's currently on main. Could you confirm? |
Yes it works We've fixed all of my little usability gripes! Huge thanks Caution offtopic 👇 Well I still wish I could open an empty buffer as an empty markdown preview and start typing 😄 Btw I suspect this is because What do you think, is this fixable as well? |
If I'm not completely mistesting this then it was a one-liner ^^ Check out jannis-baum/Vivify#150 |
I just remembered that this PR will break things if someone is on the |
Ok, I'll update AUR right away |
Close #6
TODO before merging this: