Skip to content

Update annotations documentation #9727

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

Merged
merged 7 commits into from
Mar 21, 2025
Merged

Update annotations documentation #9727

merged 7 commits into from
Mar 21, 2025

Conversation

DelazJ
Copy link
Collaborator

@DelazJ DelazJ commented Mar 14, 2025

Fixes #9175, fixes #9183, fixes #9193, fixes #9194, fixes #9211, fixes #9720

Goal:

Ticket(s): #

  • Backport to LTR documentation is requested

@DelazJ DelazJ added Annotation backport release_3.40 On merge create a backported pull request to 3.40 labels Mar 14, 2025
@DelazJ
Copy link
Collaborator Author

DelazJ commented Mar 14, 2025

@swood44, if ever you want to help review?

Comment on lines 977 to 985
.. the description below should ABSOLUTELY be REVIEWED!!!
And I don't get why there's "Lock aspect ratio" vs dedicated "width/height/lock apect ratio"
depending on the drop-down item

* :guilabel:`Scale dependent size`: the picture annotation will always be rendered at the same size, ???
* :guilabel:`Fixed size`: the picture annotation will always be rendered on screen at the same size,
regardless of the map scale.
* :guilabel:`Relative to map`: the picture annotation will always be rendered at the same position
relative to the map canvas bounds, regardless of the map scale
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@nyalldawson Can you help describe these options? I tried but failed to get into words the schema behind their behavior. And I couldn't find description in code either. Thanks.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Options description moved to #9738 in order to get this PR merged

@DelazJ DelazJ enabled auto-merge March 21, 2025 16:43
@DelazJ DelazJ merged commit 38e6fec into qgis:master Mar 21, 2025
3 checks passed
@DelazJ DelazJ deleted the annotations branch March 21, 2025 16:46
DelazJ added a commit that referenced this pull request Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment