Closed
Bug 1214939
Opened 9 years ago
Closed 9 years ago
Black Material check in Android 4* password dialog
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox45 fixed, b2g-v2.5 fixed, fennec44+)
RESOLVED
FIXED
Firefox 45
People
(Reporter: mcomella, Assigned: mcomella)
References
Details
Attachments
(2 files)
It's strange because Gecko*DialogBase inherits from Holo styles – I'm guessing this doesn't inherit from that.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → michael.l.comella
Comment 1•9 years ago
|
||
(In reply to Michael Comella (:mcomella) from comment #0)
> It's strange because Gecko*DialogBase inherits from Holo styles – I'm
> guessing this doesn't inherit from that.
But the EditText actually has Holo styles. :)
Assignee | ||
Comment 2•9 years ago
|
||
44+ for now to follow Theme.AppCompat but I feel this is lower priority than the other regressions and bugs I'm working on.
tracking-fennec: --- → 44+
Assignee | ||
Comment 3•9 years ago
|
||
We found out these are not being inflated correctly in bug 1220196 so we explicitly used the AppCompatCheckBox. It appears this one is inflated in code, so we should also explicitly use the AppCompatCheckBox. However, unlike the other bug, this is not likely to be a work around because I don't think AppCompat dynamically injects `AppCompatCheckBox`es in code, like it does in xml.
Assignee | ||
Comment 4•9 years ago
|
||
Bug 1214939 - Explicitly use AppCompatCheckBox in PromptInput. r=sebastian
This way it inherits from the AppCompat styles like accentColor.
Attachment #8681539 -
Flags: review?(s.kaspari)
Comment 5•9 years ago
|
||
Comment on attachment 8681539 [details]
MozReview Request: Bug 1214939 - Explicitly use AppCompatCheckBox in PromptInput. r=sebastian
https://reviewboard.mozilla.org/r/23853/#review21345
Nice and simple. :)
Attachment #8681539 -
Flags: review?(s.kaspari) → review+
Assignee | ||
Comment 6•9 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/af1868f1fc78a8aa6e3c8d694b07ba723563c827
Bug 1214939 - Explicitly use AppCompatCheckBox in PromptInput. r=sebastian
Comment 7•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Comment 8•9 years ago
|
||
bugherder uplift |
status-b2g-v2.5:
--- → fixed
Updated•4 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
•