MerginMaps / mobile

Mobile app for Mergin Maps 📲. Open QGIS projects and manage vector data on your phone, even offline 📶 (Android/iOS/Windows app)
https://merginmaps.com
GNU General Public License v2.0
280 stars 62 forks source link

Fixing app freeze in MMFormPage #3502

Closed VitorVieiraZ closed 1 week ago

VitorVieiraZ commented 3 weeks ago

Switching between different forms on FormPage during feature addition or editing, and clicking a button that changes the FormPage state or opens a new drawer over the existing one (e.g., the 'Edit Geometry' button or the '+' button in relations form), would cause the application to freeze.

Fixes #3483

github-actions[bot] commented 3 weeks ago

Pull Request Test Coverage Report for Build 9470291617

Details


Files with Coverage Reduction New Missed Lines %
input/app/attributes/attributecontroller.cpp 1 77.37%
input/core/merginuserinfo.cpp 3 73.76%
<!-- Total: 4 -->
Totals Coverage Status
Change from base Build 9455523246: 0.04%
Covered Lines: 7676
Relevant Lines: 12800

💛 - Coveralls
github-actions[bot] commented 3 weeks ago

Pull Request Test Coverage Report for Build 9470291617

Details


Files with Coverage Reduction New Missed Lines %
input/app/attributes/attributecontroller.cpp 1 77.37%
input/core/merginuserinfo.cpp 3 73.76%
<!-- Total: 4 -->
Totals Coverage Status
Change from base Build 9455523246: 0.04%
Covered Lines: 7676
Relevant Lines: 12800

💛 - Coveralls
github-actions[bot] commented 3 weeks ago

Pull Request Test Coverage Report for Build 9470291617

Details


Files with Coverage Reduction New Missed Lines %
input/app/attributes/attributecontroller.cpp 1 77.37%
input/core/merginuserinfo.cpp 3 73.76%
<!-- Total: 4 -->
Totals Coverage Status
Change from base Build 9455523246: 0.04%
Covered Lines: 7676
Relevant Lines: 12800

💛 - Coveralls
tomasMizera commented 2 weeks ago

Would be really nice to understand why this is happening and get a minimal reproducible code that we could send to Qt guys. It will help us to see if the issue is fixed in any future Qt versions or not.

tomasMizera commented 1 week ago

The current situation - we are still exploring all possible places for these ANRs. Unfortunately, it is not ready for testing at the moment.

coveralls commented 1 week ago

Pull Request Test Coverage Report for Build 9580910955

Details


Totals Coverage Status
Change from base Build 9455523246: 0.02%
Covered Lines: 7673
Relevant Lines: 12800

💛 - Coveralls
github-actions[bot] commented 1 week ago

Pull Request Test Coverage Report for Build 9581257493

Details


Files with Coverage Reduction New Missed Lines %
input/app/attributes/attributecontroller.cpp 1 77.37%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 9547366048: 0.008%
Covered Lines: 7670
Relevant Lines: 12797

💛 - Coveralls
github-actions[bot] commented 1 week ago

Pull Request Test Coverage Report for Build 9581257493

Details


Files with Coverage Reduction New Missed Lines %
input/app/attributes/attributecontroller.cpp 1 77.37%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 9547366048: 0.008%
Covered Lines: 7670
Relevant Lines: 12797

💛 - Coveralls
github-actions[bot] commented 1 week ago

Pull Request Test Coverage Report for Build 9581257493

Details


Files with Coverage Reduction New Missed Lines %
input/app/attributes/attributecontroller.cpp 1 77.37%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 9547366048: 0.008%
Covered Lines: 7670
Relevant Lines: 12797

💛 - Coveralls