QNX: Change base font sizes from pixels to points
authorKevin Krammer <kevin.krammer.qnx@kdab.com>
Thu, 18 Oct 2012 09:36:48 +0000 (11:36 +0200)
committerThe Qt Project <gerrit-noreply@qt-project.org>
Thu, 18 Oct 2012 20:24:16 +0000 (22:24 +0200)
Previously recommended fonts sizes in pixels do not work very well
on the new high resolution devices.

Current UX documentation recommends certain point sizes for certain
roles. Those work well on both type of devices.

Change-Id: Ia4faa4cf2d753cab9b10f91d01dfef1450683272
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
src/plugins/platforms/qnx/qqnxsystemsettings.cpp

index ae7cb67..6194a61 100644 (file)
@@ -48,11 +48,13 @@ QT_BEGIN_NAMESPACE
 
 QHash<QPlatformTheme::Font, QFont *> qt_qnx_createRoleFonts(QPlatformFontDatabase *fontDatabase)
 {
-    // See http://docs.blackberry.com/en/developers/deliverables/27299/Text_tablet_1526156_11.jsp
-    // which recommends using normal font size of 21 pixels and 36 pixels for titles (not covered
-    // by the theme system).
+    // See http://docs.blackberry.com/en/developers/deliverables/41577/typography.jsp
+    // which recommends using
+    // - small font size of 6 points
+    // - normal font size of 8 points
+    // - 11 points for titles (not covered by the theme system).
     QFont baseFont = fontDatabase->defaultFont();
-    baseFont.setPixelSize(21);
+    baseFont.setPointSize(8);
 
     QHash<QPlatformTheme::Font, QFont *> fonts;
     fonts.insert(QPlatformTheme::SystemFont, new QFont(baseFont));
@@ -70,7 +72,7 @@ QHash<QPlatformTheme::Font, QFont *> qt_qnx_createRoleFonts(QPlatformFontDatabas
     fonts.insert(QPlatformTheme::ComboLineEditFont, new QFont(baseFont));
 
     QFont smallFont(baseFont);
-    smallFont.setPixelSize(15);
+    smallFont.setPointSize(6);
     fonts.insert(QPlatformTheme::SmallFont, new QFont(smallFont));
     fonts.insert(QPlatformTheme::MiniFont, new QFont(smallFont));