Closed
Bug 130632
Opened 23 years ago
Closed 15 years ago
Mac device context code for getting system "field", "button", etc. fonts sucks
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: bzbarsky, Unassigned)
Details
In nsDeviceContextMac :: GetSystemFont we have the following code:
282 if (aID == eSystemFont_Window ||
283 aID == eSystemFont_Document ||
284 aID == eSystemFont_Button ||
285 aID == eSystemFont_PullDownMenu ||
286 aID == eSystemFont_List ||
287 aID == eSystemFont_Field) {
288 aFont->name.Assign(NS_LITERAL_STRING("sans-serif"));
289 aFont->size = NSToCoordRound(aFont->size * 0.875f); // quick hack
290 }
At least for eSystemFont_List this is utterly wrong -- see bug 53617
Comment 1•23 years ago
|
||
From the "parent" bug 25657:
James Lariviere wrote:
> mozilla uses a fixed font for
> textareas while everything else is proportional. Seems reasonable that
> textareas should use the same font as the other controls.
And, just to be clear, this should surely default to monospaced (so that the WIDTH attribute
of TEXT TYPE="INPUT" actually has any useful function in HTML at all), and only become
proportional/variable width IF the page author manually codes it that way. Right?
Comment 2•23 years ago
|
||
No, it should be monospace for input text, we are now using sans-serif, which is
what IE uses which is what most people now expect.
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → Future
Comment 3•23 years ago
|
||
cc'ing myself
Updated•22 years ago
|
Whiteboard: [dev notes]
Updated•22 years ago
|
Whiteboard: [dev notes] → [whitebox]
Comment 4•21 years ago
|
||
This bug is targeted at a Mac classic platform/OS, which is no longer supported
by mozilla.org. Please re-target it to another platform/OS if this bug applies
there as well or resolve this bug.
I will resolve this bug as WONTFIX in four weeks if no action has been taken.
To filter this and similar messages out, please filter for "mac_cla_reorg".
Reporter | ||
Comment 5•21 years ago
|
||
Simon, does OSX share this code?
Updated•21 years ago
|
OS: Mac System 9.x → MacOS X
Comment 6•21 years ago
|
||
Yes
Updated•18 years ago
|
QA Contact: ian → style-system
Comment 7•16 years ago
|
||
This bug hasen't been touched for years and is clearly unowned. Moving back to default assignee/QA so that people, who are watching those can accurately triage this bug.
Also resetting Priority, Target Milestone, Status Whiteboard and Status.
Assignee: rods → nobody
Status: ASSIGNED → NEW
Priority: P2 → --
Whiteboard: [whitebox]
Target Milestone: Future → ---
Reporter | ||
Comment 8•16 years ago
|
||
Not sure whether this is still an issue with thebes, but over to the right component for now. It does look like nsSystemFontsMac::GetSystemFont no longer has this bug.
Component: Style System (CSS) → GFX: Thebes
QA Contact: style-system → thebes
Comment 9•15 years ago
|
||
Yeah, it only does it for font: -moz-window and -moz-document now, and I don't see those being used anywhere.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•