Fix cross-platform render issues with embedded fonts

Bug #1435486 reported by jus
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
High
Owen Williams

Bug Description

This is a follow up to lp:1431943 (Embed open skin fonts in Mixxx to improve cross-platform rendering)
There are reports about embedded fonts being rendered differently (aka hard to read) in the skins on the various supported platforms.

We should investigate the issues and fix if possible.

Possible reasons:
* QT treats font hinting different per OS http://doc-snapshots.qt.io/4.8/qfont.html#HintingPreference-enum
* Different DPI settings
* Skins do not consistently use embedded fonts
* Skins use sub-optimal font sizes

Possible solution:
* Settle for nice free fronts, a sans for text widgets and a sans monospaced for labels.
Because there seems to be a bias against Open Sans (thats what we currently embed) , we can try alternatives as Fira Sans/Source Code Pro
* Other apps use the freetype.org font raster engine instead of Qt`s own.

Tags: usability font
Revision history for this message
jus (jus) wrote :

As reported by Erikmjacobs (erikmjacobs)

Revision history for this message
jus (jus) wrote :

As reported by Erikmjacobs (erikmjacobs)

Revision history for this message
jus (jus) wrote :

Clipping button labels in screenshoot #2:

The Latenight stylesheet uses ``font-family: Lucida Grande, Helvetica Neue, Arial, Open Sans;``.
The first three fonts are not installed on stock Fedora 20, so the skin defaults to Open Sans (the embedded font).

Open Sans is not as narrow as the font used by the system LateNight was made with.
The LateNight skin uses fixed widths for the buttons, e.g. 55px for the ``SPINNIES`` button.
Possible fix: Make buttons resize dynamic to label text, increase fixed button size, decrease font size, select different embedded font

Small labels in screenshoot #1
The Deere Stylesheet uses only 6px as fixed label.
Possible fix: Increase font size, select different embedded font

---
How do fonts currently look like on High DPI screens?

Revision history for this message
Owen Williams (ywwg) wrote :

Please update to the latest master, I changed the font selection in LateNight

Revision history for this message
Erikmjacobs (erikmjacobs) wrote :

@owen when did you make that change? I grabbed some images today and am on latest master -- Latenight is still clipped and deere is still too small.

This is re: https://bugs.launchpad.net/mixxx/+bug/1431943

You asked me for the character map and I haven't had time to look at it yet.

Revision history for this message
jus (jus) wrote :

@owen
The updated font-family in the css still does fall back to Open Sans (see screenshoot in #2) , because the other fonts you selected in the Latenight css are not available on the users system by default. Open Sans is a bit more wide than the font you tested the button labels with on your system. You may test with the fallback font on your system to ensure it does look ok. See #3 for possible fix.

Revision history for this message
Owen Williams (ywwg) wrote :

Maybe we can bundle Open Sans Condensed? Making the buttons dynamically-sized would be really time-consuming

Revision history for this message
Owen Williams (ywwg) wrote :

Try this -- it looks like the scons script should just find the new font files. I tested this on my system and it worked nicely.

https://github.com/mixxxdj/mixxx/pull/540

Revision history for this message
Owen Williams (ywwg) wrote :

Can someone test the fix on windows? I think it's working everywhere else

Changed in mixxx:
status: Confirmed → Fix Committed
assignee: nobody → Owen Williams (ywwg)
Revision history for this message
Erikmjacobs (erikmjacobs) wrote :

Deere is still extremely tiny on F20.

Latenight and Shade seem to look OK.

I can compile on Windows if nobody else can, unless there's a build I can just download?

Revision history for this message
Owen Williams (ywwg) wrote :

Nightly windows builds are here: http://downloads.mixxx.org/builds/master/

I'll see if I can fix Deere, but I'll have to doublecheck that it doesn't break anything on OSX

RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7921

lock status: Metadata changes locked and limited to project staff
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.