Closed Bug 589534 Opened 10 years ago Closed 10 years ago

Fix netError button for the Modern theme

Categories

(SeaMonkey :: Themes, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: neil, Assigned: neil)

Details

(Keywords: modern)

Attachments

(1 file)

netError.xhtml now uses an HTML button instead of a XUL button. We need to give it Modern style.
Attached patch Proposed patchSplinter Review
* Made button.css work for both XHTML and XUL buttons
* Added rules to remove the XHTML focus outline
* Imported button.css from netError.css

Conveniently we don't need to override moz-appearance because it is automatically disabled for styled XHTML buttons.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #468126 - Flags: review?(iann_bugzilla)
Comment on attachment 468126 [details] [diff] [review]
Proposed patch

r=me does the job, seems to have the same style as to before the core changes.
Attachment #468126 - Flags: review?(iann_bugzilla) → review+
Pushed changeset 32ab06e4961a to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.