Closed Bug 901822 Opened 11 years ago Closed 11 years ago

Expanders in error pages should toggle state

Categories

(Firefox for Android Graveyard :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 26

People

(Reporter: wesj, Assigned: wesj)

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
While retheming these, I noticed that the expanders on about:certerror don't toggle state like they do on desktop. We should try to unify our code between these two, but for now, we should just fix ours.
Attachment #786127 - Attachment is patch: true
Attachment #786127 - Flags: review?(mark.finkle)
Comment on attachment 786127 [details] [diff] [review]
Patch

Moving this review to margaret, because mfinkle's a busy man (and the error pages theming depends on this).
Attachment #786127 - Flags: review?(mark.finkle) → review?(margaret.leibovic)
Comment on attachment 786127 [details] [diff] [review]
Patch

Review of attachment 786127 [details] [diff] [review]:
-----------------------------------------------------------------

This looks like it will work with the styles in bug 808636.
Attachment #786127 - Flags: review?(margaret.leibovic) → review+
https://hg.mozilla.org/mozilla-central/rev/7f999fbb7422
Assignee: nobody → wjohnston
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: