Skip to content

Migrate @kyper/icon/Accounts to MXUI - #368

Merged
wesrisenmay-mx merged 6 commits into
masterfrom
cmwimo/CT-2394
Aug 21, 2026
Merged

Migrate @kyper/icon/Accounts to MXUI#368
wesrisenmay-mx merged 6 commits into
masterfrom
cmwimo/CT-2394

Conversation

@mwclemy

@mwclemy mwclemy commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Before(Kyper)

Screenshot 2026-08-17 at 19 06 49 Screenshot 2026-08-17 at 19 09 12

After(MXUI)

Screenshot 2026-08-17 at 19 28 46 Screenshot 2026-08-21 at 11 19 25

@mwclemy

mwclemy commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator Author

Unit tests pass locally but for some reason, there’s a failure in the pipeline and I’m not completely sure of the root cause.

@wesrisenmay-mx

Copy link
Copy Markdown
Collaborator

The new icon's placement is different in its container

image

@mwclemy

mwclemy commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator Author

The new icon's placement is different in its container

image

I was passing the size prop value as a string instead of a number. This should be fixed now.

@wesrisenmay-mx
wesrisenmay-mx merged commit cfa3c46 into master Aug 21, 2026
7 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.33.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants