fix(ThIcon): Replace ThIcon with RhUiGripHorizontalFillIcon#12477
fix(ThIcon): Replace ThIcon with RhUiGripHorizontalFillIcon#12477rebeccaalpert wants to merge 1 commit into
Conversation
|
Preview: https://pf-react-pr-12477.surge.sh A11y report: https://pf-react-pr-12477-a11y.surge.sh |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (9)
✅ Files skipped from review due to trivial changes (3)
🚧 Files skipped from review as they are similar to previous changes (6)
WalkthroughAcross four demo TSX components and five companion Markdown documentation files, the ChangesIcon Substitution in Demo Components and Docs
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related issues
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Co-authored-by: Cursor <cursoragent@cursor.com>
c2d1f09 to
cdab88e
Compare
Part of #12401. Breaking into separate PRs so it is easier to review.
This is the closest icon I could find, but it's a little off from the prior pattern (two rows of icons vs. three). We may want to wait on this and get a new icon made.
Made with Cursor
Summary by CodeRabbit
Summary by CodeRabbit
RhUiGripHorizontalFillIconfor improved visual consistency in expanded and collapsed/tooltip states.