fix(ExternalLinkAltIcon): Replace ExternalLinkAltIcon with RhUiExternalLinkIcon#12490
Conversation
|
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 (11)
WalkthroughAll references to ChangesExternalLinkAltIcon → RhMicronsExternalLinkIcon icon replacement
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~4 minutes Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 |
7e46849 to
e6ce8cc
Compare
|
Preview: https://pf-react-pr-12490.surge.sh A11y report: https://pf-react-pr-12490-a11y.surge.sh |
…alLinkIcon Co-authored-by: Cursor <cursoragent@cursor.com>
e6ce8cc to
d3beb56
Compare
…UiExternalLinkIcon Co-authored-by: Cursor <cursoragent@cursor.com>
Part of #12401. Breaking into separate PRs so it is easier to review.
Tried in first commit with RhUiExternalLinkIcon but it was a little hard to see compared to ours. Tried Micron version in second commit.
Made with Cursor
Summary by CodeRabbit