-
Notifications
You must be signed in to change notification settings - Fork 139
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
add comparison table to table guidelines #4006
Conversation
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.
one small comment, but otherwise looks good!
@@ -118,6 +118,32 @@ The actionable table provides checkboxes or radio buttons that enable users to s | |||
#### When not to use | |||
* Users can not take any actions on table items/rows. | |||
|
|||
### Comparison table |
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.
should we also add this as a list item on line 77?
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.
great catch!! Just updated
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.
🤘❗️
* add comparison table to table guidelines * Update table.md * Update table.md
* fix: bump to latest release candidates ahead of 5.2 release (#3862) * fix: bump to latest release candidates ahead of 5.2 release * bump to latest component groups prerelease * bump react to pull in bug fix * feat: release 5.3 (#3878) * feat: 5.3 release * update release highlights * fix: Bump topology to 5.2.1 (#3881) * fix: toplogy to 5.2.1 * add yarn.lock * update release highlights * Update broken link on develop.md * chore(release): Updated component groups to 5.1.0 release (#3917) * chore(release): releasing packages [ci skip] - @patternfly/[email protected] - @patternfly/[email protected] chore(component-groups): Updated component groups to 5.1.0 release chore(component-groups): Updated component groups to 5.1.0 release * Updated version.json * feat(Templates): add to org site (#3947) * feat(Templates): add to org site * update versions, fix Th * hide prev version, update screenshots * fix(docs): proper commit (#3968) * feat(docs): update screenshots (#3969) * Update link for deprecated alert (#3956) * feat(docs): fix for missing props tables (#3981) * chore(a11y): updated a11y scorecard (#3980) * chore(a11y): updated a11y scorecard * PR feedback - adjusted 'initial report' section with current data * fix(misc): fix broken about links (#3987) * add comparison table to table guidelines (#4006) * add comparison table to table guidelines * Update table.md * Update table.md * docs(tooltips): Creates separate tooltips writing guide. (#3999) * docs(tooltips): Creates separate tooltips writing guide. * Edits content. * Removes unused image. * Content edits to prep for reviews. * Adds some aria attributes. * Cross link from icons guide. * fix: Incorrectly formatted link (#4024) There was a link that was incorrectly formatted and let to a 404 page * docs: Updates PatternFly mailing list link. (#4030) * fix: update to official alpha omegas * bump docs-framework version --------- Co-authored-by: Donald Labaj <[email protected]> Co-authored-by: kmcfaul <[email protected]> Co-authored-by: Jessie <[email protected]> Co-authored-by: Evan <[email protected]> Co-authored-by: Margot <[email protected]> Co-authored-by: Erin Donehoo <[email protected]> Co-authored-by: Freya Gustavsson <[email protected]>
added these guidelines https://docs.google.com/document/d/1Z6LyyqYTxKxCtYwRVywttHjL7v15AFt7FyAXJB8zNK0/edit to table guidelines
closes #3963