Skip to content

Conversation

@sethidden
Copy link
Contributor

Description

Centered the <SvgImage> depicting the "sad green box" in the WishlistSidebar. Basically just aligned with what's in CartSidebar (this is not based on some Figma designs on anything)

This aligns only the desktop version. The wishlist on mobile has double "My wishlist" text (in sidebar header and in the sidebar itself), so the vertical position of the "sad green box" image is off by a few px between the mobile sidebar and mobile wishlist. I deemed it a separate issue so I didn't change that.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

this is not based on any Figma designs - just aligning `WishlistSidebar.vue` with `CartSidebar.vue`

This is a fix ONLY for the desktop version. There's still a few px of
vertical difference between the `<SvgImages>` on mobile CartSidebar and
mobile WishlistSidebar, but that's because one of them has a text header
and the other doesn't (separate issue)
@sethidden sethidden changed the title WishlistSidebar "sad green box" image is not centered vertically (when wishlist empty) fix(theme): WishlistSidebar "sad green box" image is not centered vertically (when wishlist empty) Mar 2, 2022
@sethidden sethidden changed the title fix(theme): WishlistSidebar "sad green box" image is not centered vertically (when wishlist empty) fix(theme): wishlist sidebar "sad green box" image is not centered vertically (when wishlist empty) Mar 2, 2022
@sethidden sethidden requested a review from bloodf March 2, 2022 08:07
@Frodigo Frodigo merged commit ac63f62 into develop Mar 2, 2022
@Frodigo Frodigo deleted the bugfix/M2-211-wishlist-your-bag-is-empty-is-not-centered branch March 2, 2022 09:46
@Frodigo Frodigo added this to the 1.0.0-rc.7 milestone Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants