Closed
Bug 928499
Opened 11 years ago
Closed 11 years ago
No background image on SIM card Mandatory screen
Categories
(Firefox OS Graveyard :: Gaia::First Time Experience, defect)
Tracking
(b2g-v1.2 fixed, b2g-v1.3 fixed)
RESOLVED
FIXED
People
(Reporter: mai, Assigned: mai)
Details
(Whiteboard: [systemsfe])
Attachments
(1 file)
256 bytes,
text/html
|
fcampo
:
review+
praghunath
:
approval-gaia-v1.2+
|
Details |
There is no default background image on the SIM Mandatory screen message. Steps to reproduce: 1. Prepare a build customized to have SIM card mandatory during FTE 2. Flash a device with such build 3. Power on the device without a SIM card inserted 4. Select a language in FTE and press 'Next' button.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → mri
Whiteboard: [systemsfe]
Assignee | ||
Comment 1•11 years ago
|
||
Please, could you test my code? Regards
Attachment #819183 -
Flags: review?(fernando.campo)
Comment 2•11 years ago
|
||
Comment on attachment 819183 [details]
patch v1.0
Looks nice, thanks!
Attachment #819183 -
Flags: review?(fer.campo.garcia+bugzilla) → review+
Comment 3•11 years ago
|
||
merged on master 1f91430b90a7fc67876860be50d8dc016c12941a
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 4•11 years ago
|
||
Comment on attachment 819183 [details] patch v1.0 NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings. [Approval Request Comment] Solution for a poor user experience [Bug caused by] (feature/regressing bug #):Feature [User impact] if declined: the font size is too small to comfortably read[Testing completed]:Yes [Risk to taking this patch] (and alternatives if risky): Only css changes, non risky [String changes made]:
Attachment #819183 -
Flags: approval-gaia-v1.2?
Comment 5•11 years ago
|
||
Marina Since it is a new feature, why can't it wait for 1.3?
Flags: needinfo?(mri)
Comment 6•11 years ago
|
||
Comment on attachment 819183 [details]
patch v1.0
Bad UX hence 1.2+
Attachment #819183 -
Flags: approval-gaia-v1.2? → approval-gaia-v1.2+
Comment 7•11 years ago
|
||
Uplifted 1f91430b90a7fc67876860be50d8dc016c12941a to: v1.2: e1ac088598c5d5115010fe82e3f6f675cea32cfd
status-b2g-v1.2:
--- → fixed
Assignee | ||
Comment 8•11 years ago
|
||
Because when i implemented this screen i didn't have UX feedback about it and moreover it was the first screen that i implemented. :( I think this screen gives a very poor image to the future phone user and if the user sight isn't very good it might be difficult for him reading the message. Regards
Assignee | ||
Updated•11 years ago
|
Flags: needinfo?(mri)
Comment 9•10 years ago
|
||
Uplifted 1f91430b90a7fc67876860be50d8dc016c12941a to: v1.3 already had this commit
status-b2g-v1.3:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•