-
Notifications
You must be signed in to change notification settings - Fork 106
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
Review Date #65
Projects
Comments
chrimesdev
added a commit
that referenced
this issue
Oct 11, 2018
Refactored the review date component to meet the component criteria, also added examples, documentation and tests for the component.
chrimesdev
added a commit
that referenced
this issue
Oct 11, 2018
Refactored the review date component to meet the component criteria, also added examples, documentation and tests for the component.
chrimesdev
added a commit
that referenced
this issue
Oct 18, 2018
Wrap the last and next review dates in the HTML5 <time> element for semantics.
chrimesdev
added a commit
that referenced
this issue
Oct 19, 2018
The review date should be formatted as 1 January 2016, rather than 01/01/2016. This has come from the standards team and there are benefits to screen readers as they struggle to read out the latter format.
chrimesdev
added a commit
that referenced
this issue
Oct 19, 2018
The review date should be formatted as 1 January 2016, rather than 01/01/2016. This has come from the standards team and there are benefits to screen readers as they struggle to read out the latter format.
davidhunter08
added a commit
that referenced
this issue
Oct 22, 2018
Changed review date component spacing to @include nhsuk-responsive-margin(7, 'top');. This will make the top margin 48px on desktop and 40px on mobile.
chrimesdev
added a commit
that referenced
this issue
Oct 23, 2018
Use double quotes on nunjucks macros rather than single quotes. Also delete the CHANGELOG because its not being updated and should only be used after the first frontend library release.
chrimesdev
added a commit
that referenced
this issue
Oct 23, 2018
Use double quotes on nunjucks macros rather than single quotes. Also delete the CHANGELOG because its not being updated and should only be used after the first frontend library release.
chrimesdev
added a commit
that referenced
this issue
Oct 23, 2018
Use double quotes on nunjucks macros rather than single quotes. Also delete the CHANGELOG because its not being updated and should only be used after the first frontend library release.
chrimesdev
added a commit
that referenced
this issue
Oct 23, 2018
Remove the time element as there is no additional benefit and we would need to output the date in 2 different formats every time. Also remove unnecessary css comments.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Use this issue to refactor the component: Review Date
Refactor checklist
Component checklist
The text was updated successfully, but these errors were encountered: