|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
1.24 KB, text/plain
58 bytes, text/x-review-board-request
|Details | Review|
These files can be found at: https://mxr.mozilla.org/mozilla-central/search?string=appcompatcheckbox&find=mobile%2Fandroid&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central I think using <CheckBox (or whatever) directly should just work.
I would like to work on this bug but the link to the files doesn't seem to work. I'm assuming that the only AppCompatCheckBox that needs changing is in PromptInput.java. Is that correct?
(In reply to Kevin Lam from comment #2) > I would like to work on this bug but the link to the files doesn't seem to > work. I'm assuming that the only AppCompatCheckBox that needs changing is in > PromptInput.java. Is that correct? It's entirely possible that my initial hypothesis, that bug 1220309 would fix the issue, is incorrect. That being said, I don't think you should change the PromptInput value. afaik, the XML is changed to dynamically change CheckBox -> AppCompatCheckBox, but references in code are not updated. See here for a list of all uses of AppCompatCheckBox: https://dxr.mozilla.org/mozilla-central/search?q=path%3Amobile%2Fandroid+AppCompatCheck&redirect=false
Hi Michael! Have a look at the attachment 8816767 [details] and if it is fine then i will upload a patch soon.
Hi, Owais. I'm no longer working on Fennec – let me see if I can get someone else to help you. Sebastian, could you take a look at this or redirect it? Thanks!
Hi sebastian! https://reviewboard-hg.mozilla.org/gecko/rev/99b97f682498 Please review my code
Can you please assign this bug to me?
Hi sebastian! Please review my patch
This patch seems to have the wrong format. The following guide should help you to submit a patch to mozreview and link it with this bug: http://mozilla-version-control-tools.readthedocs.io/en/latest/mozreview/commits.html
(In reply to Owais Kazi from comment #7) > Hi sebastian! > https://reviewboard-hg.mozilla.org/gecko/rev/99b97f682498 Please review my > code It looks like you pushed the patch but it wasn't linked to this bug. Just include the bug number and push again: "Bug XXXXX - Commit message. r?sebastian".
Comment on attachment 8819347 [details] Bug 1220316-Removed explicit use of AppCompactCheckBox in the following files Part 2. . https://reviewboard.mozilla.org/r/99168/#review99452
I got this error in 'try' http://mibpaste.com/XhuPko. Can you help me with it?
Is this bug completed https://treeherder.mozilla.org/#/jobs?repo=try&revision=92f61c240cef or it does have error?
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/7be572e3697d Removed explicit use of AppCompactCheckBox in the following files Part 2. r=sebastian.
Is this bug fixed now?
(In reply to Owais Kazi from comment #17) > Is this bug fixed now? Yes, as soon as the patch is merged to mozilla-central this bug will be closed.