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

Use simpler default example for text inputs #511

Merged
merged 1 commit into from
Aug 22, 2018

Conversation

36degrees
Copy link
Contributor

Change the default example for the text input component from an input asking for an email address to an example asking for someone’s full name.

As raised in the backlog, we are not currently using the correct type=“email”. We could fix this, but as users are likely to take this ‘default’ example and modify it to ask different questions, it makes sense to make it as generic as possible, avoiding using a type specific to a particular type of input.

Therefore changing to a more ‘generic’ question, such as full name, which does not require specific input types, makes more sense.

Change the default example for the text input component from an input asking for an email address to an example asking for someone’s full name.

As raised in the backlog [1], we are not currently using the correct `type=“email”`. We could fix this, but as users are likely to take this ‘default’ example and modify it to ask different questions, it makes sense to make it as generic as possible, avoiding using a type specific to a particular type of input.

Therefore changing to a more ‘generic’ question, such as full name, which does not require specific input types, makes more sense.

[1]: alphagov/govuk-design-system-backlog#51 (comment)
@govuk-design-system-ci
Copy link
Collaborator

You can preview this change here:

Built with commit 7c74fdc

https://deploy-preview-511--govuk-design-system-preview.netlify.com

Copy link
Contributor

@joelanman joelanman left a comment

Choose a reason for hiding this comment

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

Looks good to me

@36degrees 36degrees merged commit 9dbee7f into master Aug 22, 2018
@36degrees 36degrees deleted the use-simpler-example-text-input branch August 22, 2018 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants