51698 - Impossible to see lower part of letters in some auth dialog

Status: VERIFIED WORKSFORME

(Core :: XUL, defect, P5)

Description

[Henrik Gemal]

It's impossible to see the lower part of the letters types into the 
textfield of the authentication dialog.
Go to any auth site: http://gemal.dk/admin/log/reports/
and try to enter "gemal" into the text field. Then you cant see the lower part 
of the "g" entered. This is not the case in other textfields throughout the 
application.

Comment 1

[Henrik Gemal]

Attachment: Screent shot of problem

Comment 2

[Sylvain CUAZ]

WFM on Build 2000 09 06 04 and MacOs 9.0.4 on B/W G3

I've tried a few auth site (including yours) and none truncated any letter.

Comment 3

[Peter Trudelle]

I see this, at the given url, using the mozilla fonts on Linux, if that matters.
Cosmetic, ->future
Much worse is that I couldn't get rid of the auth dialog, it kept coming back
after OK, Cancel and close, so I had to destroy the window, and the app went
with it.  Is that a known bug?

Comment 4

[Peter Trudelle]

future, even.

Comment 5

[Henrik Gemal]

I see the problem on Win2000

Comment 6

[Peter Trudelle]

Not sure who should own this, is it part of common dialog? cc ben for possible
triage

Comment 7

[Blake Ross]

I see this in many places (not just in common dialogs), fwiw...

Comment 8

[Peter Trudelle]

Any idea where it belongs? editor?  box?

Comment 9

[Blake Ross]

I just looked into it a little, and it seems that any <textfield/>'s with 
type="text" have this problem.  I'm guessing this is a Ben thing...

Comment 10

[Peter Trudelle]

Okay, thanks.  ->ben for triage.

Comment 11

[Blake Ross]

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

Comment 12

[Blake Ross]

Probably a dup of bug 42821.  Leaving this open just in case it's not, and to 
remove type="text" from most or all of the textfields as a workaround (since I 
don't think it's necessary in any case).

Comment 13

[Henrik Gemal]

I've found type="text" on textfields here:
/mailnews/addrbook/resources/content/addressbook.xul, line 213
/mailnews/base/prefs/resources/content/am-main.xul, line 23
/mailnews/compose/resources/content/messengercompose.xul, line 456
/rdf/tests/domds/resources/DOMDataSourceViewer.xul, line 14
/xpfe/components/bookmarks/resources/bm-find.xul, line 190
/xpfe/components/search/resources/find.xul, line 44
/xpfe/global/resources/content/commonDialog.xul, line 31

should I provide a patch where I remove the type="text" ?

Comment 14

[Blake Ross]

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

Comment 15

[Blake Ross]

Attachment: [patch] this fixes it

Comment 16

[Blake Ross]

This fixes it, but I assume the 1px was added intentionally (added as part of 
bug 25580).  Rod, can the 1px be removed, or is a better fix just to remove 
type="text" from all these textfields?

Comment 17

[Stephen P. Morse]

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

Comment 18

[Blake Ross]

--> Rod for fix/answer to question

Comment 19

[rods (gone)]

works now

Comment 20

[Henrik Gemal]

v 20020221