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)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 26
People
(Reporter: wesj, Assigned: wesj)
Details
Attachments
(1 file)
2.18 KB,
patch
|
Margaret
:
review+
|
Details | Diff | Splinter 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.
Assignee | ||
Updated•11 years ago
|
Attachment #786127 -
Attachment is patch: true
Attachment #786127 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 1•11 years ago
|
||
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 2•11 years ago
|
||
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+
Assignee | ||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/7f999fbb7422
Comment 4•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7f999fbb7422
Assignee: nobody → wjohnston
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•