-
-
Notifications
You must be signed in to change notification settings - Fork 219
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
Information drawer functionality #5251
Conversation
when we drag navigatingDrawer it should change position of both compassArrow and ScaleBar too.
based on showing always or not in settings + formatting NavigatingDrawer.
🎉 Ta-daaa, freshly created APKs are available for f8cd5a8: arm64-android |
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.
@mohsenD98 , great work here, super excited to see this coming along! Some comments added.
I'm thrilled to have been able to help and can't wait to dive into the next steps! Thank you for your kind words and for taking the time to add some comments. I'm always eager to learn from your feedback and make adjustments to ensure that the code is accurate, clear, and helpful. |
Add mainBackgroundTransparentColor. Rename navigatingDrawer to informationDrawer. Move some input logic into the InformationDrawer.
Move SensorInformationView from qgisMobileApp.qml to InformationDrawer.
also add elevationProfile.visible to navigationInformationViewEnabled. because when elevationProfile is enabled, navigationInformation causes more margin. we should hide it in background.
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.
This looks good to me now. Let's merge this when the CI is green and work on follow up tweaks in separate PRs to avoid too many line changes :)
This PR aims at improving both the style as well as the UX of the way we display positioning/navigation/sensor information on top of the main canvas.
@mohsenD98 , I've given you write access to QField's main repository, I'd recommend you update you branch to use this.