305673 - Problems redrawing bottom margin

Status: RESOLVED WORKSFORME

(Firefox :: General, defect)

Description

[Ximin Luo]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.7.10) Gecko/20050717 Firefox/1.0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.7.10) Gecko/20050717 Firefox/1.0.6

When a margin is set dynamically (using JS), the bottom margin is not redrawn.

More specifically: when a (?CSS class)? is added to a (?block-level) element

I havent tested setting the margin directly yet, nor testing it on inline elements

Reproducible: Always

Steps to Reproduce:
1.Go to http://webxl.5gigs.com/scratch/bugtest.html
2.Type "t6" into the top text-box and click "addClass"

Actual Results:  
3.Notice the lack of a bottom margin on the test <div>
4.Click the left link.
5.Then, click the right link.

Notice that the left link is static, and does not fix the re-draw. However the
right link is repositioned on :active and this also re-draws the bottom margin.


Expected Results:  
After 2), the bottom margin should have been redrawn and a 20px gap appear
between the bottom white border and the links.

Comment 1

[Ximin Luo]

Further information:

Bug doesn't show up for inline elements (partly because the vertical margins are
not set)

Bug DOES show up when setting the margin directly. (see updated page)

Bug is ONLY fixed when :active is repositioned; Positioning the static link
doesn't work.

http://webxl.5gigs.com/scratch/bugtest.html

Comment 2

[Jaime Mitchell (use [e-mail] for email)]

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

Comment 3

[Jaime Mitchell (use [e-mail] for email)]

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

Comment 4

[Stefan [:stefanh]]

Seems to worksforme with a one week old SeaMonkey trunk build (mac). Have you
tried it in a more recent build (like Deer park a2)?

Comment 5

[Ximin Luo]

(In reply to comment #4)
> Seems to worksforme with a one week old SeaMonkey trunk build (mac). Have you
> tried it in a more recent build (like Deer park a2)?

Ah, yes, it's fixed. Thanks for pointing that out :)

Comment 6

[Peter van der Woude [:Peter6]]

per last comment wfm