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

Remove custom back button in VCs #789

Merged
merged 3 commits into from
Sep 11, 2017

Conversation

bdorfman-stripe
Copy link
Contributor

Only use system back button or Cancel buttons everywhere.

Our fake Back button we had been using doesn't match iOS 11 back buttons and doesn't behave correctly. It will be even harder to fix given iOS 11's different nav bar sizes. Instead just use Cancel buttons or the system back button everywhere instead.

Only use system back button or Cancel buttons everywhere.
@joeydong-stripe
Copy link
Contributor

👍

Copy link
Contributor

@bg-stripe bg-stripe left a comment

Choose a reason for hiding this comment

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

Hm, my one comment is that the shipping methods page now looks a bit odd. Maybe we should change the titles to "Shipping Address" and "Shipping Method" instead of just "Shipping" on both pages?

@bdorfman-stripe
Copy link
Contributor Author

Yeah maybe? I don't think it's that big a deal tbh and adding extra text to the end will just lead to the back button text being truncated anyway.

Copy link
Contributor

@bg-stripe bg-stripe left a comment

Choose a reason for hiding this comment

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

fair - lgtm then

@joeydong-stripe
Copy link
Contributor

There some other adjustments I would make to this page, anyway, some other time:

  • Remove or shrink the top package image so that we're aligned with our add card view controller

  • Move the shipping price into the lefthand side so that we're not showing both text and a checkmark on the righthand side

screen shot 2017-09-07 at 4 32 18 pm

  • Adjust the view controller title to match its purpose (like @bg-stripe mentioned), make sure the button that we tapped to get to this page matches that title, and remove any redundant section headers if needed

Remove "Back" string from all translations.
Fix Xcode project to correctly interpret the simplified Chinese strings file as UTF-8
Remove unused methods.
@stripe-ci stripe-ci removed the approved label Sep 8, 2017
@bdorfman-stripe bdorfman-stripe merged commit 1597bc6 into master Sep 11, 2017
@bdorfman-stripe bdorfman-stripe deleted the bdorfman-remove-custom-back-button branch September 11, 2017 22:26
davidme-stripe pushed a commit that referenced this pull request Feb 18, 2022
Fixes SwiftUI crash from double presentation (custom flow)
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.

4 participants