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

Turn off autocapitalize and autocorrect attributes in text boxes for iOS (BugID: 3508912) #40

Merged
merged 1 commit into from
Apr 19, 2012

Conversation

honzap
Copy link
Contributor

@honzap honzap commented Apr 18, 2012

I've added code for BugID 3508912 (http://sourceforge.net/tracker/?func=detail&aid=3508912&group_id=23067&atid=377408) based on suggested solution in commends there. Attributes are set only for iP* devices matched from UA identification.

I've tried it on iPad 2 and it works well. I agree, to start writing with upper-case letter (in login, for new database, table etc.) is really annoying.

@roccivic
Copy link
Contributor

What about an android or a blackberry or another mobile device?
Could they have the same problem and also need this fix?

@honzap
Copy link
Contributor Author

honzap commented Apr 19, 2012

There was a problem with filling input boxes (first letter was upper-case) and this patch corrects it. Android doesn't do it, I think (tested at browser on Android 2.3.3). I can't test a blackberry because I don't know about someone who has got it. Of course, other devices make upper-case letter after dot (in textareas) but I thing this is not a major problem. I google anything other about this problem and it seems it is only at iOS devices.

@nijel
Copy link
Contributor

nijel commented Apr 19, 2012

I've tested it with N9 browser and it does not seem to need it. Anyway I don't think this could cause any harm so I'm merging this.

nijel added a commit that referenced this pull request Apr 19, 2012
Turn off autocapitalize and autocorrect attributes in text boxes for iOS (BugID: 3508912)
@nijel nijel merged commit fb77654 into phpmyadmin:master Apr 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants