Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Index Alsos - Add support for RTL #1923

Merged
merged 5 commits into from
Aug 28, 2019
Merged

Index Alsos - Add support for RTL #1923

merged 5 commits into from
Aug 28, 2019

Conversation

DenisHdz
Copy link
Contributor

Resolves bbc/simorgh#3419

Overall change:
Add support for RTL to Index Alsos.

Code changes:

  • Pass dir prop to apply the right padding-left/right to the IndexAlsosMediaIndicator
  • Refactor Index Alsos container

  • I have assigned myself to this PR and the corresponding issues
  • Automated (jest and/or cypress) tests added (for new features) or updated (for existing features)
  • This PR requires manual testing

@DenisHdz DenisHdz added ws-home Tasks for the WS Home Team ws-fp-phase2 labels Aug 27, 2019
@DenisHdz DenisHdz self-assigned this Aug 27, 2019
Copy link
Contributor

@andrew-nowak andrew-nowak left a comment

Choose a reason for hiding this comment

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

lgtm

@david-boydell david-boydell self-assigned this Aug 28, 2019
@david-boydell
Copy link

Sweet, ready for merge :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ws-home Tasks for the WS Home Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Index Alsos icons missing padding on RTL sites
4 participants