-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: form elements - dropdown documentation #583
feat: form elements - dropdown documentation #583
Conversation
houbly
commented
Jan 8, 2021
- Added properties and examples for wmndsDropdown component
- added & updated params to reflect GDS
- moved component to new page
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are accessibility issues in these changes.
🚀 Deployed on https://deploy-preview-583--wmnetwork.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are accessibility issues in these changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments! :)
<select class="wmnds-fe-dropdown__select" id="{{id}}" name="{{id}}"> | ||
<option value="" selected="selected">Choose from list</option> | ||
{%- for i in range(1,6) %} | ||
<select class="wmnds-fe-dropdown__select" id="{{id}}" name="{{name}}" {%if describedBy%}aria-describedBy="{{describedBy}}"{%endif%}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dropdown must have the individual errorclass not sure if there is one for select, but on input it is wmnds-fe-input--error. It should have a red border if there's an error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There isn't a class for it in the css and I've checked the zeplin styleguide but there are no error styles for dropdown so maybe its best to leave this for a separate issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are accessibility issues in these changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 👍
# [1.2.0](v1.1.0...v1.2.0) (2021-01-20) ### Bug Fixes * Better formatting of HTML code snippets ([2442708](2442708)) * JS examples now work correctly. ([b3f7709](b3f7709)) * Nunjucks props appear better in table ([6ef138a](6ef138a)) * SCSS can now stream into the browser ([58a7f41](58a7f41)) * turned accessibility linting off as it is slowing down builds ([bdcccf7](bdcccf7)) * updated the copyright date ([#576](#576)) ([698a873](698a873)), closes [#567](#567) ### Features * Add entries for rail zone and app to A-Z content style guide ([#542](#542)) ([b1d69e6](b1d69e6)) * Adds new icons for cycle hire stations and e-bikes ([#547](#547)) ([1c68363](1c68363)), closes [#489](#489) [#489](#489) * Breadcrumbs documentation ([#551](#551)) ([bc65dbb](bc65dbb)) * buttons documentation ([#534](#534)) ([a80acbf](a80acbf)) * changes to content for deprecated components and patterns ([#541](#541)) ([3741c01](3741c01)) * content card documentation ([#553](#553)) ([13ad0fd](13ad0fd)) * details docs ([#535](#535)) ([e08cb09](e08cb09)) * display options for array type properties ([#548](#548)) ([6e14630](6e14630)) * Enhances metadata for Design System ([#546](#546)) ([20c3482](20c3482)) * file download documentation ([#539](#539)) ([0b449db](0b449db)) * form elements - checkbox documentation ([#580](#580)) ([5061996](5061996)) * form elements - date input documentation ([#588](#588)) ([19a4a1d](19a4a1d)) * form elements - dropdown documentation ([#583](#583)) ([17cf936](17cf936)) * Form elements - error message documentation ([#582](#582)) ([1ee82f0](1ee82f0)) * form elements - File upload documentation ([#587](#587)) ([61c0e24](61c0e24)) * form elements - form label documentation ([#571](#571)) ([f4380eb](f4380eb)) * form elements - progress indicator documentation ([#573](#573)) ([c01f434](c01f434)) * form elements - question documentation ([#572](#572)) ([9763203](9763203)) * form elements - radios documentation ([#584](#584)) ([03e1a31](03e1a31)) * form elements - text input documentation ([#585](#585)) ([640f68d](640f68d)) * form elements - textarea documentation ([#581](#581)) ([3122e64](3122e64)) * Improve imports of njk components ([#599](#599)) ([5ac01fe](5ac01fe)) * In-text step documentation ([#569](#569)) ([3853e1d](3853e1d)) * Inset text documentation ([#538](#538)) ([eae5a64](eae5a64)) * lists documentation ([#589](#589)) ([05bd895](05bd895)) * loader documentation ([#537](#537)) ([217c02f](217c02f)) * Markdown tags are now supported ([c4d460f](c4d460f)) * message documentation ([#554](#554)) ([adc7037](adc7037)) * phase indicator documentation ([#555](#555)) ([023d9cd](023d9cd)) * Updated homepage for the Design System ([#552](#552)) ([568eb49](568eb49)), closes [#543](#543) * Updated links docs ([#536](#536)) ([503c2ea](503c2ea)) * Warning text docs ([#545](#545)) ([444c341](444c341))
🎉 This PR is included in version 1.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |