-
Notifications
You must be signed in to change notification settings - Fork 166
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.
I left 2 questions to double-check, otherwise LGTM
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.
I am not sure why it looks like there is an example missing on the top line of this png?
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.
There is a link missing. However, this is the same behavior seen in the existing tests. We might need to reevaluate the unit test and how the snapshot is captured. I can log a JIRA, if desired.
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.
Just double-checking to ensure that overlay looks as expected and should not have any text?
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 question. This path shows that text should not be present: raw/tests/cerner-terra-core-docs/overlay/overlay/default-overlay
Summary
What was changed:
The Terra Badge, Base, Card, Divider, Hyperlink, Image, Overlay, and Signature components were updated to support the Redwood theme
Why it was changed:
The change was made to start the rebranding of our solutions to match Oracle standards.
Testing
This change was tested using:
Reviews
In addition to engineering reviews, this PR needs:
Additional Details
This PR resolves:
UXPLATFORM-10149
UXPLATFORM-10031
UXPLATFORM-10011
UXPLATFORM-10009
UXPLATFORM-10003
UXPLATFORM-9987
UXPLATFORM-9950
Thank you for contributing to Terra.
@cerner/terra