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

SHS-5572: The arrows on a Spotlight (with no image) are displaying incorrectly on mobile #1503

Merged

Conversation

mariannuar
Copy link
Collaborator

READY FOR REVIEW

Summary

Fix arrows position on mobile for Spotlight (with no image)

Need Review By (Date)

04/19

Urgency

medium

Steps to Test

  1. Create a Flexible Page on Colorful and Traditional and add a component of Spotlight without adding images and add another with images
  2. Confirm the position of the arrow in all viewports (specially on mobile) in both components look better

PR Checklist

@mariannuar mariannuar requested a review from cienvaras April 12, 2024 01:39
@mariannuar mariannuar self-assigned this Apr 12, 2024
@cienvaras cienvaras changed the base branch from develop to fk-stnfd-sprint-48 April 17, 2024 21:17
… shs-5572--fix-arrows-for-spotlights-with-no-image
Copy link
Collaborator

@cienvaras cienvaras left a comment

Choose a reason for hiding this comment

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

🚀

@cienvaras cienvaras merged commit bba5a7d into fk-stnfd-sprint-48 Apr 17, 2024
14 of 15 checks passed
@cienvaras cienvaras deleted the shs-5572--fix-arrows-for-spotlights-with-no-image branch April 17, 2024 22:52
cienvaras added a commit that referenced this pull request May 3, 2024
* SHS-5571: A11y Level A: Empty headings on Vertical Timeline components (#1502)

* fix(shs-5571): A11y Level A: Empty headings on Vertical Timeline components

* fix(shs-5571): minor code consistency fix in timeline template

---------

Co-authored-by: Mari Núñez <mariannuar@admins-MacBook-Pro.local>
Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* fix(shs-5572): arrows on a Spotlight (with no image) displaying incorrectly on mobile (#1503)

Co-authored-by: Mari Núñez <mariannuar@admins-MacBook-Pro.local>
Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* feat(shs-5574): cleanup grid styles - remove flexbox fallback

* SHS-5568: Allow editors to select style on Accordion component (#1501)

* feat(shs-5568): add new accordion style field

* feat(shs-5568): add hb-accordion-secondary class if new field is set to

* feat(shs-5568): add hb-accordion-secondary class to a parent wrapper class

* feat: add hs-accordion-secondary to details element

* chore: remove unused statement

* feat(shs-5568): update styles for handling secondary accordion paragraph option

* feat(shs-5568): add update hook to set field_accordion_style to default on existing accordion components

---------

Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* fix(shs-5091): fix empty headings on views (#1509)

Co-authored-by: Mari Núñez <mariannuar@admins-MacBook-Pro.local>
Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* SHS-5583: empty headings in banners and spotlights (#1510)

* fix(shs-5583): empty headings in banners and spotlights

* fix(shs-5583): minor fix in spotlight and gradient-hero templates

---------

Co-authored-by: Mari Núñez <mariannuar@admins-MacBook-Pro.local>
Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* fix(shs-5574): prevent grid element overflow

* chore(shs-5574): update percy cli version / fix percy default test to consolidate colorful and traditional

---------

Co-authored-by: Mariana Núñez <48533432+mariannuar@users.noreply.github.com>
Co-authored-by: Mari Núñez <mariannuar@admins-MacBook-Pro.local>
Co-authored-by: Hector Lopez <hectorlopeztrejo9@gmail.com>
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.

2 participants