Summary: | horizontal mouse-drag wraparound | ||
---|---|---|---|
Product: | [Applications] okular | Reporter: | azrdev |
Component: | general | Assignee: | David Hurka <david.hurka> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | nate, okular-devel |
Priority: | NOR | ||
Version: | 0.26.1 | ||
Target Milestone: | --- | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
See Also: |
https://bugs.kde.org/show_bug.cgi?id=343910 https://bugs.kde.org/show_bug.cgi?id=439417 |
||
Latest Commit: | https://invent.kde.org/graphics/okular/commit/2e1ddaa1095fc2b564512851f55eee880e964746 | Version Fixed In: | 21.08 |
Description
azrdev
2016-10-31 18:30:47 UTC
This is not just relevant for horizontal scrolling mode, but also for landscape documents in Overview mode with the current limit of 8 facing pages. Horizontal scrolling is difficult with most mouse wheels, so horizontal drag wraparound makes not less sense than the existing vertical wraparound. A possibly relevant merge request was started @ https://invent.kde.org/graphics/okular/-/merge_requests/199 A possibly relevant merge request was started @ https://invent.kde.org/graphics/okular/-/merge_requests/394 Git commit 2e1ddaa1095fc2b564512851f55eee880e964746 by David Hurka. Committed on 11/04/2021 at 09:56. Pushed by davidhurka into branch 'master'. Enable horizontal mouse wrap for Browse tool drag scrolling Also fixes calculation of wrap edges, where top and bottom edge were swapped. M +4 -2 part/pageview.cpp https://invent.kde.org/graphics/okular/commit/2e1ddaa1095fc2b564512851f55eee880e964746 Nice! David, do you think this might be worth cherry-picking into stable too, or is it too risky or dependent on master-only code? It depends on 67325270e “Fix viewport jumps when drag-scrolling beyond screen edges” which introduces CursorWrapHelper and is only on master. So I think it’s too risky. |