-
Notifications
You must be signed in to change notification settings - Fork 11
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 Legend for Parameter Color-Coding #458
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.
Minor formatting required, otherwise looks good to merge.
src/components/OSCALComponentDefinitionControlImplementation.test.js
Outdated
Show resolved
Hide resolved
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.
This looks great so far! I noticed that running npm run test
does produce some warnings. I'd love to get this dealt with before merging into develop
.
I only looked into it a bit and do not see any obvious issues. From experience, I'd look into places where <div>
is placed into a <Typography>
.
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.
Looks good to me! Thanks for your work on this @Bronstrom
In order to enhance user experience, a parameter legend is displayed highlighting the difference between parameter placeholders (orange) and parameter values (blue).
This adds a legend to all control grouping schemes:
OSCALCatalogGroup
,OSCALControlImplementation
,OSCALComponentDefinitionControlImplementation
, andOSCALProfile
groups. The componentOSCALControlParamLegend
is built for purposes of avoiding duplicate code. In addition,react-testing-library
tests are written to determine the legend displays in each of the control groupings.Resolves #195, resolves #428, resolves #427