Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing Pinterest Save button positioning for Twenty Twenty-Four theme. #1055

Merged
merged 3 commits into from
Aug 24, 2024

Conversation

message-dimke
Copy link
Contributor

Changes proposed in this Pull Request:

Closes #1054.

Pinterest Save button has CSS absolute positioning when the product card has no CSS relative position rule on it which makes Pinterest Save button align relatively to the document, instead of a product card element.

Screenshots:

Cursor_and_Shop_–_Wildly_Involved Music_–_Wildly_Involved

Detailed test instructions:

  1. Install Pinterest extension with Twenty Twenty-Four.
  2. Navigate to the Shop.
  3. Hover any product card.
  4. Observe Pinterest Save button is out of bounds.

Changelog entry

Fix - Pinterest Save button positioning.

…n with its absolute positioning would stay above the product card.
@message-dimke message-dimke self-assigned this Aug 23, 2024
@github-actions github-actions bot added changelog: fix Took care of something that wasn't working. type: bug The issue is a confirmed bug. labels Aug 23, 2024
@message-dimke message-dimke marked this pull request as ready for review August 23, 2024 08:38
Copy link
Member

@ecgan ecgan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@message-dimke , leaving one finding below.

@ecgan
Copy link
Member

ecgan commented Aug 23, 2024

@message-dimke , I was testing around and I noticed that the Pinterest "Save" button does not show up in the product page with Twenty Twenty-Four theme.

Screenshot of product image in product page:

image

Screenshot of related products in product page - note that for this one, it is using wp-block-post CSS class, not wc-block-product, and it does not have the pinterest-for-woocommerce-image-wrapper div:

image

Maybe you could consider opening separate issues / PRs to fix them.

message-dimke and others added 2 commits August 24, 2024 12:17
Fixing class name typo and adding wp-block-post back to support related products section.

Co-authored-by: Gan Eng Chin <[email protected]>
@message-dimke message-dimke added the v5 Pinterest v5 related issue. label Aug 24, 2024
@message-dimke message-dimke merged commit f0c1e44 into develop Aug 24, 2024
2 checks passed
@message-dimke message-dimke deleted the fix/2024-theme-save-button-positioning branch August 24, 2024 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: fix Took care of something that wasn't working. type: bug The issue is a confirmed bug. v5 Pinterest v5 related issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Twenty-Twenty Four theme Pinterest save button is out of product card.
2 participants