-
Notifications
You must be signed in to change notification settings - Fork 106
chore(backstop): update baseline #8076
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
chore(backstop): update baseline #8076
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (122)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Preview: https://pf-pr-8076.surge.sh A11y report: https://pf-pr-8076-a11y.surge.sh |
mcoker
left a comment
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.
LGTM. I'm assuming the border--width--box--clicked change is on purpose - now default/hover/clicked are all unique widths.
- --pf-t--global--border--width--box--clicked: var(--pf-t--global--border--width--200);
+ --pf-t--global--border--width--box--clicked: var(--pf-t--global--border--width--300);
--pf-t--global--border--width--box--default: var(--pf-t--global--border--width--100);
- --pf-t--global--border--width--box--hover: var(--pf-t--global--border--width--100);
+ --pf-t--global--border--width--box--hover: var(--pf-t--global--border--width--200);
Diffs shown in report
I see a lot of on-disabled, a border change in cards, and backdrop color.
See changes in https://github.com/patternfly/patternfly/pull/8062/changes#diff-bdf0e7fecf8c9f7b350a2936aa70c255f25271ddd9d1df9b552f2ebfb303152f