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

Cursor Goes Beyond in Text Input in IE10 #19

Closed
wesleyboar-fka-iosulfur opened this issue Feb 12, 2015 · 3 comments
Closed

Cursor Goes Beyond in Text Input in IE10 #19

wesleyboar-fka-iosulfur opened this issue Feb 12, 2015 · 3 comments
Labels

Comments

@wesleyboar-fka-iosulfur
Copy link

wesleyboar-fka-iosulfur commented Feb 12, 2015

This stack overflow (no valid answer at the time of my re-posting here) explains the problem well:
http://stackoverflow.com/questions/25402513/cursor-goes-beyond-in-text-input-in-ie10

This plugin sets padding-right on the input. Then the above issue occurs on IE10 with long passwords.

@tylersticka
Copy link
Member

Hi @iosulfur,

Sorry to hear you're encountering this issue.

The goal of the innerToggle functionality is to have an inset password visibility toggle with a great touchscreen experience. An unfortunate side effect of this is that the options for changing its relationship to the <input> element are somewhat rigid. Because of this (and the relatively low usability impact and audience for this particular bug), there is a low likelihood we'll be able to address this specific issue for users of IE10 without making the experience worse for everyone else (though we welcome pull requests from folks who are smarter than us when it comes to solving these sorts of problems).

If this issue is a show-stopper for you, I have two recommendations you might consider...

Disabling the functionality for IE10+

Microsoft actually ships their own password visibility toggle in IE10 and up if you're using Windows 8. As long as you are not hiding their controls (see here), you could use the enable option to only activate the plugin for other browsers:

$('#example').hideShowPassword({
  enable: ! isIE // Just an example, insert your own browser detection logic
});

Obviously browser detection is not ideal, but since there's no way to feature detect for the password toggle, I can't see any other way to pull this technique off.

Roll your own solution

The innerToggle is neat, but you don't need it to use the plugin. You can markup and style your own toggle and controls. See the third example in the demo.

@wesleyboar-fka-iosulfur
Copy link
Author

wesleyboar-fka-iosulfur commented Feb 14, 2015

Thank you. Disabling it for IE10, and allowing their browser implementation to persist is a feasible solution. I like the plugin, and continue to use it. IE10's behavior is acceptable given the application's market, but I will offer the disabling option to the designer.

Great plugin. Thanks for the support.

@tylersticka
Copy link
Member

Thanks, @iosulfur !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants