changing image in titledbutton forces reflow

VERIFIED INVALID

Status

()

Core
XUL
P3
normal
VERIFIED INVALID
18 years ago
18 years ago

People

(Reporter: Mike Pinkerton (not reading bugmail), Assigned: Eric Vaughan)

Tracking

({css-moz})

Trunk
css-moz
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: need to rewrite titledbutton to use pseudo-style)

(Reporter)

Description

18 years ago
turn on paint flashing, mouse over a gfx scrollbar (like in bookmarks window) and
you'll see the whole tree reflow/repaint.

this is happening because the :hover rule for each component of the titledbutton
sets the list-style-image property in CSS. While the titledbutton code can tell
that the new image is the same size as the old one and only a paint is needed,
the style system assumes a reflow is needed.
(Reporter)

Updated

18 years ago
Assignee: pinkerton → evaughan
Summary: mousing over gfx scrollbars causes extra reflow → changing image in titledbutton forces reflow
Whiteboard: need to rewrite titledbutton to use pseudo-style
Target Milestone: M15
(Reporter)

Comment 1

18 years ago
after talking to evaughan and peterl, we need to suck up and deal and do the work
to not use the list-stlye-image tag but use a pseudo-element and the content
property.

changing the summary, moving out to M15, assigning to evaughan
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Summary: changing image in titledbutton forces reflow → {css-moz} changing image in titledbutton forces reflow
Keywords: css-moz

Updated

18 years ago
Summary: {css-moz} changing image in titledbutton forces reflow → changing image in titledbutton forces reflow
(Assignee)

Comment 2

18 years ago
targeting
Target Milestone: M15 → M16
(Assignee)

Updated

18 years ago
Target Milestone: M16 → M17

Comment 3

18 years ago
Mass moving M17 bugs to M18
Target Milestone: M17 → M18
(Assignee)

Comment 4

18 years ago
titledbutton are deprecated and should never be used. This bug is now invalid.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → INVALID

Comment 5

18 years ago
allrighty then. Marking VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.