-
Notifications
You must be signed in to change notification settings - Fork 72
[terra-flowsheet-data-grid] Added column highlighting for flowsheet grids #1985
Conversation
49bb09b
to
29b91be
Compare
The new props should only be visible on doc site for flowsheet-data-grid and everywhere else it should be private right? |
Yes. Changed in f316f91 |
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.
Commented on the API changes as well as a few other things.
Additionally, there's a few a11y requirements that need to be met:
-
Column header cell also needs its visually hidden text to include indication that the column is highlighted. Consumer should be able to provide a string label to append a meaningful indication like "most recent time" or "selected time". I believe that's what these are intended to represent. Clarification should be sought on this so the examples can be show real-world applications.
-
Column header needs a visual indication other than color to show the column's highlight and differentiate the colors from each other. Stella timeline uses a circle and a square icon. I know that's not ideal so design should be consulted on this.
...cs/src/terra-dev-site/doc/data-grid/FlowsheetDataGrid.4/Examples.6/ColumnHighlight.6.doc.mdx
Outdated
Show resolved
Hide resolved
...c/terra-dev-site/doc/data-grid/FlowsheetDataGrid.4/Examples.6/ColumnHighlightGreen.7.doc.mdx
Outdated
Show resolved
Hide resolved
...cs/src/terra-dev-site/doc/data-grid/FlowsheetDataGrid.4/Examples.6/ColumnHighlight.6.doc.mdx
Outdated
Show resolved
Hide resolved
...cs/src/terra-dev-site/doc/data-grid/FlowsheetDataGrid.4/Examples.6/ColumnHighlight.6.doc.mdx
Outdated
Show resolved
Hide resolved
2234831
to
e58239a
Compare
Fixed in d16d6e9 |
e51d7d6
to
4cda8e2
Compare
Summary
What was changed:
Why it was changed:
Testing
This change was tested using:
Reviews
In addition to engineering reviews, this PR needs:
Additional Details
This PR resolves:
UXPLATFORM-9542
Thank you for contributing to Terra.
@cerner/terra