-
Notifications
You must be signed in to change notification settings - Fork 679
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
PWA-712: Fix Chrome prev / next carousel button placement #2512
PWA-712: Fix Chrome prev / next carousel button placement #2512
Conversation
- Ensure click area is large and focus state is shown correctly
|
Performance Test Results The following fails have been reported by WebpageTest. These numbers indicates a possible performance issue with the PR which requires further manual testing to validate. https://pr-2512.pwa-venia.com : LH Performance Expected 0.85 Actual 0.57, LH Best Practices Expected 1 Actual 0.92 |
packages/venia-ui/lib/components/ProductImageCarousel/carousel.css
Outdated
Show resolved
Hide resolved
- Update teal variable
QA Approved. |
Description
They should be in the center of the image but they are instead at the top of the image:
https://i.gyazo.com/98b75a0fb607669e627f2d1a3091b415.gif
Related Issue
https://jira.corp.magento.com/browse/PWA-712
Acceptance
Verification Stakeholders
@jimbo
@sirugh
Specification
Verification Steps
Screenshots / Screen Captures (if appropriate)
https://i.gyazo.com/98b75a0fb607669e627f2d1a3091b415.gif
Checklist