Skip to content

Conversation

@nitinog10
Copy link
Contributor

Fixes an issue where the "Complete Order" button could be hidden on the checkout page due to viewport height and safe-area behavior.

Root Cause

The checkout container relied on 100vh, which includes browser UI on iOS Safari and some desktop layouts, causing content near the bottom (including the button) to be cut off.

Solution

  • Replaced 100vh with 100dvh and added a 100vh fallback
  • Added env(safe-area-inset-bottom) padding to ensure visibility on iOS devices with notches
  • Applied changes consistently across desktop and mobile breakpoints

How to Test

  1. Open checkout page
  2. Proceed to ticket checkout
  3. Scroll to the bottom
  4. Confirm the Complete Order button is visible and accessible

Related Issue

Closes #887

@github-actions
Copy link
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@daveearley
Copy link
Contributor

Thanks for the effort here @nitinog10.

I believe #887 may no longer be an issue, as we've since updated the design. The new layout does not have a fixed footer.

Can you confirm

@nitinog10
Copy link
Contributor Author

Hi @daveearley, I've checked the current codebase and can confirm that #887 is no longer an issue - there's no fixed footer in the current checkout layout.

However, the changes in this PR address a different issue: iOS Safari's viewport height behavior where 100vh includes the address bar, which can cause the "Complete Order" button to be cut off on mobile Safari. The fixes add 100dvh fallback and safe-area-inset-bottom padding.

Should I close this PR since it's addressing a separate concern, or would you like me to create a new issue/PR specifically for the iOS Safari viewport fix?

@daveearley
Copy link
Contributor

@nitinog10 No problem, I can merge this. Thank you.

@daveearley daveearley merged commit c7a64c8 into HiEventsDev:develop Jan 24, 2026
1 check failed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛Missing 'Complete Order' button on the checkout page.

2 participants