Closed Bug 1021343 Opened 6 years ago Closed 6 years ago
[Settings] Duplicated Homescreen entry
21.61 KB, image/png
10.15 KB, image/png
12.67 KB, image/png
46 bytes, text/x-github-pull-request
|Details | Review|
Steps to reproduce: 1. Open Settings app 2. Scroll down until "Account Management" shows at the bottom of the screen. Actual results: You should see the entries: - Homescreen - Notifications - Date & Time - Language - Homescreens - Keyboards Notice that "Homescreen" is duplicated. Expected results: See the entries: - Homescreen - Notifications - Date & Time - Language - Homescreens - Keyboards Without duplicated "Homescreen". Environment information: - OS version: 184.108.40.206-prerelease - Hardware revision: qcom - Platform version: 31.0a1 - Build Identifier: 20140422082646 - Git commit info: 2014-06-05 20:54:15 bc40fdbe
We had this fixed, not sure what caused it to appear again.
Component: Gaia::Settings → Gaia::Build
Assignee: nobody → kgrandon
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Target Milestone: --- → 2.0 S4 (20june)
Hey guys, anyone got time for a review? I think what happened is a commit got backed out that this one was working with?? At least I saw something about a BUILD_STAGE thing getting backed and and this not working. At least we have a test now.
Comment on attachment 8440249 [details] [review] Github pull request LGTM r=me although Yurenju is the expert here
Attachment #8440249 - Flags: review?(crdlc) → review+
Comment on attachment 8440249 [details] [review] Github pull request r=me too, thanks Kevin to fix it and this test case.
Attachment #8440249 - Flags: review?(yurenju.mozilla) → review+
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Comment on attachment 8440249 [details] [review] Github pull request This is a rather simple patch that's necessary for the vertical homescreen and has been well tested.
Attachment #8440249 - Flags: approval-gaia-v2.0?(bbajaj)
Attachment #8440249 - Flags: approval-gaia-v2.0?(bbajaj) → approval-gaia-v2.0+
This issue is verified fixed on Flame 2.1 Results: Homescreen is not duplicated in settings Flame 2.1 Device: Flame 2.1 (319mb)(Kitkat Base)(Shallow Flash) BuildID: 20141125001201 Gaia: 1bdd49770e2cb7a7321e6202c9bf036ab5d8f200 Gecko: db893274d9a6 Version: 34.0 (2.1) Firmware: V188-1 User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
You need to log in before you can comment on or make changes to this bug.