Closed
Bug 949982
Opened 11 years ago
Closed 11 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•11 years ago
|
||
Hi Fang,
can you help me review the UI for SimManager ?
Thanks :D
Attachment #8356463 -
Flags: ui-review?(fshih)
| Assignee | ||
Comment 2•11 years ago
|
||
Attachment #8356464 -
Flags: ui-review?(fshih)
| Assignee | ||
Comment 3•11 years ago
|
||
Attachment #8356465 -
Flags: ui-review?(fshih)
Comment 4•11 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•11 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•11 years ago
|
Attachment #8356464 -
Flags: ui-review?(fshih) → ui-review+
Updated•11 years ago
|
Attachment #8356463 -
Flags: ui-review?(fshih) → ui-review+
| Reporter | ||
Comment 6•11 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•11 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•11 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•11 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
•