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

Make NTP Grid layout adaptive for various screen sizes #3657

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

dus7
Copy link
Contributor

@dus7 dus7 commented Dec 2, 2024

Task/Issue URL: https://app.asana.com/0/1206226850447395/1208868059663007/f
Tech Design URL:
CC:

Description:

Addresses NTP layout issues when it's too wide to fit 4 items. It mostly changes how the spacings and item sizes are being defined.

Steps to test this PR:

  1. Create a bunch of Favorites.
  2. Test layout in various environments: iPod touch, iPhone, iPad, iPad with Split View, different orientations.

Definition of Done (Internal Only):

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPhone 14 Pro
  • iPad
  • iPod touch

OS Testing:

  • iOS 15
  • iOS 16
  • iOS 17

Internal references:

Software Engineering Expectations
Technical Design Template

@dus7 dus7 force-pushed the mariusz/fix-ntp-grid branch 2 times, most recently from 78a579b to 0757ace Compare December 2, 2024 11:33
@dus7 dus7 force-pushed the mariusz/fix-ntp-grid branch from 0757ace to 07e8bf6 Compare December 2, 2024 11:49
@dus7 dus7 marked this pull request as ready for review December 2, 2024 11:50
@dus7 dus7 requested review from brindy and samsymons December 2, 2024 11:51
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

LGTM

Tested on these with more than 1 screen's worth of favorites:

Screenshot 2024-12-02 at 13 35 08

Copy link
Contributor

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

LGTM, this fixes the layout issues on an iPod touch. Thanks!

@dus7 dus7 changed the base branch from main to release/7.148.0 December 3, 2024 09:58
@dus7 dus7 merged commit d2e604d into release/7.148.0 Dec 3, 2024
32 checks passed
@dus7 dus7 deleted the mariusz/fix-ntp-grid branch December 3, 2024 10:01
samsymons added a commit that referenced this pull request Dec 3, 2024
* release/7.148.0:
  Release 7.148.0-1 (#3661)
  Make a simple state machine to identify incorrect transitions  (#3660)
  support local storage fireproofing (#3612)
  Make NTP Grid layout adaptive for various screen sizes (#3657)
  Release 7.148.0-0 (#3659)
  Bump BSK, add Free Trials Feature Flag (#3655)
samsymons added a commit that referenced this pull request Dec 5, 2024
* main:
  Release 7.148.0-2 (#3676)
  Merge hotfix changes from 7.147.1 (#3674)
  Add `install` pixel (#3656)
  AI Chat browsing menu (#3635)
  Release 7.147.1-0 (#3672)
  Use actor isolation for `SubscriptionFeatureMappingCache` (#3670)
  cherry picked - already approved in #3666
  fix suggestion scrolling (#3654)
  Update autoconsent to v12.1.0 (#3650)
  Malware protection: bump BSK (#3617)
  Release 7.148.0-1 (#3661)
  Make a simple state machine to identify incorrect transitions  (#3660)
  support local storage fireproofing (#3612)
  Make NTP Grid layout adaptive for various screen sizes (#3657)
  Release 7.148.0-0 (#3659)
  Bump BSK, add Free Trials Feature Flag (#3655)
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.

3 participants