Let mouse wheel zoom snap to view resolution levels - #7210
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
66c7420 to
b5c8c5d
Compare
| zoomFactor: 1.1, | ||
| extent, | ||
| constrainOnlyCenter: true, | ||
| smoothExtentConstraint: true, | ||
| showFullExtent: true, | ||
| padding: [20, 20, 20, 20] | ||
| padding: [20, 20, 20, 20], | ||
| constrainResolution: true |
There was a problem hiding this comment.
I tested this and did not notice a real difference with regard to the necessary mouse scrolls. Setting zoomFactor to 1.3 had more effect on the aspects @matthias-ronge seems to talk about in the issue.
There was a problem hiding this comment.
I tested this and did not notice a real difference with regard to the necessary mouse scrolls. Setting
zoomFactorto 1.3 had more effect on the aspects @matthias-ronge seems to talk about in the issue.
Really? That's interesting - and exactly the opposite of what I have observed!
Zooming 10 "ticks" in the current main branch looks like this:
issue-4244-main.mov
Zooming 10 "ticks" with zoomFactor: 1.3:
issue-4244-zoom-factor-1-3.mov
Zooming 10 "ticks" with constrainResolution: true:
issue-4244-constrain-resolution.mov
I am using Firefox, version 154 on MacOS 26.6.3 (Tahoe), btw.
@BartChris maybe have to reload your browsers cache to see the real difference with using contrainResolution?
7dbe816 to
752a0fe
Compare
752a0fe to
e56d750
Compare
2 participants
Fixes #4244 by using option constrainResolution to link mouse wheel "steps" to zoom levels in OpenLayer map of gallery detail view.
Note: I will open backports of this fix for Kitodo
3.9.xand4.0.xonce this pull request is approved