-
-
Notifications
You must be signed in to change notification settings - Fork 931
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
docs(ui): Improved accessibility #3072
Conversation
Awesome that you're working on this! Please add some screenshots with how it looks before and after the change. |
sure thing! |
@alt-shreya did you want any input here before continuing? |
@spydon no, my apologies for the delay, I had to prioritise a project for another client. |
@spydon Adding some screenshots as requested: BeforeAfter |
@spydon there seems to be a conflict with markdown-lint tests. I have to include long URLs that are greater than 100 characters, but I don't seem to be able to wrap them like normal text. Should I just ignore the failures of the MD013 tests? |
… into docs-accessibility
You can just move them to the next line in the markdown file, they will still render as continues text as long as you don't do two line breaks. |
I've tried this, but it raises another error: |
All tests seem to be passing now 🤔 |
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.
Lgtm! Thanks for your contribution
Lessgoo! Looking forward to contributing more in the future! 🎉 |
Description
Checklist
docs
and added dartdoc comments with///
.examples
ordocs
.Breaking Change?
Related Issues
Closes #2895