Download & Extend

"Lucida Sans Unicode" may match ugly "Lucida Bright" instead of "Lucida Sans"

Project:Drupal core
Version:8.x-dev
Component:Seven theme
Category:bug report
Priority:normal
Assigned:emclaughlin
Status:needs review
Issue tags:Novice, Quick fix

Issue Summary

Seven theme uses the following font preferences:

font-family: Lucida Grande, Lucida Sans Unicode, sans-serif;

On my system Lucida Sans Unicode selects Lucida Bright which is serif font and looks ugly due to lack of hinting. The only Lucida fonts on my system (Debian 5.0 Lenny) are those installed by sun java: Lucida Bright and Lucida Sans.

When I replace Lucida Sans Unicode with Lucida Sans (or Lucida), Lucida Sans is selected and it looks as expected, that is, great.

Please change font-family in such a way so that Lucida Bright is not selected by fontconfig. Seven is default admin theme and install theme, so I think it's important this to be fixed.

Comments

#1

Title:Lucida can match bad font on some installations» "Lucida Sans Unicode" may match ugly "Lucida Bright" instead of "Lucida Sans"
Status:active» needs review

It turns out that just adding Lucida Sans explicitly solves the issue.

AttachmentSizeStatusTest resultOperations
seven-lucida-bright.diff621 bytesIdleFailed to install Drupal on MySQL 5.0 InnoDB.View details | Re-test

#2

Curious to see a screenshot (but not required).

#3

Screenshots are attached for before and after applying the patch. The combination of low dpi, small size, serif font and no hinting is what makes this professional font look ugly. I took the opportunity to demonstrate (in after patch screenshoot d7-lucida-sans.png) that Java's Lucida Sans doesn't have italics, so system default sans-serif is used for italics (DejaVu Sans) - it looks like bold italics compared to Lucida Sans but it's regular italics.

As I see it, Lucida Grande is Mac font, Lucida Sans Unicode is Windows font, so the natural choice for Linux would be DejaVu Sans because it's installed by default practically on all Linux distributions.

AttachmentSizeStatusTest resultOperations
d7-lucida-bright.png175.76 KBIgnoredNoneNone
d7-lucida-sans.png166.6 KBIgnoredNoneNone

#4

Status:needs review» needs work

The last submitted patch failed testing.

#5

Like to hear some more feedback on this, seems like a very easy fix. We have a lot of linux users and no real complaints from them over the past year, although offline I have heard gripes regarding Sevens font stack.

#6

Status:needs work» needs review

Basically just a re-roll.

AttachmentSizeStatusTest resultOperations
drupal-seven-662940-5.patch912 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 25,498 pass(es).View details | Re-test

#7

Status:needs review» needs work

I've got Lucida Sans installed, which is pretty common on a Linux system as part of the Java package. But DejaVu Sans is default and even more common, according to http://www.codestyle.org/css/font-family/sampler-UnixResults.shtml

But Lucida Sans is also fairly common on Windows/Mac, so to target Linux specifically and provide additional fallback for other systems, I would vote to add DejaVu Sans to the stack just ahead of Lucida Sans.

#8

Status:needs work» needs review

Cross-post. Adding a completely new font is up to Jeff.

#9

Well I think stephthegeek knows a lot more about fonts that I ever will :)

I take it the proposal is to do this:

"Lucida Grande", "Lucida Sans Unicode", "DejaVu Sans", "Lucida Sans", sans-serif;

I've added some screen shots to show each font and the subtle differences. Obviously on my system Lucida Grande gets selected although even then in IE9 with the full stack the kerning gets ever so slightly adjusted, which is kind of weird.

Frankly I'm a big fan of Bitstream fonts and would not be opposed to adding DejaVu Sans to the stack.

FWIW I am just one lone voice in the cacophony - my job is more about managing this queue, reviewing patches etc, the real decisions are made by everyone.

AttachmentSizeStatusTest resultOperations
Seven_Lucida_Grande.png1.07 MBIgnoredNoneNone
Seven_Lucida_Sans_Unicode.png1.07 MBIgnoredNoneNone
Seven_DejaVu_Sans.png1.06 MBIgnoredNoneNone
Seven_Lucida_Sans.png1.07 MBIgnoredNoneNone

#10

Bah, ignore those screens, try these ones, those grabbed my other screen as well and are huge, sorry.

AttachmentSizeStatusTest resultOperations
Seven_Lucida_Grande.png304.41 KBIgnoredNoneNone
Seven_Lucida_Sans_Unicode.png310.15 KBIgnoredNoneNone
Seven_DejaVu_Sans.png309.91 KBIgnoredNoneNone
Seven_Lucida_Sans.png309.85 KBIgnoredNoneNone

#11

Read that as a feature request, I'm in that sort of mode.

I agree with this, it looks good.

AttachmentSizeStatusTest resultOperations
drupal-seven-662940-10.patch942 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 25,500 pass(es).View details | Re-test

#12

Still ugly in Linux.

#13

Version:7.x-dev» 8.x-dev

#14

Status:needs review» needs work

Needs a reroll because of the new directory structure.

#15

The problem that I'm seeing is that, because Lucida Sans Unicode is before Lucida Sans, Linux is still matching it to Lucida Bright for whatever reason. Is there a particular reason why Lucida Sans Unicode is preferred over Lucida Sans, or could Lucida Sans go in front of it?

#16

Assigned to:Anonymous» emclaughlin
Status:needs work» needs review

The first patch is just the latest patch rerolled for the new directory structure. The second patch reorders the list of fonts to be "Lucida Grande", "Lucida Sans", "Lucida Sans Unicode" since ogi said the problem still existed with them ordered with Lucida Sans at the end.

I'm not sure which one would be more appropriate, but they were both easy enough to do that I figured it wouldn't hurt to leave them here for someone else to make the decision on.

AttachmentSizeStatusTest resultOperations
drupal-seven-662940-16.patch795 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 34,388 pass(es).View details | Re-test
drupal-seven-reordered-662940-16.patch795 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 34,385 pass(es).View details | Re-test
nobody click here