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

Updated welcome screen to a newer style #360

Merged
merged 6 commits into from
Nov 14, 2023

Conversation

heathermh
Copy link
Collaborator

Updated layout to single screen, updated to be in line with apple standard designs. Friendly for dark mode and dynamic text. Includes accessibility improvements to work better with VoiceOver and Voice Control.

Partially resolves #331

Updated layout to single screen, updated to be in line with apple standard designs
Friendly for dark mode and dynamic text
FiveCalls/FiveCalls/Localizable.strings Outdated Show resolved Hide resolved
FiveCalls/FiveCalls/Welcome.swift Outdated Show resolved Hide resolved
FiveCalls/FiveCalls/Welcome.swift Outdated Show resolved Hide resolved
FiveCalls/FiveCalls/Welcome.swift Outdated Show resolved Hide resolved
Switched to grid
Adjusted size/color of symbols
@heathermh
Copy link
Collaborator Author

Adjusted alignment of symbols, made symbols bigger, and adjusted the colors a bit. We could add a little red in the spend 5 minutes symbol, if you like. I removed the period after "Spend 5 minutes, make 5 calls" so it's more consistent with other content.

Still ready to go for dark mode, dynamic text, accessibility support (VoiceOver, Voice Control). One small issue I see is that on the iPhone SE, the continue button is a little cut off. Not sure if it's worth adjusting for? I'll include a screen shot of it below.
image
image
image
image

Grouped items, updated accessibility labels to make it easier to navigate top portion of dashboard and select a location using VoiceOver.
Copy link
Member

@nickoneill nickoneill left a comment

Choose a reason for hiding this comment

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

Looks great, a few comments to adjust and then feel free to merge.

In the future it would be nice to see unrelated changes like the location header accessibility changes in another small PR. it's easier to review smaller changesets and I certainly don't mind having more PRs to review

FiveCalls/FiveCalls/LocationHeader.swift Outdated Show resolved Hide resolved
FiveCalls/FiveCalls/LocationHeader.swift Outdated Show resolved Hide resolved
.opacity(subMessageOpacity)
.gridColumnAlignment(.leading)
}
Spacer()
Copy link
Member

Choose a reason for hiding this comment

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

Do we need two spacers here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

For the welcome view to accommodate dynamic text, I wrapped it in a ScrollView. That changes how spacers work, and two spacers gives us good space / drops the button almost to the bottom with default text sizes and a modern phone. This is also why the button gets cut off a bit on the small phone.

FiveCalls/FiveCalls/Localizable.strings Outdated Show resolved Hide resolved
@heathermh heathermh marked this pull request as ready for review November 14, 2023 00:53
@heathermh heathermh merged commit 9338d27 into main Nov 14, 2023
1 check passed
@heathermh heathermh deleted the heathermh/swiftui-accessibility-updates branch November 14, 2023 01:05
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.

SwiftUI design / 2.0 release
2 participants