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)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: ankit93040, Assigned: chens)

Details

(Whiteboard: [g+][LibGLA, Dev, B] )

Attachments

(1 file)

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.
Component: Gaia::SMS → Gaia::Keyboard
Assignee: nobody → shchen
Attached file Pull request
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)
Extra info: Android + iOS both do not insert a space there. But need feedback from UX first I'd guess.
Flags: needinfo?(fdjabri)
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+
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 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-
In the current design, "1)" will append "dog" if "dog" was highlighted.
Flags: needinfo?(fdjabri)
FYI, Android doesn't show a space, but when you type a new char it will insert one after dismissing a suggestion.
Status: UNCONFIRMED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
What's the rationale for WONTFIX this?
Whiteboard: [g+]
Whiteboard: [g+] → [g+][LibGLA, Dev, B]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: