-
-
Notifications
You must be signed in to change notification settings - Fork 213
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
Sync tabs with swipe view in FeatureForm. #5281
Conversation
also fix elide in MapValue.qml by setting a width.
🎉 Ta-daaa, freshly created APKs are available for 5fe5929: arm64-android |
src/qml/FeatureForm.qml
Outdated
/** | ||
* The main form content area | ||
*/ | ||
SwipeView { | ||
id: swipeView | ||
Layout.fillWidth: true | ||
Layout.fillHeight: true | ||
currentIndex: tabRow.currentIndex | ||
currentIndex: tabBarListView.currentIndex |
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.
For the sake of avoiding noise, I would actually keep the same id, i.e. tabRow.
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.
Can't spot a regression, and it does fix the issue you were trying to address. All good.
In this PR, we address two minor issues:
1- Fix #5242: The issue where tabs are not following the selected container has been resolved.
2- Additionally, we've fixed the elide problem in MapValue.
These changes aim to improve the overall user experience and stability of application.
Resolved video:
Screencast.from.2024-05-31.07-34-16.webm