Font size too small in browser with Qt 5.1 / DPR rework
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| ubuntu-ui-toolkit (Ubuntu) |
High
|
Florian Boucault |
Bug Description
The font size seems too small in browser with Qt 5.1. The device pixel ratio patch was dropped and there is reportedly something new in Qt 5.1 that can be used instead.
To get Qt 5.2 on device:
See instructions _on the front page of_ https:/
All Qt 5.2 bugs: https:/
Related branches
- Unity8 CI Bot: Needs Fixing (continuous-integration) on 2016-03-24
- Daniel d'Andrada (community): Needs Information on 2015-10-19
- Gerry Boland (community): Needs Fixing on 2014-04-04
- PS Jenkins bot: Approve (continuous-integration) on 2014-03-11
- Loïc Molinari (community): Approve on 2014-03-06
-
Diff: 261 lines (+74/-17)8 files modifiedsrc/modules/application/input_filter_area.cc (+16/-1)
src/modules/application/input_filter_area.h (+3/-0)
src/platforms/base/input.cc (+5/-4)
src/platforms/base/window.cc (+6/-0)
src/platforms/base/window.h (+3/-0)
src/platforms/ubuntu/ubuntucommon/screen.cc (+29/-7)
src/platforms/ubuntu/ubuntucommon/screen.h (+4/-0)
src/platforms/ubuntu/ubuntucommon/window.cc (+8/-5)
- Tim Peeters: Resubmit on 2014-07-15
- PS Jenkins bot: Approve (continuous-integration) on 2014-03-10
- Zsombor Egri: Pending requested 2013-09-03
-
Diff: 525 lines (+219/-61)11 files modifiedmodules/Ubuntu/Components/Themes/Ambiance/HeaderStyle.qml (+1/-0)
modules/Ubuntu/Components/Themes/Ambiance/TabBarStyle.qml (+2/-2)
modules/Ubuntu/Components/plugin/shapeitem.cpp (+20/-5)
modules/Ubuntu/Components/plugin/ucqquickimageextension.cpp (+42/-4)
modules/Ubuntu/Components/plugin/ucscalingimageprovider.cpp (+6/-0)
modules/Ubuntu/Components/plugin/ucunits.cpp (+30/-7)
modules/Ubuntu/Components/plugin/ucunits.h (+4/-0)
modules/Ubuntu/Components/plugin/unitythemeiconprovider.cpp (+5/-2)
tests/unit/tst_qquick_image_extension/tst_qquick_image_extension.cpp (+7/-1)
tests/unit/tst_scaling_image_provider/tst_scaling_image_provider.cpp (+12/-4)
tests/unit/tst_units/tst_units.cpp (+90/-36)
- Florian Boucault (community): Disapprove on 2014-03-10
- PS Jenkins bot (community): Approve (continuous-integration) on 2014-03-06
- Ricardo Mendoza: Pending requested 2014-03-06
-
Diff: 72 lines (+24/-4)2 files modifiedsrc/modules/Unity/Application/inputarea.cpp (+21/-4)
src/modules/Unity/Application/inputarea.h (+3/-0)
- Zsombor Egri: Approve on 2015-08-13
- PS Jenkins bot: Approve (continuous-integration) on 2015-08-12
- Loïc Molinari (community): Approve on 2015-08-11
- Gerry Boland (community): Needs Information on 2015-04-27
-
Diff: 1044 lines (+747/-26)22 files modifieddebian/control (+5/-0)
src/Ubuntu/Components/plugin/ucqquickimageextension.cpp (+23/-3)
src/Ubuntu/Components/plugin/ucubuntushape.cpp (+17/-13)
src/Ubuntu/Components/plugin/ucunits.cpp (+46/-5)
src/Ubuntu/Components/plugin/ucunits.h (+1/-0)
tests/unit/add_makecheck.pri (+12/-1)
tests/unit/custom_qpa/README (+15/-0)
tests/unit/custom_qpa/custom.json (+3/-0)
tests/unit/custom_qpa/custom_qpa.pro (+15/-0)
tests/unit/custom_qpa/main.cpp (+54/-0)
tests/unit/custom_qpa/qcustombackingstore.cpp (+64/-0)
tests/unit/custom_qpa/qcustombackingstore.h (+55/-0)
tests/unit/custom_qpa/qcustomintegration.cpp (+112/-0)
tests/unit/custom_qpa/qcustomintegration.h (+74/-0)
tests/unit/runtest.sh (+2/-0)
tests/unit/tst_units/dpr1/dpr1.pro (+3/-0)
tests/unit/tst_units/dpr2/dpr2.pro (+3/-0)
tests/unit/tst_units/dpr2/tst_units_dpr2.cpp (+143/-0)
tests/unit/tst_units/dpr3/dpr3.pro (+3/-0)
tests/unit/tst_units/dpr3/tst_units_dpr3.cpp (+88/-0)
tests/unit/tst_units/tst_units.pro (+6/-3)
tests/unit/unit.pro (+3/-1)
- Ubuntu SDK team: Pending requested 2015-04-16
-
Diff: 245 lines (+61/-29)5 files modifiedmodules/Ubuntu/Components/plugin/ucqquickimageextension.cpp (+10/-3)
modules/Ubuntu/Components/plugin/ucubuntushape.cpp (+8/-10)
modules/Ubuntu/Components/plugin/ucunits.cpp (+37/-12)
modules/Ubuntu/Components/plugin/ucunits.h (+5/-3)
ubuntu-sdk.pro (+1/-1)
tags: | added: qt5.1 |
description: | updated |
description: | updated |
Changed in webbrowser-app: | |
assignee: | nobody → Florian Boucault (fboucault) |
status: | New → In Progress |
importance: | Undecided → High |
Dmitry Shachnev (mitya57) wrote : | #2 |
This happens with any QML application that uses WebView, not only with webbrowser-app.
no longer affects: | unity8 |
description: | updated |
Timo Jyrinki (timo-jyrinki) wrote : | #3 |
Bumped importance since Qt 5.1.1 is planned to land and this is clearly a blocker.
summary: |
- Font size too small in browser with Qt 5.1 + Font size too small in browser with Qt 5.1 / DPR rework |
Changed in webbrowser-app: | |
importance: | High → Critical |
description: | updated |
Florian Boucault (fboucault) wrote : | #4 |
Fixes are in:
https:/
https:/
They seem to work pretty well with one exception (QML's BorderImage) for which a patch is being worked on.
affects: | webbrowser-app → ubuntu-ui-toolkit |
Pat McGowan (pat-mcgowan) wrote : | #5 |
lower importance given 5.1 will be deferred
Changed in ubuntu-ui-toolkit: | |
importance: | Critical → High |
Changed in ubuntu-ui-toolkit: | |
milestone: | none → february-2014 |
tags: |
added: qt5.2 removed: qt5.1 |
Timo Jyrinki (timo-jyrinki) wrote : | #6 |
See https:/
Changed in ubuntu-ui-toolkit: | |
importance: | High → Critical |
Alan Pope 🍺🐧🐱 🦄 (popey) wrote : | #7 |
unity is all odd, occupying only a quarter of the screen, and badly cropped with these changes (from ppa:canonical-
Alan Pope 🍺🐧🐱 🦄 (popey) wrote : | #8 |
Florian asked me to try again with surfaceflinger instead of mir. Here's the result.. Similarly amusing, but different.
http://
http://
http://
http://
http://
http://
Florian Boucault (fboucault) wrote : | #9 |
Issues with MIR have been identified and worked on right now.
Issues with SurfaceFlinger are unexpected and installing the .debs manually from the qt5-beta-proper PPA does not let me reproduce it. I suspect the toolkit was not installed properly.
Timo Jyrinki (timo-jyrinki) wrote : | #10 |
A quick test result: looks about like Alan's surfaceflinger but this time on Mir.
Using:
apt-add-repository ppa:ci-
apt-add-repository ppa:canonical-
added with
apt-add-repository ppa:canonical-
... which has qtubuntu DPR branch bzr181 https:/
So more fixes needed.
Note: if you're using qt5-beta2 PPA instead of the two mentioned above, the results should be the same. Functionally the "final" landing PPA added with qt5-beta1 is almost identical to qt5-beta2. But if you reflash your device you may want to use landing-006 + qt5-beta1 instead (and add/remove qt5-beta-proper as needed).
Florian Boucault (fboucault) wrote : | #11 |
So, latest tests doing:
phablet-flash ubuntu-system --channel devel-proposed
phablet-config writable-image
Then on device:
apt-add-repository ppa:ci-
apt-add-repository ppa:canonical-
apt-get update
apt-get dist-upgrade
Plus downloading and installing the 3 debs from https:/
qtdeclarative5-
qtubuntu-
ubuntu-
The UI looks right but has some input issues. Working on it.
Florian Boucault (fboucault) wrote : | #12 |
Input issues with MIR is now resolved thanks to a patch in lp:ubuntu-mir.
Timo Jyrinki (timo-jyrinki) wrote : | #13 |
Hi! Test results with latest builds from the branches as they are curently:
Image now looks correct. However some issues remain:
- An odd graphical glitch in titles, see image
- It's hard to unlock screen, a swipe from outside of the screen does not work, instead it works when starting a little more inward from the border. See video.
Aside from those two problems, the functionality seems correct.
Instead of manually downloading deb:s, the qt5-beta-proper is made so that only those packages get updated. So:
apt-add-repository ppa:canonical-
apt-get update
apt-get dist-upgrade
yields:
The following packages will be upgraded:
libunity-mir1 qtdeclarative5-
4 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.
Need to get 699 kB of archives.
After this operation, 9216 B of additional disk space will be used.
Do you want to continue? [Y/n]
Get:1 http://
Get:2 http://
Get:3 http://
Get:4 http://
Fetched 699 kB in 6s (108 kB/s)
Timo Jyrinki (timo-jyrinki) wrote : | #14 |
Timo Jyrinki (timo-jyrinki) wrote : | #15 |
Timo Jyrinki (timo-jyrinki) wrote : | #16 |
One more note on the packages: the version numbers are meaningless, they are just mean to be a) above anything else in other PPA:s (including qt5-beta2 if someone still uses that), b) bump the upstream version number so that also orig tarball gets regenerated.
The changelogs include the information on what they actually are:
--
* Build from lp:~fboucault/qtubuntu/dpr_rebase_qt_5.1 bzr185
--
* Uploading from lp:~fboucault/ubuntu-ui-toolkit/dpr_rebase_qt_5.1 bzr763
* For this test build, also include the test fix Qt 5.2 branches too:
- lp:~zsombi/ubuntu-ui-toolkit/datepicker-fix1287009
- lp:~mzanetti/ubuntu-ui-toolkit/more-stable-ubuntulistview-tests
--
* Build from lp:~fboucault/unity-mir/dpr_rebase_qt_5.1 bzr187
description: | updated |
Tim Peeters (tpeeters) wrote : | #17 |
I installed today's trusty-proposed image, and added the ppas for landing-006, qt5-beta1 and qt5-beta-proper on maguro.
I have the same problems unlocking that mirv describes above, but after I manager, the chevron was a little bit too large (old size from before the font-size changes?).
When I change the height of the chevron image in our TabBarStyle.qml from units.dp(25) (this came with one of the packages from the PPA I guess) to 0.82*sourceSize
After purging the qt5-beta-proper ppa https:/
Tim Peeters (tpeeters) wrote : | #18 |
Summarizing: without qt5-beta-proper PPA chevron size is correct, with it chevron is a bit too large, and with height: 0.82*sourceSize
Timo Jyrinki (timo-jyrinki) wrote : | #19 |
Ok, correction, it may've been that LP was slow to publish the "published" binaries. Now looks like attachment. So, input issue the only issue?
Tim Peeters (tpeeters) wrote : | #20 |
height: units.dp(20) looks good for the chevron
Changed in ubuntu-ui-toolkit: | |
milestone: | february-2014 → march-2014 |
Florian Boucault (fboucault) wrote : | #21 |
Changes were made to the toolkit and qtubuntu MRs so that the new code path is only activated for the webbrowser-app and the webapps (using "webapp-
Timo Jyrinki (timo-jyrinki) wrote : | #22 |
I'm getting a build error with the qtubuntu bzr187:
error: 'gridUnit' was not declared in this scope
Timo Jyrinki (timo-jyrinki) wrote : | #23 |
There's the option B) from rsalveti of re-enabling DPR fix in qtwebkit. It's available in his PPA, but also with proper packaging very soon at:
ppa:canonical-
https:/
I don't know how to test the issue much, but the thing I do notice that Facebook app seems correct after the upgrade and too small without the upgrade.
The benefit of that ^ recompilation is that it can be directly copied to the landing PPA if tested to be fine.
Timo Jyrinki (timo-jyrinki) wrote : | #24 |
qtwebkit-
Pat McGowan (pat-mcgowan) wrote : | #25 |
Since we have the patched qtwebkit will remove the qt5.2 tag but leave open to track the eventual fix
tags: | removed: qt5.2 |
Changed in ubuntu-ui-toolkit: | |
milestone: | march-2014 → april-2014 |
Changed in ubuntu-ui-toolkit: | |
milestone: | april-2014 → june-2014 |
Changed in ubuntu-ui-toolkit: | |
importance: | Critical → High |
milestone: | june-2014 → none |
Changed in ubuntu-ui-toolkit (Ubuntu): | |
assignee: | nobody → Florian Boucault (fboucault) |
importance: | Undecided → High |
status: | New → In Progress |
no longer affects: | ubuntu-ui-toolkit |
Launchpad Janitor (janitor) wrote : | #26 |
This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1627+
---------------
ubuntu-ui-toolkit (1.3.1627+
[ Zsombor Egri ]
* Implement list item expansion.
* Turn AdaptivePageLayout page creation to be asynchronous.
* Fix ListItem swipe handling when gesture is initiated over an overlay
MouseArea which does not accept pressed event. Fixes LP: #1484545
* AbstractButton to C++. Fixes LP: #1365471, LP: #1458028
* Haptics singleton moved to C++.
* ActionItem moved to C++.
* Making columns resizable in an AdaptivePageLayout.
* Introducing column configuration into AdaptivePageLayout.
[ Benjamin Zeller ]
* Fix build inside qt and shadowbuild.
[ Tim Peeters ]
* Remove old and unused header-related files, and update unit test filenames.
* Don't unset the subheader style when Page.head changes because that unsets
the parent of Page.head.contents. Fixes LP: #1488922
* Fix the theming of the MainView.
* Fix the background color of the overflow panel.
[ Christian Dywan ]
* Include change signals with arguments in .api.
* A public QML type with no version is an error. Rather than silently breaking
as QML does normally at least apicheck can make this fatal.
* UbuntuShape shouldn't emit redundant deprecation warnings. Fixes LP: #1481791
* Update .bzrignore to new upstream style layout. Fixes LP: #1433308
* Revert unnecessary change of Popover minimumWidth. Fixes LP: #1483708
[ Ken VanDine ]
* Fixed a few typos in the api docs for StateSaver.
[ Olivier Tilloy ]
* Take custom key indexes into account to allow sorting on custom roles.
Fixes LP: #1485674
[ Albert Astals Cid ]
* Properly free shapeTextures[
will still be around when the opengl context is destroyed, but that seldom
happens so connect the signal to a lambda that will be there and does the
cleanup properly. Without this patch testDirectional
hitting the qFatal because it ran out of space in shapeTextures.
[ Richard Huddie ]
* Remove all usage of the autopilot simulated keyboard when the OSK is being
used. Fixes LP: 31483668
[ Gerry Boland ]
* Compensate for Qt's device pixel ratio multiplier.
Fixes: LP: #1207270, LP: #1275748, LP: #1468402
[ Nick Dedekind ]
* Added relative date time i18n.
[ CI Train Bot ]
* Resync trunk.
-- Zoltan Balogh <email address hidden> Tue, 08 Sep 2015 04:10:50 +0000
Changed in ubuntu-ui-toolkit (Ubuntu): | |
status: | In Progress → Fix Released |
Not a unity8 bug, then.