Fix missing highlight for Qt 6.10#4511
Merged
Merged
Conversation
Coverage Report for CI Build 26752217054Coverage increased (+0.06%) to 57.837%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions6 previously-covered lines in 2 files lost coverage.
Coverage Stats
💛 - Coveralls |
📦 Build Artifacts Ready
|
📦 Build Artifacts Ready
|
📦 Build Artifacts Ready
|
📦 Build Artifacts Ready
|
tomasMizera
approved these changes
Jun 1, 2026
Withalion
added a commit
that referenced
this pull request
Jun 2, 2026
* Initial build for macos and ios * Fix Bluetooth model syntax * Update QGIS to 3.44.10 * Format cmake files * Fix failing Linux build * Add log to Linux CI * Install mono on linux CI * Remove mono from deleted packages * Remove debug output from Linux CI * Update Geodiff to 2.2.0 * Fix nuget for Linux CI again * Reintroduce Geodiff warning flag * Update runner versions and cmake minor version * Update xCode version to 26.4.1 * Remove deployment target for apple builds * Fix dep install on Linux CI * Update or remove manual versions of Qt in CI * Remove unused env var in Gallery CI * Fix missing mono package on newer Ubuntu * Return xvfb package on Linux CI * Wrap geodiff compilation flag to apple only * Add package for libxcrypt * Update signing certs for iOS * Another try to fix iOS certs * Fix iOS signing cert * Format geodiff port * Update QtDeclarative to 6.10.3 * Remove QtKeychain overlay port * Update cmake info in INSTALL.md * Remove QtMultimedia overlay port, use VCPKG default * Update QtPositioning to 6.10.3 * Remove GStreamer overlay port and use VCPKG default * Remove libb2 overlay port, vcpkg bug should be fixed now * Update libiconv overlay port to match VCPKG port * Add LibSpatialIndex port info to README * Remove unused qca patches, update README * Remove libspatialindex overlay port & pin version in vcpkg.json * Remove qtbase overlay port in place of VCPKG default port * Add wayland support for linux build * Update xcode in Android CI * Update cmake version in CI to match used by VCPKG * Add wayland packages for linux build * Add pcre2 port to fix android linker error * Update android CI * Update Android setup based on Qt 6.10 * Fix Android build issues * Format cmake * Fix Android ARMv7 build * WIP fixed safe area rendering on iOS and cmake configuration for android (#4472) * Add libpq port for iOS * Fix another libpq issue on iOS * Fix iOS build arg for make in VCPKG * Remove libiconv port * Fix qtpositioning patches for windows * Fix qgis patches for Windows * Add fix for windows build failing on lazperf * Amend last windows fix to target only windows * Fix local ios VCPKG build failing on race condition * Fix qtmultimedia build on iOS * Fix windows QGIS again * Fix windows packager * Fix ios Spatialite build * Fix missing symbols for ios * Format cmake * Upload AABs to Google play * Fix failing maptools tests QGIS 3.44 started enforcing constness, where we relied on semi-constness * Fix lcov * Polish linux CI file * Mitigate coveralls breaking CI * Fix Linux install * Fixed the top and bottom overpadding for feature list page drawer (#4510) * Update Linux cleanup action * Fix windows gallery CI * Fix missing highlight for Qt 6.10 (#4511) * Fix linux build * Explore Linux CI mirror list * Explore Linux CI debug info * Remove broken enzu mirror * Remove unnecessary step * Fix permission problem --------- Co-authored-by: Matej Bagar <matej.bagar@lutraconsulting.co.uk> Co-authored-by: Gabriel Bolbotina <80618569+gabriel-bolbotina@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Alternative approach to #4506
In Qt 6.9+ instead of triggering
onPropertyChangedsignal first the value checks for equality.QgsGeometrydoesn't have equality operator, but hasequals()method instead. We wrapQgsGeometryinMMGeometryclass which adds equality operator (this just callsequals()). AllQgsGeometryvalues should be exposed to QML throughMMGeometryfor bindings to work properly.In the end we decided to patch
QgsGeometryto include equality operator, which callsequals(). AlsoQgsGeometryhas been exposed to QML asqgsGeometryvalue type.Just a thought: should we prohibit the usage ofQgsGeometryin the app?