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

Fix focus outline cut off in product tile #729

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

Conversation

spencercanner
Copy link
Contributor

@spencercanner spencercanner commented Sep 16, 2020

  • Previously, the focus would be cut off for any elements that were near the edge of the iframe. This includes select elements, buttons, and isButton products
  • To address this, padding was added to the product container in the iframe giving its contents some space from the iframe boundary

To 🎩 :

  • Create a product buy button with product > isButton: true
  • Verify that the focus outline is shown around the entire product tile
  • Create a product buy button for a product with options
  • Verify that the focus outline is shown around the entire variant select element and the button

Browsers:

  • Safari - Mac
  • Firefox - Mac
  • Chrome - Mac
  • Edge - Mac
  • Legacy Edge - Windows
  • Edge - Windows
  • Chrome - Windows
  • Firefox - Windows

@spencercanner spencercanner changed the title Fix focus outline cutoff in product tile Fix focus outline cut off in product tile Sep 16, 2020
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.

1 participant