Skip to content

Allow for the attribute form model to manually set a given feature's attribute values and geometry#7067

Merged
nirvn merged 3 commits intomasterfrom
set_attribute_form
Feb 22, 2026
Merged

Allow for the attribute form model to manually set a given feature's attribute values and geometry#7067
nirvn merged 3 commits intomasterfrom
set_attribute_form

Conversation

@nirvn
Copy link
Member

@nirvn nirvn commented Feb 20, 2026

This PR unlocks a nice set of invokable functions that ultimately allows for QML editor widget to become not only read-only but with capability to edit feature attributes and geometry.

In turn, it allows for coming up with nice custom widgets. Here's an example (the nice icon horizontal list at the top):

Screencast.From.2026-02-20.18-11-53.mp4

@qfield-fairy
Copy link
Collaborator

qfield-fairy commented Feb 20, 2026

🍎 MacOS DMG universal builds

Download a MacOS DMG universal build of this PR for testing.
(Built from commit bcaa564)

📱 Android builds

Download an Android arm64 build of this PR for testing.
(Built from commit bcaa564)

Other Android architectures

🪟 Windows builds

Download a Windows build of this PR for testing.
(Built from commit bcaa564)

🐧 Linux AppImage builds

Download a Linux AppImage build of this PR for testing.
(Built from commit bcaa564)

Comment on lines +1130 to +1131
if ( !mLayer || !mFeatureModel )
return false;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if ( !mLayer || !mFeatureModel )
return false;
if ( !mLayer || !mFeatureModel )
{
return false;
}

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can do these later when we upgrade the whole .cpp -- something on my radar.

@nirvn nirvn merged commit 30edf6a into master Feb 22, 2026
28 checks passed
@nirvn nirvn deleted the set_attribute_form branch February 22, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants