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

Update about screen #2152

Merged
merged 7 commits into from
Nov 15, 2023
Merged

Update about screen #2152

merged 7 commits into from
Nov 15, 2023

Conversation

brindy
Copy link
Contributor

@brindy brindy commented Nov 13, 2023

Task/Issue URL: https://app.asana.com/0/414709148257752/1205916913109009/f
Tech Design URL:
CC:

Description:

Convert about screen to SwiftUI and update content.

Steps to test this PR:

  1. Launch the app on iOS 15, 16, 17 phone and iPad and check about screen looks OK.

Internal references:

Software Engineering Expectations
Technical Design Template

@brindy brindy requested review from a team, jaceklyp and amddg44 and removed request for a team and jaceklyp November 13, 2023 12:15
Copy link
Contributor

@amddg44 amddg44 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 the following in portrait, landscape, light / dark mode (and also larger font sizes and that the links work) 😅 :

  • iOS 17

  • iPhone 15 Pro Max

  • iPhone 15 Pro

  • iPhone 15

  • iPhone 13 Mini

  • iPhone SE (Gen 2)

  • iPad Pro

  • iPad Air

  • iPad Mini

  • iOS 16

  • iPhone 13 Pro Max

  • iPhone 13 Pro

  • iPhone 13

  • iPhone 13 Mini

  • iPhone SE (Gen 2)

  • iPad Pro

  • iPad Air

  • iPad Mini

  • iOS 15:

  • iPhone 13 Pro Max

  • iPhone 13 Pro

  • iPhone 13

  • iPhone 12 Mini

  • iPhone SE (Gen 1)

  • iPad Pro

  • iPad Air

  • iPad Mini

@brindy
Copy link
Contributor Author

brindy commented Nov 14, 2023

Epic @amddg44 ! Thank you :)

@brindy brindy merged commit ddb189f into develop Nov 15, 2023
10 checks passed
@brindy brindy deleted the brindy/about-screen branch November 15, 2023 16:29
samsymons added a commit that referenced this pull request Nov 16, 2023
* develop:
  Use upstream fastlane 2.217.0 (#2161)
  Update about screen (#2152)
  Fixes a crasher due to a test line merged by mistake. (#2162)
samsymons added a commit that referenced this pull request Nov 16, 2023
* develop: (36 commits)
  Rolls back a BSK change by mistake
  Updates BSK
  Updates BSK
  Document LinkPresentation usage (#2172)
  Autofill "Never Save for this Site" feature (#2104)
  Use upstream fastlane 2.217.0 (#2161)
  Update about screen (#2152)
  Fixes a crasher due to a test line merged by mistake. (#2162)
  remove return user pixel (#2146)
  Revert to auto signing (#2158)
  BSK changes for NetP iOS Geoswitching (#2141)
  Update sync e2e tests to new setup flow (#2151)
  Fix for duplicated "atb" in Pixel request (#2139)
  Updating tests to support latest reference tests (#2145)
  Add NetP widget (#2142)
  Normalize ampUrl in breakage pixel (#2154)
  Update embedded files
  Update version number
  Bump submodules/privacy-reference-tests from `0d23f76` to `7519c3d` (#2135)
  Properly refresh home page favorites view when updating favorites display mode (#2148)
  ...
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.

2 participants