Closed
Bug 924244
Opened 13 years ago
Closed 12 years ago
[B2G][Contacts] After exporting a contact the UI still displays Export in the top right
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect)
Tracking
(blocking-b2g:koi+, b2g-v1.2 fixed)
| Tracking | Status | |
|---|---|---|
| b2g-v1.2 | --- | fixed |
People
(Reporter: dwatson, Assigned: jmcf)
Details
(Keywords: regression, Whiteboard: burirun2)
Attachments
(2 files)
Description:
When the user exports a contact in the contact settings, the UI displays the word Export in the top right.
Repro Steps:
1. Load Buri build 20131007004003
2. Open up contacts app
3. Tap on the plus icon and add a contact
4. Tap on the Gear icon after creating a contact
5. Select "Export Contacts"
6. Tap on SIM card or Memory Card
7. Export a contact
8. Tap Done
Actual Results:
The word Exported sticks on the screen.
Expected:
After exporting a contact/s the UI will display the proper buttons and icons.(Gear icon and + icon)
Environmental Variables
Device: Buri 1.2 Mozilla
Build ID: 20131007004003
Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/f067799dff06
Gaia: 85c4af3f48a91878d565f518ba0eed68f0628e21
Platform Version: 26.0a2
Base Build: 20130912
Comment 1•13 years ago
|
||
Sounds like we're leaving behind broken UI then.
blocking-b2g: --- → koi?
Keywords: regression,
regressionwindow-wanted
Updated•13 years ago
|
QA Contact: nkot
Comment 2•13 years ago
|
||
Regression range:
-Reproduces-
Gaia def8e152db6a317162c03a316f68c409f3af3979
SourceStamp b955a00f4167
BuildID 20131002004001
Version 26.0a2
-Does NOT Reproduce-
Gaia 5e0d0df6a762cf1e1812eeb735fba72e2539dc0c
SourceStamp 5689e4237ab7
BuildID 20131001004003
Version 26.0a2
Keywords: regressionwindow-wanted
| Assignee | ||
Comment 3•13 years ago
|
||
Attachment #815321 -
Flags: review?(francisco.jordano)
Comment 4•13 years ago
|
||
Hi Jose,
Codewise looking perfect. In github I'm asking to rewrite the unit tests that we have to check exiting from select mode.
They didn't catch this error, and now that we are exposing the method to public, makes sense to rewrite the tests to use that function.
Once rewritten the tests and travis is green, please merge.
Thanks a lot!
F.
Updated•13 years ago
|
Attachment #815321 -
Flags: review?(francisco.jordano) → review+
Updated•13 years ago
|
Assignee: nobody → jmcf
Target Milestone: --- → 1.2 C3(Oct25)
| Assignee | ||
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 7•12 years ago
|
||
Uplifted 3d51729e128f4e74328dc57023ae999cf33f307f to:
v1.2: a802734900856f481fc7ea7d6c473f3b8c0a956f
You need to log in
before you can comment on or make changes to this bug.
Description
•