-
Notifications
You must be signed in to change notification settings - Fork 199
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
Impl bottom floating material glassed tabbar in iOS #483
Impl bottom floating material glassed tabbar in iOS #483
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
But, please wait a review from @shin-usu about #483 (comment) 🙏🏼
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shimastripe
Sorry for the late response 🙇🏻
I left some comments. Thanks!
// bottom floating tabbar padding | ||
Color.clear.padding(.bottom, 60) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TimetableView has list mode and grid mode, and user can switch each mode. (Grid mode is developing now)
Grid mode also has floating bottom tab, like list mode. Therefore, I think it would be appropriate for this code to be added to the TimetableView, not the TimetableListView,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I understand your suggestion. However TimetableListView and TimetableGridView are ScrollView.
So if the padding is set in TimetableView, the scrollable area is above the floating tab bar.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I understand.
It seems to be fine as it is👌🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉 🍾
Issue
Overview (Required)
Links
Screenshot (Optional if screenshot test is present or unrelated to UI)
Movie (Optional)
Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-08-15.at.20.39.24.mp4