Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix autocomplete problem #1

Closed
wants to merge 1 commit into from
Closed

Fix autocomplete problem #1

wants to merge 1 commit into from

Conversation

nomadturk
Copy link
Owner

Updated the form to not auto complete what we wrote before by removing action="" and adding autocomplete="off"

This patch denies erming#518 and removes that. Because without autocomplete, writing suggestions for mobile keyboards was enabled but at the mean time also whenever you clicked on text area to write a new message, you were given a number of your last typed sentences. Especially on mobile, this was a huge buzzkill because after 3-5 sentences on any channel, these suggestions covered your whole screen.

So, in order to remove it I did some changes, ensuring writing suggestions still work.
I have added autocomplete=off .

This commit is related to erming#532
@JocelynDelalande you might give it a go. ;) I don't have google keyboard.

Updated the form to not auto complete what we wrote before by removing action="" and adding autocomplete="off"
Making so now, especially when I'm on mobile I don't get my screen filled with previously typed sentences.

This commit is related to erming#532
@JocelynDelalande
Copy link

@nomadturk is there a reason that PR is not on erming's source repo ?

@nomadturk
Copy link
Owner Author

Probably because I messed it up :D
Let me fix it.

Repository owner locked and limited conversation to collaborators Jan 8, 2016
@nomadturk nomadturk closed this Jan 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants