-
Notifications
You must be signed in to change notification settings - Fork 219
Make the customer account icon bigger to match the mini-cart one #8594
Conversation
Hi @vivialice, we have increased the size of the |
The release ZIP for this PR is accessible via:
Script Dependencies ReportThere is no changed script dependency between this branch and trunk. This comment was automatically generated by the TypeScript Errors Report
🎉 🎉 This PR does not introduce new TS errors. |
Size Change: +133 B (0%) Total Size: 1.1 MB
ℹ️ View Unchanged
|
6982247
to
9d4e57d
Compare
@Aljullu could you have a look at this approach to increase the icon size on new blocks only? |
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.
That's a really clever approach, I like it! I did some testing and it worked flawlessly on my end: there weren't changes in the block if it was already in the template, but when adding a new one, the sizes matched the Mini Cart block. 👏
I left one small suggestion below about the class name. Besides that, I would also add some explanation somewhere in the code (maybe when you register the variation) to explain the approach.
Even though the PR is a draft, I'm pre-approving, but happy to take another look if you want. 🙂
The goal of this PR is to increase the size of the
Customer account
icon to make it more similar to theMini cart
one.Fixes #8128
Testing
Check the icon size
Customer account
andMini-cart
blocks next to the menu, to look like this 👇Customer account
icon is bigger than before and matches theMini cart
icon size.Check the icon size of existing blocks does not change
trunk
.Customer account
block, and save.update/8128-account-icon-size
).Customer account
block and save.WooCommerce Visibility
Changelog