Closed
Bug 856468
Opened 13 years ago
Closed 10 years ago
Missing space in last chance of entering PIN
Categories
(Firefox OS Graveyard :: Gaia::First Time Experience, defect)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: piecu, Assigned: aniruddhc)
Details
(Whiteboard: ux-most-wanted)
Attachments
(4 files)
|
27.78 KB,
image/png
|
Details | |
|
46 bytes,
text/x-github-pull-request
|
Omega
:
ui-review+
|
Details | Review |
|
15 bytes,
patch
|
Details | Diff | Splinter Review | |
|
33.82 KB,
image/png
|
Details |
If you enter the PIN code two times wrong you get an information about your last chance. There should be a space put after the sentence: "The PIN was incorrect" and before the following sentence.
See the attachment for the details.
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → aniruddhc
| Assignee | ||
Comment 1•12 years ago
|
||
I cannot replicate this bug in the Nightly build, because the Nightly build does not come with support to mock a locked sim card.
However, I spoke to qDot in the #gaia IRC and he told me that I should be fine with adding the extra space and then sending out a pull request.
[Approval Request Comment] https://github.com/mozilla-b2g/gaia/pull/16100
[Bug caused by] (feature/regressing bug #):
[User impact] if declined:
[Testing completed]:
[Risk to taking this patch] (and alternatives if risky):
[String changes made]: Added an extra space
Attachment #8372722 -
Flags: approval-gaia-v1.3?
| Assignee | ||
Updated•12 years ago
|
Attachment #8372722 -
Flags: approval-gaia-v1.3? → ui-review?(firefoxos-ux-bugzilla)
Updated•12 years ago
|
Whiteboard: ux-most-wanted
Comment 2•12 years ago
|
||
Comment on attachment 8372722 [details] [review]
Github Pull Request for this issue
Flagging Omega for a quick patch review here, which it sounds like :qdot may already have blessed.
Attachment #8372722 -
Flags: ui-review?(firefoxos-ux-bugzilla) → ui-review?(ofeng)
Comment 3•12 years ago
|
||
Comment on attachment 8372722 [details] [review]
Github Pull Request for this issue
This patch is reviewed+
Attachment #8372722 -
Flags: ui-review?(ofeng) → ui-review+
Comment 4•11 years ago
|
||
Comment 5•10 years ago
|
||
Last change / pin error screen - looks good to me.
Updated•10 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•