Closed
Bug 981498
Opened 11 years ago
Closed 11 years ago
Auto Prediction if cancelled gives a space
Categories
(Firefox OS Graveyard :: Gaia::Keyboard, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ankit93040, Assigned: chens)
Details
(Whiteboard: [g+][LibGLA, Dev, B] )
Attachments
(1 file)
|
46 bytes,
text/x-github-pull-request
|
rudyl
:
review+
Omega
:
ui-review-
|
Details | Review |
1.Go to New Message
2.Type the Message in the Text Box
3.Default suggestion starts cancel it by clicking on the x button
Actual result:-
Automatically the cursor moves by a step.
Expected Result:-
The cursor shouldn't move.
Updated•11 years ago
|
Component: Gaia::SMS → Gaia::Keyboard
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → shchen
| Assignee | ||
Comment 1•11 years ago
|
||
Remove additional space if a user cancels autocomplete/suggestion by IME.
Add a extra space have already been done when user select one of the suggestions in the candidate list [1], this is necessary because the word is completed after choosing one of the candidates, thus we have to provide a space for user to continue typing.
But in this case, when user trying to cancel the suggestion, I suspect we don't need to provide extra space because he might want to ignore the candidate and continue the incomplete word. So this patch is trying to remove adding extra space when user dismisses the candidates.
[1]: https://github.com/shamenchens/gaia/blob/Bug981498-RemoveKeyboardAdditionalSpace/apps/keyboard/js/imes/latin/latin.js#L662-L663
Attachment #8400485 -
Flags: review?(rlu)
Comment 2•11 years ago
|
||
Extra info: Android + iOS both do not insert a space there. But need feedback from UX first I'd guess.
Flags: needinfo?(fdjabri)
Comment 3•11 years ago
|
||
Comment on attachment 8400485 [details] [review]
Pull request
Chens,
This looks good to me, but as Jan mentioned, this will need UX input.
Omega,
Could you please confirm the modified behavior is ok?
Thank you.
Attachment #8400485 -
Flags: ui-review?(ofeng)
Attachment #8400485 -
Flags: review?(rlu)
Attachment #8400485 -
Flags: review+
Comment 4•11 years ago
|
||
Just to mention that there is another corner case I saw:
* write some text
* tap at the end of a word to correct it
* press backspace, type something else, choose another word
=> there is now 2 spaces instead of 1
I don't know if this patch corrects this behavior as well?
Comment 5•11 years ago
|
||
Comment on attachment 8400485 [details] [review]
Pull request
As offline discussion with Sherman, when a user enters some characters, say, "do", he may want a "do" or a "dog". In the original design,
1) if he wants a "do", he can press Space to skip suggestion and append a space, or he can tap X in suggestion bar to skip suggestion and append a space;
2) if he wants a "dog", he tap "dog" in suggestion bar to use "dog" and append a space;
3) if he wants to modify "do", he presses Back Space rather than tapping X.
So, we should keep 1) & 2) consistent that means we should not change the original design.
Attachment #8400485 -
Flags: ui-review?(ofeng) → ui-review-
Comment 6•11 years ago
|
||
In the current design, "1)" will append "dog" if "dog" was highlighted.
Updated•11 years ago
|
Flags: needinfo?(fdjabri)
Comment 7•11 years ago
|
||
FYI, Android doesn't show a space, but when you type a new char it will insert one after dismissing a suggestion.
| Assignee | ||
Updated•11 years ago
|
Status: UNCONFIRMED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Comment 8•11 years ago
|
||
What's the rationale for WONTFIX this?
| Reporter | ||
Updated•11 years ago
|
Whiteboard: [g+]
| Reporter | ||
Updated•11 years ago
|
Whiteboard: [g+] → [g+][LibGLA, Dev, B]
You need to log in
before you can comment on or make changes to this bug.
Description
•