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

Add history scroll restoration for scroll views #2366

Merged
merged 8 commits into from
Jul 27, 2024

Conversation

guanglinn
Copy link
Contributor

@guanglinn guanglinn commented Jul 25, 2024

Add support for auto scroll/history scroll restoration for scroll views. These features can improve the user experience.

In this PR:

  1. Scroll view in FontSizeDialog
    can auto scroll to the selection of current font size;
  2. Scroll view in SpecialKeyDialog
    can auto save and restore previous scroll position;
  3. Scroll view in TextActionsBar
    can keep scroll position even being reloaded;
  4. Padding in view and edit mode
    minor improvements.

@guanglinn
Copy link
Contributor Author

guanglinn commented Jul 25, 2024

It is painful and not user-friendly to scroll the view from the origin to the previous position with your fingers every time.

Here is the 3 improved scroll views, you can make a comparison with a previous version.

2024-07-25-auto-restore-position.mp4

@guanglinn guanglinn changed the title Add auto restore position for scroll views Add history scroll restoration for scroll views Jul 27, 2024
Copy link
Owner

@gsantner gsantner left a comment

Choose a reason for hiding this comment

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

Thank you, I tried it and looked good! Merging

@gsantner gsantner added this to the Markor v2.13 milestone Jul 27, 2024
@gsantner gsantner merged commit 80e813e into gsantner:master Jul 27, 2024
1 check passed
@guanglinn guanglinn deleted the auto_scroll_dialog branch July 28, 2024 11:05
elyahw pushed a commit to writing-tools/marelias-notepad that referenced this pull request Sep 26, 2024
* feat(font size dialog): add auto scroll for selection list
* feat(special key dialog): add auto restore scroll for key list
* feat(text actions bar): add auto restore scroll position
* refactor: remove unnecessary font size in range 1 to 4
* refactor: minor improvements to padding of Markdown in view mode
* refactor: minor improvements to padding of editor in edit mode

(cherry picked from commit 80e813e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants