Font size too big on phone, and display too tall with devicePixelRatio

Bug #1532950 reported by Andrea Cimitan
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu Terminal App
Fix Released
Undecided
Niklas Wenzel
ubuntu-terminal-app (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

as summary, needs some size adjustments with devicePixelRatio*

* To enable manually, easiest is to edit /etc/environment and add the line "QT_DEVICE_PIXEL_RATIO=2", and reboot.

Tags: qt-dpr

Related branches

Revision history for this message
Stefano Verzegnassi (verzegnassi-stefano) wrote :

My 2 cents. Would it make more sense to use the font size of a UITK Label (textSize: Label.Medium) as a reference unit, and provide only a setting for a multiplier of the resulting pixelSize?

That way the font size would be automatically adjusted according to GRID_UNIT_PX or QT_DEVICE_PIXEL_RATIO variables, and it would behave consistently with the rest of the platform.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in ubuntu-terminal-app (Ubuntu):
status: New → Confirmed
Revision history for this message
SB (emehntehtt) wrote :

After OTA-9 fonts are too large on Terminal, should be size 11 on my phone but it appears bigger, also it seems to be a wider problem as font it too large in messaging application too. When you open a sms the sender's name is too big.

Revision history for this message
Niklas Wenzel (nikwen) wrote :

There have been some comments on how that was fixed in other apps in the following mailing list thread: https://lists.launchpad.net/ubuntu-phone/msg17950.html

They opted for font.pixelSize in combination with units.gu() instead of using font.pointSize.

However, units.gu() probably isn't the best option. Stefano is right in my opinion and we should probably use something based around the FontUtils component [1].

I'll give this a try as soon as I find some time. :)

[1] https://developer.ubuntu.com/api/apps/qml/sdk-15.04.1/Ubuntu.Components.FontUtils/

Changed in ubuntu-terminal-app:
assignee: nobody → Niklas Wenzel (nikwen)
Niklas Wenzel (nikwen)
Changed in ubuntu-terminal-app:
status: New → In Progress
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Can't add this to the MP, so I'm adding it here.

Revision history for this message
Niklas Wenzel (nikwen) wrote :

Another one.

Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :

Fix committed into lp:ubuntu-terminal-app at revision 165, scheduled for release in ubuntu-terminal-app, milestone 2014-12-11

Changed in ubuntu-terminal-app:
status: In Progress → Fix Committed
Niklas Wenzel (nikwen)
Changed in ubuntu-terminal-app:
status: Fix Committed → Fix Released
Changed in ubuntu-terminal-app (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.