Synchronize scrolling between the input textarea and the preview zone #22

Open
silviubogan wants to merge 4 commits from silviubogan/master into master
silviubogan commented 2017-10-23 11:48:27 -07:00 (Migrated from github.com)

To synchronize the scroll position between the #wmd-input and #wmd-preview. I tested the changes rapidly and they seem to work well.

To synchronize the scroll position between the #wmd-input and #wmd-preview. I tested the changes rapidly and they seem to work well.
josephernest commented 2017-10-23 13:10:04 -07:00 (Migrated from github.com)

Nice idea! Can you share a live demo link, so that we can try it? Thanks!

Nice idea! Can you share a live demo link, so that we can try it? Thanks!
silviubogan commented 2017-10-23 13:24:56 -07:00 (Migrated from github.com)
https://silviubogan.github.io/writing/
josephernest commented 2017-10-23 14:00:25 -07:00 (Migrated from github.com)

Interesting, it works most of the time, but not 100% reliable: try with a code containing a half-page comment:

<!-- Hello
     This is a long comment
     This is a long comment
     ...
 -->

or a many-lines matrix:

$$A = \begin{pmatrix}1 \\ 2 \\ 3 \\ 4 \\ 5 \\ 6\end{pmatrix} $$

Then the synchronisation is not always good. The fact we cannot turn it off could make the editing difficult in some cases. I suggest we experiment this a bit, a few weeks to see how it behaves on every day use. After a few weeks of use, we'll be able to tell if it's reliable.

Interesting, it works most of the time, but not 100% reliable: try with a code containing a half-page comment: <!-- Hello This is a long comment This is a long comment ... --> or a many-lines matrix: $$A = \begin{pmatrix}1 \\ 2 \\ 3 \\ 4 \\ 5 \\ 6\end{pmatrix} $$ Then the synchronisation is not always good. The fact we cannot turn it off could make the editing difficult in some cases. I suggest we experiment this a bit, a few weeks to see how it behaves on every day use. After a few weeks of use, we'll be able to tell if it's reliable.
This pull request has changes conflicting with the target branch.
  • README.md
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin silviubogan/master:silviubogan/master
git switch silviubogan/master

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff silviubogan/master
git switch silviubogan/master
git rebase master
git switch master
git merge --ff-only silviubogan/master
git switch silviubogan/master
git rebase master
git switch master
git merge --no-ff silviubogan/master
git switch master
git merge --squash silviubogan/master
git switch master
git merge --ff-only silviubogan/master
git switch master
git merge silviubogan/master
git push origin master
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
brad/writing!22
No description provided.