125432 - Font Too Small in Form Input Fields

Status: VERIFIED FIXED

(Core :: Layout: Form Controls, defect)

Description

[Felix Miata]

2002021216

See bug 124164. Font used when that bug was filed was a user specified font big
enough to read. Current font is the same size used for chrome, which is
marginally useful for chrome and unacceptably small for normal page text.
Compare to the font size used for Netscape 4, which uses a user specified font
size for forms input.

Comment 1

[Mike Kaply [:mkaply]]

Ok this is definitely the same as 53617

I changed forms.css myself for 0.9.8 which is why it worked when you opened the 
other bug.

Nightlies do not have this change.

You can manually edit forms.css as is mentioned on Steve Wendt's page:

http://www.os2bbs.com/os2news/Warpzilla.html

*** This bug has been marked as a duplicate of 53617 ***

Comment 2

[Felix Miata]

I read bug 53617, and it doesn't look like it is necessarily a functional
equivalent of this. This is not a serif vs. sans-serif issue.

Manual edit of forms.css is not a practical fix. I download a nightly virtually
every day, and would have to make this manual change each time, since I have no
way to know I would not blow away other fixes by copying over a previously
edited version each time.

Comment 3

[Mike Kaply [:mkaply]]

Attachment: Fix for problem

There was already some platform specific code in nsRuleNode.cpp to handle font
stuff, and I decided to add OS/2 to the mix.

On OS/2, for entryfields, lists, and buttons, we will use the same font as the
webpage content unless we are in 4.x quirks mode where we will use monospaced
for entryfields.

This means that you will never get a pretty WarpSans font for controls on Web
Pages on OS/2, but this is necessary on OS/2 because WarpSans is a bitmap font
and does not scale. It should not be used for web content.

Looking for review

Comment 4

[Mike Kaply [:mkaply]]

*** Bug 116629 has been marked as a duplicate of this bug. ***

Comment 5

[rods (gone)]

Comment on attachment 73555 [details] [diff] [review]
Fix for problem

r=rods

Comment 6

[Johnny Stenback (:jst)]

Comment on attachment 73555 [details] [diff] [review]
Fix for problem

sr=jst

Comment 7

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 73555 [details] [diff] [review]
Fix for problem

a=roc+moz

Comment 8

[Mike Kaply [:mkaply]]

Fix checked in.

Comment 9

[Felix Miata]

Verified