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

DARK: Harmony > Disable logical properties postCSS plugin feature #93

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

eihabkhan1
Copy link
Collaborator

@eihabkhan1 eihabkhan1 commented Nov 21, 2024

JIRA Ticket

DARK

Prerequisites

  • Check this branch locally and run pnpm run release
  • 3 new files will be generated in this format theme name + date + .zip
  • Upload generated file locally or in seller-area test env

QA Steps

To make sure that no breaks happen after this change, the best course of action is to visually check each component that was affected by this change, these components are:

  • cart
  • featured products
  • main
  • navbar
  • product-column
  • product-listing
  • product
  • thankyou
    Let's QA each of these

Cart

  • Summary box item price and table
Screenshot 2024-11-21 at 11 38 58 AM
  • mobile > table product details
Screenshot 2024-11-21 at 11 39 17 AM

Featured Products

  • Section title
Screenshot 2024-11-21 at 11 41 16 AM

Main

  • Dropdown button
Screenshot 2024-11-21 at 11 41 37 AM

Product Column

  • Product description
Screenshot 2024-11-21 at 11 43 28 AM

Navbar

  • Close drawer button
Screenshot 2024-11-21 at 11 38 32 AM

Product

  • product images splide list and thumbnail
Screenshot 2024-11-21 at 11 37 42 AM

Note

Leave empty when you have nothing to say.

@eihabkhan1 eihabkhan1 self-assigned this Nov 21, 2024
@eihabkhan1 eihabkhan1 added in progress Currently being developed. Ready for review Requires a review from another developer. and removed in progress Currently being developed. labels Nov 21, 2024
@eihabkhan1 eihabkhan1 marked this pull request as ready for review November 21, 2024 10:43
@bj-anas bj-anas added Reviewed PR has been reviewed and approved. and removed Ready for review Requires a review from another developer. labels Nov 21, 2024
@eihabkhan1 eihabkhan1 merged commit 4537cdc into master Nov 21, 2024
@eihabkhan1 eihabkhan1 deleted the postcss-logical-disable-harmony branch November 21, 2024 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Reviewed PR has been reviewed and approved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants