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.

Sign up for GitHub

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

Jump to bottom

Apply transactions to all views #4733

Merged
merged 4 commits into from
Nov 23, 2022
Merged

Apply transactions to all views #4733

merged 4 commits into from
Nov 23, 2022

Commits on Nov 23, 2022

  1. Add a test case for updating jumplists across windows

    @the-mikedavis
    the-mikedavis committed Nov 23, 2022
    Configuration menu
    Copy the full SHA
    6e423bb View commit details
    Browse the repository at this point in the history

  2. Apply transactions to all views on history changes 

    This ensures that jumplist selections follow changes in documents, even
when there are multiple views (for example a split where both windows
edit the same document).
    @the-mikedavis
    the-mikedavis committed Nov 23, 2022
    Configuration menu
    Copy the full SHA
    f650ce0 View commit details
    Browse the repository at this point in the history

  3. Leave TODOs for cleaning up View::apply

    @the-mikedavis
    the-mikedavis committed Nov 23, 2022
    Configuration menu
    Copy the full SHA
    8f08375 View commit details
    Browse the repository at this point in the history

  4. Use Iterator::reduce to compose history transactions 

    Co-authored-by: Blaž Hrastnik <[email protected]>
    @the-mikedavis @archseer
    the-mikedavis and archseer committed Nov 23, 2022
    Configuration menu
    Copy the full SHA
    187858f View commit details
    Browse the repository at this point in the history