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

Placeholder + issue 7 #8

Closed
wants to merge 1 commit into from
Closed

Placeholder + issue 7 #8

wants to merge 1 commit into from

Conversation

pulsar895
Copy link

Bonjour,

J'ai ajouté un test sur la version d'IE, mais je pense ça un peu dommage, surtout que quelqu'un qui n'a pas IE10 n'utilise sans doute pas IE. Pour rappel XP n'est plus supporté et les postes en vista sont rare, enfin, au moins, ceux qui ne sont pas en 10 verront le label.

J'ai également intégré l'issue de Rubén en mettant "required" partout.

Par contre, comme il n'y a pas de label avec les navigateurs modernes, le css est peut être à retravailler.
plxmycontact

@Pluxopolis
Copy link
Owner

Bonjour

Peux-tu me rappeler le problème avec placeholder sous IE stp ?

Pour info: placeholder est une instruction html5 supportée par IE10+
Je ne souhaite pas supporter les anciens navigateurs < à IE10.

http://www.alsacreations.com/tuto/lire/1370-formulaire-html5-placeholder-required-pattern.html

@pulsar895
Copy link
Author

Moi je n'ai pas de problème avec ça : #6
Si ça ne te pose aucun soucis d'être compatible uniquement "HTML5", je modifierais le code en conséquence (virer ma fonction).

Dis moi :)

@pulsar895 pulsar895 closed this Jan 28, 2015
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.

2 participants