Skip to content
This repository has been archived by the owner on Jan 26, 2024. It is now read-only.

Added: Safe Area of Elements in the Bottom Elements(#499) #500

Merged
merged 8 commits into from
Sep 3, 2021

Conversation

AishwaryShrivastav
Copy link
Contributor

@AishwaryShrivastav AishwaryShrivastav commented Sep 26, 2020

Related Issues

Closes #499

Short Description and Why It's Useful

Added Safe area padding on cookie notification, offline badge and bottom navigation

Screenshots of Visual Changes before/after (If There Are Any)

Video sharing visuals changes for this Improvement

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change

Contribution and Currently Important Rules Acceptance

Added Safe area padding on cookie notification, offline badge and bottom navigation
@AishwaryShrivastav
Copy link
Contributor Author

Currently only the bottom Elements in the Capybara need to have safe area, I have added them in Cookie notification, Bottom navigation and Offline Badge.
Many thanks to @dt2patel for the suggestion.

@AishwaryShrivastav
Copy link
Contributor Author

Here is a tome coded video where we discussed this implementation

@AishwaryShrivastav
Copy link
Contributor Author

Also added safe area padding for dropdowns.

Copy link
Collaborator

@Fifciu Fifciu left a comment

Choose a reason for hiding this comment

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

Please update changelog

@AishwaryShrivastav
Copy link
Contributor Author

Added change logs in #573

@AishwaryShrivastav
Copy link
Contributor Author

Hey @Fifciu Please have a look at these changes, they looks good to me, we can merge them in the next release

@dt2patel dt2patel merged commit 2491448 into vuestorefront:develop Sep 3, 2021
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.

Add Webkit Safe Area in Capybara
3 participants