WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
5567
Use the theme font instead of "Lucida Grande" for buttons
https://bugs.webkit.org/show_bug.cgi?id=5567
Summary
Use the theme font instead of "Lucida Grande" for buttons
Rosyna
Reported
2005-10-30 17:11:54 PST
WebKit/khtml/css/html4.css hard codes Lucida Grande as the theme font for form controls rather than properly asking the system for the theme font. This makes it look out of whack if the user changes the theme font. This can be tested with the following command in the terminal (/Applications/Utilities/ Terminal): defaults write com.apple.Safari NSSystemFont ComicSansMS alternatively defaults write com.apple.Safari NSSystemFont Bauhaus93 or defaults write com.apple.Safari NSSystemFont Copperplate Depending on how ugly you want the result to be. Hardcoding the theme font is bad.
Attachments
Add attachment
proposed patch, testcase, etc.
Joost de Valk (AlthA)
Comment 1
2006-02-13 15:57:50 PST
Reassigning to webkit-unassigned, to make sure more people see this.
Joost de Valk (AlthA)
Comment 2
2006-02-15 15:34:26 PST
Seems very valid. This is an enhancement though, not a bug.
Sören Kuklau
Comment 3
2006-02-15 17:12:41 PST
> This is an enhancement though, not a bug.
Is it? I'd be surprised if Apple doesn't have a guideline somewhere not to hardcode fonts when there's no real need to. In which case it would be a bug.
Alexey Proskuryakov
Comment 4
2006-02-15 22:12:24 PST
I don't see this problem at the provided URL. However, I do see it on some Bugzilla pages, such as <
http://bugzilla.opendarwin.org/attachment.cgi?id=6473&action=edit
>. I think that some localizations use something other than 11px (?!) Lucida Grande as a system font; this bug would be more important for those. Based on the comment in html4.css, I'm setting this as dependent on
bug 3303
. And I also think that this is a bug, not an enchancement request.
Dave Hyatt
Comment 5
2006-08-15 15:36:40 PDT
Fixed.
David Kilzer (:ddkilzer)
Comment 6
2006-08-15 16:23:06 PDT
r15888
with layout tests in
r15889
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug