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

Hide floating tab bar on iPad when scrolling #5200

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

mazevedofs
Copy link
Collaborator

Phabricator: https://phabricator.wikimedia.org/T383759

Notes

  • Works on current UIKit implementation

Test Steps

  1. Scroll up and down on Article View, Explore, Saved, Search and History.
  2. Make sure it behaves as expected
  3. Check if iPhone is not affected

@mazevedofs mazevedofs requested review from a team and tonisevener and removed request for a team February 13, 2025 20:51
@tonisevener tonisevener added 7.7.2 and removed 7.7.2 labels Feb 14, 2025
Copy link
Collaborator

@tonisevener tonisevener left a comment

Choose a reason for hiding this comment

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

I notice the tab bar can get "stuck" in a hidden state when the content is short. I faced a similar issue with the navigation bar hiding - you can reference calculateNavigationBarHiddenState for how I fixed. Not a must-fix though, in my opinion, as long as design review is good with it.

in.mov

@tonisevener tonisevener removed the 7.7.2 label Feb 20, 2025
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.

2 participants