Closed
Bug 949982
Opened 11 years ago
Closed 10 years ago
[Gaia][DSDS] Fine tune SIM manager UI
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(firefox29 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox29 | --- | fixed |
People
(Reporter: arthurcc, Assigned: eragonj)
References
Details
Attachments
(3 files)
46 bytes,
text/x-github-pull-request
|
arthurcc
:
review+
fang
:
ui-review+
|
Details | Review |
43.26 KB,
image/png
|
fang
:
ui-review+
|
Details |
47.54 KB,
image/png
|
fang
:
ui-review+
|
Details |
SIM manager UI should follow the spec[1]. [1]: https://bugzilla.mozilla.org/attachment.cgi?id=8347148
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → ejchen
Assignee | ||
Comment 1•10 years ago
|
||
Hi Fang, can you help me review the UI for SimManager ? Thanks :D
Attachment #8356463 -
Flags: ui-review?(fshih)
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8356464 -
Flags: ui-review?(fshih)
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8356465 -
Flags: ui-review?(fshih)
Comment 4•10 years ago
|
||
Comment on attachment 8356465 [details]
screenshot2.png
Hi Ej,
Good to go! Thanks for the effort!
Attachment #8356465 -
Flags: ui-review?(fshih) → ui-review+
Assignee | ||
Comment 5•10 years ago
|
||
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)
Updated•10 years ago
|
Attachment #8356464 -
Flags: ui-review?(fshih) → ui-review+
Updated•10 years ago
|
Attachment #8356463 -
Flags: ui-review?(fshih) → ui-review+
Reporter | ||
Comment 6•10 years ago
|
||
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)
Assignee | ||
Comment 7•10 years ago
|
||
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)
Reporter | ||
Comment 8•10 years ago
|
||
Comment on attachment 8356463 [details] [review] patch on github r=me. Thank you EJ!
Attachment #8356463 -
Flags: review?(arthur.chen) → review+
Assignee | ||
Comment 9•10 years ago
|
||
Thanks all, landed on Gaia/master : e1688b6be665f2f25fb898c1809bf9f1efd10ce5
You need to log in
before you can comment on or make changes to this bug.
Description
•