Merge lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/qtScaleFactor into lp:ubuntu-ui-toolkit/staging

Proposed by Cris Dywan
Status: Merged
Approved by: Zsombor Egri
Approved revision: 2055
Merged at revision: 2056
Proposed branch: lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/qtScaleFactor
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 72 lines (+10/-7)
4 files modified
src/Ubuntu/UbuntuToolkit/ucunits.cpp (+4/-5)
tests/unit/units/dpr1/tst_units.cpp (+2/-2)
tests/unit/units/dpr2/tst_units_dpr2.cpp (+2/-0)
tests/unit/units/dpr3/tst_units_dpr3.cpp (+2/-0)
To merge this branch: bzr merge lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/qtScaleFactor
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Zsombor Egri Approve
Review via email: mp+302648@code.launchpad.net

Commit message

Update scaling docs, set QT_SCALE_FACTOR and unset GRID_UNIT_PX

To post a comment you must log in.
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Zsombor Egri (zsombi) wrote :

Tiny typo.

review: Needs Fixing
2055. By Cris Dywan

Typo fix

Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Zsombor Egri (zsombi) wrote :

Looks like arm dev is again out of order...

review: Approve
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Ubuntu/UbuntuToolkit/ucunits.cpp'
--- src/Ubuntu/UbuntuToolkit/ucunits.cpp 2016-07-20 07:49:54 +0000
+++ src/Ubuntu/UbuntuToolkit/ucunits.cpp 2016-08-11 15:13:26 +0000
@@ -71,16 +71,15 @@
71*/71*/
7272
73/*73/*
74 * Note on the interaction between GRID_UNIT_PX and QT_DEVICE_PIXEL_RATIO74 * In Qt5.6 the following variables affect scaling in QWidget and QML:
75 * 1. QT_SCALE_FACTOR: global integer scale factor, including point-sized fonts
76 * 2. QT_SCREEN_SCALE_FACTORS: a list of scale factors
77 * 3. QT_AUTO_SCREEN_SCALE_FACTOR: enables automatic scaling heuristics
75 *78 *
76 * In Qt5.4 there is a single means to scale the UI: the QT_DEVICE_PIXEL_RATIO environment79 * In Qt5.4 there is a single means to scale the UI: the QT_DEVICE_PIXEL_RATIO environment
77 * variable. This accepts only integer values, thus allowing a x2 or x3 scaling of any80 * variable. This accepts only integer values, thus allowing a x2 or x3 scaling of any
78 * Qt-based UI, that includes QWidget as well as any QML UI.81 * Qt-based UI, that includes QWidget as well as any QML UI.
79 *82 *
80 * Setting QT_DEVICE_PIXEL_RATIO=2 implies one density-independent pixel corresponds to 2
81 * physical pixels. Developers describe their UI in terms of density-independent pixels.
82 * Qt scales accordingly.
83 *
84 * The Ubuntu UI Toolkit has solved the scaling problem with the GRID_UNIT_PX variable.83 * The Ubuntu UI Toolkit has solved the scaling problem with the GRID_UNIT_PX variable.
85 * It offers more flexibility, but only scales QML applications written to use the UITK84 * It offers more flexibility, but only scales QML applications written to use the UITK
86 * (since it uses this Units class) as it is built on top of QML.85 * (since it uses this Units class) as it is built on top of QML.
8786
=== modified file 'tests/unit/units/dpr1/tst_units.cpp'
--- tests/unit/units/dpr1/tst_units.cpp 2016-07-07 15:33:34 +0000
+++ tests/unit/units/dpr1/tst_units.cpp 2016-08-11 15:13:26 +0000
@@ -28,7 +28,7 @@
28private Q_SLOTS:28private Q_SLOTS:
2929
30 void defaultGridUnit() {30 void defaultGridUnit() {
31 qputenv("GRID_UNIT_PX", "");31 qunsetenv("GRID_UNIT_PX");
32 UCUnits units;32 UCUnits units;
33 QCOMPARE(units.gridUnit(), 8.0f);33 QCOMPARE(units.gridUnit(), 8.0f);
34 }34 }
@@ -46,7 +46,7 @@
46 qputenv("GRID_UNIT_PX", gridUnit);46 qputenv("GRID_UNIT_PX", gridUnit);
47 UCUnits units;47 UCUnits units;
48 QCOMPARE(units.gridUnit(), 11.0);48 QCOMPARE(units.gridUnit(), 11.0);
49 qputenv("GRID_UNIT_PX", "");49 qunsetenv("GRID_UNIT_PX");
50 }50 }
5151
52 void dpGridUnitDefault() {52 void dpGridUnitDefault() {
5353
=== modified file 'tests/unit/units/dpr2/tst_units_dpr2.cpp'
--- tests/unit/units/dpr2/tst_units_dpr2.cpp 2016-07-07 15:33:34 +0000
+++ tests/unit/units/dpr2/tst_units_dpr2.cpp 2016-08-11 15:13:26 +0000
@@ -135,6 +135,8 @@
135int main(int argc, char *argv[])135int main(int argc, char *argv[])
136{136{
137 qputenv("QT_DEVICE_PIXEL_RATIO", "2");137 qputenv("QT_DEVICE_PIXEL_RATIO", "2");
138 qputenv("QT_SCALE_FACTOR", "2");
139 qunsetenv("GRID_UNIT_PX");
138140
139 QGuiApplication app(argc, argv);141 QGuiApplication app(argc, argv);
140 app.setAttribute(Qt::AA_Use96Dpi, true);142 app.setAttribute(Qt::AA_Use96Dpi, true);
141143
=== modified file 'tests/unit/units/dpr3/tst_units_dpr3.cpp'
--- tests/unit/units/dpr3/tst_units_dpr3.cpp 2016-07-07 15:33:34 +0000
+++ tests/unit/units/dpr3/tst_units_dpr3.cpp 2016-08-11 15:13:26 +0000
@@ -80,6 +80,8 @@
80int main(int argc, char *argv[])80int main(int argc, char *argv[])
81{81{
82 qputenv("QT_DEVICE_PIXEL_RATIO", "3");82 qputenv("QT_DEVICE_PIXEL_RATIO", "3");
83 qputenv("QT_SCALE_FACTOR", "3");
84 qunsetenv("GRID_UNIT_PX");
8385
84 QGuiApplication app(argc, argv);86 QGuiApplication app(argc, argv);
85 app.setAttribute(Qt::AA_Use96Dpi, true);87 app.setAttribute(Qt::AA_Use96Dpi, true);

Subscribers

People subscribed via source and target branches