Skip to content

Conversation

RichSutherSky
Copy link
Contributor

@RichSutherSky RichSutherSky commented Aug 13, 2025

This PR adds additional spacing values that we missing for Android and iOS

Remember to include the following changes:

  • README.md
  • Tests
  • Storybook examples created/updated for changes to tokens and icons

@RichSutherSky RichSutherSky added the minor minor label Aug 13, 2025
@RichSutherSky RichSutherSky force-pushed the DON-1958-android-backpack-add-missing-sizes branch from b7ebb64 to 955c67b Compare August 13, 2025 13:54
@RichSutherSky RichSutherSky changed the title [DON-1958] Add missing spacing tokens for Android, iOS and common. [DON-1958] Add missing spacing tokens for Android and iOS Aug 13, 2025
@RichSutherSky RichSutherSky marked this pull request as ready for review August 13, 2025 13:58
@RichSutherSky RichSutherSky requested review from a team as code owners August 13, 2025 13:58
@RichSutherSky RichSutherSky marked this pull request as draft August 13, 2025 13:59
@tykayoshi
Copy link
Contributor

Tested on iOS backpack, this doesn't break the build of backpack everything is fine. However the new spaces aren't recognized by the scripts. Need to update them to be able to see these new values in backpack. Ticket here: https://skyscanner.atlassian.net/browse/DON-1974

@tykayoshi
Copy link
Contributor

tykayoshi commented Aug 13, 2025

This has been fixed on iOS now, we are ready for the merge on our end so can approve on iOS side. Will need approval for android

Copy link
Contributor

@tykayoshi tykayoshi left a comment

Choose a reason for hiding this comment

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

Good on iOS

@RichSutherSky RichSutherSky marked this pull request as ready for review August 14, 2025 16:00
Copy link

@xiaogliu xiaogliu left a comment

Choose a reason for hiding this comment

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

LGTM

@RichSutherSky RichSutherSky merged commit a435841 into main Aug 15, 2025
4 checks passed
@RichSutherSky RichSutherSky deleted the DON-1958-android-backpack-add-missing-sizes branch August 15, 2025 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants