Skip to content
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

Relation editor widget's print child feature atlas layout to PDF #6031

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

nirvn
Copy link
Member

@nirvn nirvn commented Feb 22, 2025

This PR adds the possibility to print an atlas PDF tied to children of a parent within the parent feature form through the addition of actions within the relation editor widget's list. Here's how it looks like:

Screenshot From 2025-02-22 16-48-42

As the screenshot shows, the work here led to the delete button being relocated into the 3-dot menu. Since it has become far less easy to accidentally tap on the delete button, it's now enabled all the time (irrespective of the feature form's editing state), just like we can delete the parent through its feature form all the time.

For good taste, I've also added a copy feature attributes to clipboard action. Interesting possibilities ahead of us! Thanks to Vevey for supporting this development.

Note that I've taken the time to de-duplicate tons of code shared across the relation editor and the ordered relation editor items. This makes the maintenance of these two sub types much easier as most fixes won't have to be applied in two different locations.

}
}

Loader {
Copy link
Member Author

Choose a reason for hiding this comment

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

The use of a loader here is to defer the cost involved in atlas-related item / model generation until we open the menu.

Copy link
Member

Choose a reason for hiding this comment

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

Do you want to leave that as a comment in the file, so someone working on this in the future will directly know?

@qfield-fairy
Copy link
Collaborator

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