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

Fix centering current index #659

Merged
merged 2 commits into from
Aug 10, 2024

Conversation

laomuon
Copy link
Contributor

@laomuon laomuon commented Aug 6, 2024

Fixes: #615 by making the scrolling to be execute after everything else

@laomuon laomuon changed the title 615 fix centering current index Fix centering current index Aug 6, 2024
@jdujava
Copy link
Contributor

jdujava commented Aug 7, 2024

Seems to work alright, thanks for looking into this!

@laomuon laomuon force-pushed the 615-fix-centering-current-index branch from 38c9cb2 to c299cfe Compare August 7, 2024 21:18
@laomuon laomuon marked this pull request as ready for review August 7, 2024 21:18
@laomuon laomuon requested a review from sebastinas August 9, 2024 01:54
zathura/zathura.h Outdated Show resolved Hide resolved
@laomuon laomuon force-pushed the 615-fix-centering-current-index branch from 0d234d8 to 3cd6d99 Compare August 9, 2024 12:01
@laomuon laomuon force-pushed the 615-fix-centering-current-index branch from 3cd6d99 to 491c999 Compare August 9, 2024 12:03
@laomuon laomuon requested a review from sebastinas August 9, 2024 12:03
@sebastinas sebastinas merged commit b787fad into pwmt:develop Aug 10, 2024
5 checks passed
@laomuon laomuon deleted the 615-fix-centering-current-index branch August 15, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Centering of current index row is wrong on the first index toggle
3 participants