61830 - modern skin: progress bar lower border misaligned

Status: VERIFIED FIXED

(SeaMonkey :: Themes, defect, P2)

Description

[fudge_factor]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux 2.4.0-test11 i686; en-US; m18) Gecko/20001202
BuildID:    2000120206

The lower black line border that bounds the progress bar located at the lower
left corner of the modern skin, is out of alignment by 1 pixel.

Reproducible: Always
Steps to Reproduce:
1. Run mozilla
2. Look at progress bar.

Comment 1

[R.K.Aa.]

if the bottom line is supposed to be aligned text on the status-bar, it is
actually the *descenders* that are stretching one pixel too far down.
(left/right side borders)
Resolving as new, adding keywords.
Been seeing this one forever, currently Linux build ID 2000113006

Comment 2

[hangas]

Sending to Joe

Comment 3

[R.K.Aa.]

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

Comment 4

[R.K.Aa.]

OS: All.

Comment 5

[Joe Hewitt (gone)]

Fixeed - see fix for 16729.

Comment 6

[R.K.Aa.]

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

Comment 7

[Gilles Durys]

I still see this with build 2000122020 trunk win32.

Comment 8

[Gilles Durys]

And same with 2000122105 after a clean install

Comment 9

[sairuh (rarely reading bugmail)]

reopening. this is still a problem for me --using 2000.12.20.09 comm verif bits
on winNT.

Comment 10

[R.K.Aa.]

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

Comment 11

[R.K.Aa.]

[e-mail] comments in dup bug 63714:
modifying global.css helps a little

  .progressmeter-internal-box {
    vertical-align: middle;
    min-height    : 9px;
    height        : 9px;
  }

Comment 12

[Joe Hewitt (gone)]

I can't duplicate this at all.  If anyone sees anything wrong with the
progressmeter border, can you please post a screen shot illustrating what is wrong?

Comment 13

[sairuh (rarely reading bugmail)]

Attachment: download progress dialog, 2001.01.04.04 winNT build

Comment 14

[sairuh (rarely reading bugmail)]

Attachment: webpage status progress bar, 2001.01.04.04 winNT build

Comment 15

[sairuh (rarely reading bugmail)]

i've attached two screenshots from this morning's winNT comm verif bits. Joe, if
you need more info, feel free to ping me on irc [nick of 'se'], or i'll be happy
to help in person.

Comment 16

[Joe Hewitt (gone)]

I'm mightily confused as to why you are seeing this.  I am positive I checked in
a fix for this, and I am not able to duplicate it on any recent trunk builds.
The screen shots you are showing me demonstrate what USED TO happen when the
progress meter was stretched.  Are you testin with recent trunk builds or branch
builds?

Comment 17

[Boris Zbarsky [:bzbarsky]]

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

Comment 18

[Boris Zbarsky [:bzbarsky]]

According to bug 64782, this is still a problem in current nightlys.

Comment 19

[R.K.Aa.]

yes, i saw it too, in a download dialog. The "legs" there are longer than in the
progressbar in browser however. Browsers is only a pixel too deep, while the
ones in download dialog are hairyer - perhaps 7 pix too deep, and also
misaligned at top (descenders start one pixel too low, leaving the upper corners
"uncovered")

2001010821

Comment 20

[Gilles Durys]

Just a remark. I see this problem with windows nightlies, not linux nightlies

Comment 21

[Boris Zbarsky [:bzbarsky]]

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

Comment 22

[Stephen Walker]

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

Comment 23

[R.K.Aa.]

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

Comment 24

[Fabian Guisset]

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

Comment 25

[Jesse Ruderman]

Nominating for nsbeta1: trivial but very visible.

Comment 26

[Joe Hewitt (gone)]

Marking P1 since it's an easy fix and highly visible... the problem is the text 
element in the progressmeter binding is growing too large and expanding the 
borders.  I've tried hiding this element with display: none and visibility: 
collapse and neither works, so I tried font-size: 1px and that worked for me, 
but apparently on some systems it is too large... 

Comment 27

[hangas]

Themes Triage Team nsbeta1+

Comment 28

[Håkan Waara]

Attachment: [Patch] Fix

Comment 29

[Joe Hewitt (gone)]

r=hewitt on hwaara's patch

Comment 30

[hangas]

a=hangas

Comment 31

[Joe Hewitt (gone)]

fixed

Comment 32

[Patty Mac]

Marking verified on all platforms (2001-01-30-10-Mtrunk).