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

Refactor the core of the app using SwiftUI #492

Merged
merged 42 commits into from
Jan 21, 2024
Merged

Conversation

anli5005
Copy link
Member

@anli5005 anli5005 commented Nov 10, 2023

The long-awaited refactor of our app in SwiftUI is here.

New home screen

Note that this PR updates the minimum deployment target to iOS 16.0.

Some tasks that I've deferred until later PRs:

  • Rewriting the PAC code tab in SwiftUI (as that relies on navigation bar controls)
  • Fixing a rare crash when switching to the GSR/laundry tabs after re-pinning them to the tab bar (those are also related to navigation bar controls) now fixed!
  • Deleting now-unused code
  • Refactoring the "Edit Features" screen to use the new AppFeature structs
  • Tweaking the home screen with feedback from design

@george-botros
Copy link
Contributor

Should the feature icons be white on light mode?
Uploading Simulator Screenshot - iPhone 15 Pro - 2023-11-12 at 12.49.33.png…

Copy link
Member

@JHawk0224 JHawk0224 left a comment

Choose a reason for hiding this comment

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

Looks good so far

@george-botros
Copy link
Contributor

Btw, aesthetics of poll history page is inconsistent with aesthetics of a poll on the homepage

simulator_screenshot_5D1D4774-7FAF-4110-BDC5-6DF97073E28B

@JHawk0224
Copy link
Member

We should also remove the debug button

@JHawk0224
Copy link
Member

Btw, aesthetics of poll history page is inconsistent with aesthetics of a poll on the homepage

fixed on mass-deletion branch

@anli5005 anli5005 merged commit b321d5a into main Jan 21, 2024
1 check passed
@JHawk0224 JHawk0224 deleted the anli/09-17-refactor branch February 4, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants