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

Add variant option color on product page in demo store #1751

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sergejasadchij
Copy link

Improve UI/UX of product colour option selection on product page in demo-store

WHAT is this pull request doing?

This implementation allows the user to see the colour of the product variants and select the product of the desired colour

изображение

  • Clicking on a coloured square will take you to the page of the selected product variant.
  • The selected colour variant is highlighted with a frame
  • Unavailable colour variant is crossed out with a diagonal line.

it was also added colours to the options listbox (it shows up when there are more than 7 colours)

изображение

HOW to test your changes?

  • Go to a page of product that has several colour variants
  • Here you will see squares with the colours of the product variants.
  • Click on a coloured square to take you to the page of the selected product variant.
  • Chek selected and unavailabled pvoduct variants
  • Go to a page of product that has more then 7 colour variants
  • Here you will see variant colors in listbox
  • Check with browser dark theme

Checklist

  • I've read the Contributing Guidelines
  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've added a changeset if this PR contains user-facing or noteworthy changes
  • I've added tests to cover my changes
  • I've added or updated the documentation

@michenly
Copy link
Contributor

cc @mynameisadamf for UX review

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

Successfully merging this pull request may close these issues.

None yet

3 participants