-
Notifications
You must be signed in to change notification settings - Fork 55
docs(Accessibility): improve acc descriptions for components #1371
Conversation
Changed dependencies in
Generated by 🚫 dangerJS |
Codecov Report
@@ Coverage Diff @@
## master #1371 +/- ##
=======================================
Coverage 73.47% 73.47%
=======================================
Files 778 778
Lines 5859 5859
Branches 1706 1706
=======================================
Hits 4305 4305
Misses 1548 1548
Partials 6 6
Continue to review full report at Codecov.
|
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.
pls check the new dependency with the people I mentioned
@@ -112,6 +112,7 @@ | |||
"react-dom": "^16.8.6", | |||
"react-element-to-jsx-string": "^14.0.2", | |||
"react-hot-loader": "^4.8.2", | |||
"react-markdown": "^4.0.8", |
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.
@layershifter @kuzhelov @miroslavstastny
- are we ok with adding this dependency?
- can we suggest some existing tooling to achieve this?
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.
@layershifter added this
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.
can we suggest some existing tooling to achieve this?
Nope
are we ok with adding this dependency?
Yep, it's only for docs
Improve acc descriptions for components