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.
Comment | File | Size | Author |
---|---|---|---|
#34 | drupal-seven-662940-34.patch | 973 bytes | sqndr |
#10 | Seven_Lucida_Grande.png | 304.41 KB | Jeff Burnz |
#10 | Seven_Lucida_Sans_Unicode.png | 310.15 KB | Jeff Burnz |
#10 | Seven_DejaVu_Sans.png | 309.91 KB | Jeff Burnz |
#10 | Seven_Lucida_Sans.png | 309.85 KB | Jeff Burnz |
Comments
Comment #1
ogi CreditAttribution: ogi commentedIt turns out that just adding
Lucida Sans
explicitly solves the issue.Comment #2
Dries CreditAttribution: Dries commentedCurious to see a screenshot (but not required).
Comment #3
ogi CreditAttribution: ogi commentedScreenshots 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.
Comment #5
Jeff Burnz CreditAttribution: Jeff Burnz commentedLike 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.
Comment #6
tim.plunkettBasically just a re-roll.
Comment #7
stephthegeek CreditAttribution: stephthegeek commentedI'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.
Comment #8
tim.plunkettCross-post. Adding a completely new font is up to Jeff.
Comment #9
Jeff Burnz CreditAttribution: Jeff Burnz commentedWell I think stephthegeek knows a lot more about fonts that I ever will :)
I take it the proposal is to do this:
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.
Comment #10
Jeff Burnz CreditAttribution: Jeff Burnz commentedBah, ignore those screens, try these ones, those grabbed my other screen as well and are huge, sorry.
Comment #11
tim.plunkettRead that as a feature request, I'm in that sort of mode.
I agree with this, it looks good.
Comment #12
ogi CreditAttribution: ogi commentedStill ugly in Linux.
Comment #13
tim.plunkettComment #14
Niklas Fiekas CreditAttribution: Niklas Fiekas commentedNeeds a reroll because of the new directory structure.
Comment #15
emclaughlin CreditAttribution: emclaughlin commentedThe 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?
Comment #16
emclaughlin CreditAttribution: emclaughlin commentedThe 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.
Comment #17
lucascaro CreditAttribution: lucascaro commented#16: drupal-seven-662940-16.patch queued for re-testing.
Comment #18
lucascaro CreditAttribution: lucascaro commented#16: drupal-seven-reordered-662940-16.patch queued for re-testing.
Comment #20
rootworkRerolled both patches from #16 against HEAD. As emclaughlin did, I'll leave it to Jeff or Tim to decide which is the better approach.
Comment #21
socketwench CreditAttribution: socketwench commentedNovice issue cleanup.
Comment #22
rootworkRealized I should unassign myself from this since it needs review -- though I'm guessing these patches no longer apply.
Comment #23
Bojhan CreditAttribution: Bojhan commentedThis needs Lewis to look at it.
Comment #24
LewisNyman20: drupal-seven-662940-20.patch queued for re-testing.
Comment #26
LewisNymanI'm happy with the idea of adding the new font to the stack (second patch in #20).
This needs a reroll to account for the change in file structure but after that we are ready to go.
Comment #27
rootworkRerolled.
Comment #28
LewisNymanI quickly checked to make sure the patch applies correctly. This stack has already been manually tested on a variety of machines so I think we're ready to go. Thanks.
Comment #30
Jeff Burnz CreditAttribution: Jeff Burnz commented27: drupal-seven-662940-27.patch queued for re-testing.
Comment #31
rootworkLooks like it's good again.
Comment #32
chx CreditAttribution: chx commentedWhile a little bit sad over the time this took, the attention to detail of the Drupal community humbles me.
Comment #34
sqndr CreditAttribution: sqndr commentedRe-rolling the patch.
Comment #35
rootworkThanks! Setting to review to trigger the testbot.
Comment #36
sqndr CreditAttribution: sqndr commentedSeems like testbot likes the patch? ;-)
Comment #37
LewisNymanLooks identical to me :)
Comment #38
alexpottCommitted dd24d32 and pushed to 8.x. Thanks!
Comment #41
ogi CreditAttribution: ogi commentedIs this going to be commited to 7.x?