bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

mouse over radio button triggers reflow

VERIFIED FIXED in M9

Status

()

Core
Event Handling
P3
normal
VERIFIED FIXED
19 years ago
19 years ago

People

(Reporter: karnaze (gone), Assigned: rods (gone))

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

19 years ago
I'm not sure if this is event handling or form controls. Tables are getting
reflowed unnecessarily when the mouse goes over a radio button on the above url
or the test case below. There is different bug associated with this in that the
area frame reflow is causing the radio buttons to dissappear when the mouse goes
over them.

<TABLE WIDTH="100%" BORDER="0" CELLPADDING="15" CELLSPACING="0">
 <TR>
  <TD WIDTH="100%" VALIGN="top">foo</TD>
  <TD BGCOLOR="#FFFFFF" WIDTH="175" VALIGN="TOP">
   <table width="100%" cellspacing="0" cellpadding="0" border="0">
    <tr>
     <td bgcolor="#669900">
      <form>
       <input type="radio" name="vote" value="1">
      </form>
     </td>
    </tr>
   </table>
  </TD>
 </TR>
</TABLE>
foobar
(Reporter)

Updated

19 years ago
Assignee: joki → kmcclusk
Target Milestone: M9
(Reporter)

Comment 1

19 years ago
Reassigning to Kevin, CCing Tom.
(Assignee)

Comment 2

19 years ago
Removing the :hover rule in the ua.css doesn't do anything, but removing
:-moz-radio does. Or is I comment out:
nsFormControlFrame::CaptureStyleChangeFor(this, oldRadioButtonFaceStyle,
                                          mRadioButtonFaceStyle,
                                          aParentChange, aChangeList,
                                          aLocalChange);

inside ReResolveStyleContext it works fine.
(Assignee)

Comment 3

19 years ago
The newly resolved for style "-moz-radio" is changed, the only two values that
are different are mTextAlign went from 0 to 1, and mLineHeight is now Inherit
instead of Normal. Any ideas?
(Assignee)

Updated

19 years ago
Assignee: kmcclusk → rods
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 4

19 years ago
Through Kevin's suggestion I change the way the original radioPseudo style gets
created in the CSSFrameConstructor.

Comment 5

19 years ago
i am not sure if this is what you mean, but when you mouseover the radio
buttons on mozillzine, the entire page gets redrawn but without that table.
(Assignee)

Comment 6

19 years ago
I have the fix for this, the pseudo class needed to be created with it's
paernt's style context.
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 7

19 years ago
Changed the input params so the pseudoContext for the radio button could be
created with it parent's style context. It fixed.

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 8

19 years ago
Seems to work fine now.
Marking verified
[19990901]
You need to log in before you can comment on or make changes to this bug.