Closed Bug 949982 Opened 11 years ago Closed 10 years ago

[Gaia][DSDS] Fine tune SIM manager UI

Categories

(Firefox OS Graveyard :: Gaia, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(firefox29 fixed)

RESOLVED FIXED
Tracking Status
firefox29 --- fixed

People

(Reporter: arthurcc, Assigned: eragonj)

References

Details

Attachments

(3 files)

SIM manager UI should follow the spec[1].

[1]: https://bugzilla.mozilla.org/attachment.cgi?id=8347148
Depends on: 949986
No longer depends on: 949986
Assignee: nobody → ejchen
Attached file patch on github
Hi Fang,

can you help me review the UI for SimManager ? 

Thanks :D
Attachment #8356463 - Flags: ui-review?(fshih)
Attached image screenshot.png
Attachment #8356464 - Flags: ui-review?(fshih)
Comment on attachment 8356465 [details]
screenshot2.png

Hi Ej,

Good to go! Thanks for the effort!
Attachment #8356465 - Flags: ui-review?(fshih) → ui-review+
Comment on attachment 8356463 [details] [review]
patch on github

Hi Arthur,

just confirmed with Fang about the UI change and it looks nice to us. 

Please take sometime help me review this part (few styles change) when you have time. 

Big thanks !! :)
Attachment #8356463 - Flags: review?(arthur.chen)
Attachment #8356464 - Flags: ui-review?(fshih) → ui-review+
Attachment #8356463 - Flags: ui-review?(fshih) → ui-review+
Comment on attachment 8356463 [details] [review]
patch on github

Please check my comments regarding the item description in github. Thanks!
Attachment #8356463 - Flags: review?(arthur.chen)
Comment on attachment 8356463 [details] [review]
patch on github

Thanks Arthur, 

I just updated the code to make it do as what we did in `call settings`.
Attachment #8356463 - Flags: review?(arthur.chen)
Comment on attachment 8356463 [details] [review]
patch on github

r=me. Thank you EJ!
Attachment #8356463 - Flags: review?(arthur.chen) → review+
Thanks all, 

landed on Gaia/master : e1688b6be665f2f25fb898c1809bf9f1efd10ce5
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: