Use a xul:checkbox for "Only user styles" option

RESOLVED FIXED in Firefox 10

Status

()

Firefox
Developer Tools
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: miker, Assigned: dao)

Tracking

unspecified
Firefox 10
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [styleinspector])

Attachments

(1 attachment, 4 obsolete attachments)

When xul:checkboxes are used in xhtml docs they are rendered incorrectly
Depends on: 672748
Created attachment 571280 [details] [diff] [review]
Patch used to reproduce the problem

To reproduce:
1. Go to the web console (Firefox -> Web Developer -> Web Console)
2. Enter "inspectstyle(document.body)"

The style inspector will appear with a xul:checkbox from the land of ugly (see attached image).

Clicking the checkbox, which looks more like a text field, does nothing.
Created attachment 571285 [details]
Screenshot of xul:checkbox from the land of ugly
Isn't that checkbox there this one:

+        <input class="onlyuserstyles" save="${onlyUserStylesCheckbox}"
+               type="checkbox" onchange="${onlyUserStylesChanged}" checked=""/>

?  If so, that's an html:input, not a xul:checkbox, unless I'm seriously missing something....
Created attachment 571322 [details] [diff] [review]
Patch used to reproduce the problem

(In reply to Boris Zbarsky (:bz) from comment #3)
> Isn't that checkbox there this one:
> 
> +        <input class="onlyuserstyles" save="${onlyUserStylesCheckbox}"
> +               type="checkbox" onchange="${onlyUserStylesChanged}"
> checked=""/>
> 
> ?  If so, that's an html:input, not a xul:checkbox, unless I'm seriously
> missing something....

lol ... I attached the wrong patch. It is reproducible with this one.
Attachment #571280 - Attachment is obsolete: true
That last patch doesn't apply to m-c.  On m-c there is no <xul:hbox class="header" flex="1"> or <xul:textarea>.  Instead there is a <div class="filters"> and an HTML <input>.
(Assignee)

Comment 6

6 years ago
Created attachment 572438 [details] [diff] [review]
WIP patch

Michael's patch was missing align="center".
Assignee: nobody → dao
Attachment #571285 - Attachment is obsolete: true
Attachment #571322 - Attachment is obsolete: true
Status: NEW → ASSIGNED
(Assignee)

Updated

6 years ago
Component: XUL → Developer Tools
Product: Core → Firefox
QA Contact: xptoolkit.widgets → developer.tools
Summary: xul:checkbox rendered incorrectly when used in xhtml docs → Use a xul:checkbox for "Only user styles" option
Whiteboard: [styleinspector]
(Assignee)

Comment 7

6 years ago
Created attachment 572441 [details] [diff] [review]
patch
Attachment #572438 - Attachment is obsolete: true
Attachment #572441 - Flags: review?(mihai.sucan)
Comment on attachment 572441 [details] [diff] [review]
patch

 
-.onlyuserstyles,
-.userStylesLabel {
+.onlyuserstyles {

not a fan of this capitalization (would prefer onlyUserStyles).

but here we are.
Attachment #572441 - Flags: review+
(Assignee)

Updated

6 years ago
Attachment #572441 - Flags: review?(mihai.sucan)
(Assignee)

Comment 9

6 years ago
https://hg.mozilla.org/integration/fx-team/rev/900478e42d35
Target Milestone: --- → Firefox 10
So is there still a core rendering issue I should be worried about here?
(Assignee)

Comment 11

6 years ago
(In reply to Boris Zbarsky (:bz) from comment #10)
> So is there still a core rendering issue I should be worried about here?

nope
https://hg.mozilla.org/mozilla-central/rev/900478e42d35
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.