Follow zoom maximum/minimum limit in view controls
Bug #1501424 reported by
Stefano Verzegnassi
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu Document Viewer App |
Fix Released
|
Medium
|
Stefano Verzegnassi | ||
Bug Description
We have decided that the minimum zoom limit should be 50% and maximum limit 400%.
At the moment we have hardcoded those limits in the ZoomSelector.qml component of the LibreOffice Viewer, but other components (e.g. zoom-in and zoom-out buttons) are not following those limits.
These limits should be set directly in the Viewer component from DocumentViewer.
Eventually, we could consider to export these limits and make them settable from QML.
Related branches
lp://qastaging/~verzegnassi-stefano/ubuntu-docviewer-app/lok-content-centered
- Alan Pope 🍺🐧🐱 🦄 (community): Approve
- Jenkins Bot: Approve (continuous-integration)
-
Diff: 393 lines (+249/-33)5 files modifiedsrc/app/qml/common/ScalingMouseArea.qml (+116/-0)
src/app/qml/common/ScalingPinchArea.qml (+35/-31)
src/app/qml/loView/LOViewPage.qml (+65/-1)
src/plugin/libreofficetoolkit-qml-plugin/lozoom.cpp (+8/-0)
src/plugin/libreofficetoolkit-qml-plugin/qml/Viewer.qml (+25/-1)
| tags: |
added: loviewer removed: libreoffice-viewer |
| Changed in ubuntu-docviewer-app: | |
| milestone: | none → ww02-2016 |
| importance: | Undecided → Medium |
| Changed in ubuntu-docviewer-app: | |
| status: | New → Confirmed |
| Changed in ubuntu-docviewer-app: | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Stefano Verzegnassi (verzegnassi-stefano) |
Revision history for this message
| Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote | #1 |
| Changed in ubuntu-docviewer-app: | |
| status: | In Progress → Fix Committed |
| Changed in ubuntu-docviewer-app: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fix committed into lp:ubuntu-docviewer-app at revision None, scheduled for release in ubuntu-