Skip to content
This repository has been archived by the owner on Jul 30, 2019. It is now read-only.

Remove inputmode. Resolves #1015. #1030

Merged
merged 2 commits into from
Oct 10, 2017
Merged

Remove inputmode. Resolves #1015. #1030

merged 2 commits into from
Oct 10, 2017

Conversation

cynthia
Copy link
Member

@cynthia cynthia commented Sep 25, 2017

No description provided.

@@ -10547,7 +10282,7 @@ out of 233 257 824 bytes available</meter></p>
<form method=post action="step2.cgi">
<input type=hidden autocomplete=transaction-currency value="CHF">
<input type=hidden autocomplete=transaction-amount value="15.00">
<p><label>Credit card number: <input type=text inputmode=numeric autocomplete=cc-number></label>
<p><label>Credit card number: <input type=text autocomplete=cc-number></label>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about using pattern here instead? Because that actually works... right?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But it complicates the example, so let's skip it too.

@@ -395,50 +395,11 @@
</form>
</pre>

<h5 id="improving-the-user-experience-on-mobile-devices">Improving the user experience on mobile devices</h5>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of just removing this section, it would be helpful to make it consistent with the use of pattern.

There is nothing that prevents a user agent from customising the input UI to match the expected pattern - e.g. if there are a restricted set of acceptable input characters such as numerals and the plus sign and space, it would be good to offer a keyboard that did just that. In any event, using validity the application can directly help users... IFF designed well. But what isn't well-designed on the web?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change request seems editorial, and we can live without it.

@chaals chaals removed the request for review from travisleithead October 2, 2017 10:46
@chaals chaals merged commit d62035f into w3c:master Oct 10, 2017
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