Skip to content

Conversation

dmarteau
Copy link
Member

Update to sphinx 8.

@dmarteau dmarteau mentioned this pull request May 12, 2025
1 task
@dmarteau dmarteau mentioned this pull request May 12, 2025
1 task
@dmarteau dmarteau force-pushed the update-sphinx-8 branch 2 times, most recently from 060bf44 to 9061465 Compare May 12, 2025 10:05
Copy link
Contributor

@Gustry Gustry left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.
Can you backport to lizmap_3_8 branch ? Thanks

@Gustry
Copy link
Contributor

Gustry commented May 13, 2025

Note, I think the build on the gitlab side should be checked.

Copy link
Contributor

@Gustry Gustry left a comment

Choose a reason for hiding this comment

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

Can you check the commit history, it seems there are 2 commits from another PR which has been merged here ?
Did the build has been checked on GitLab, with the correct versions used ?

* Since |qgis_3|, you can use QGIS expressions, in your labels for instance. You can create automatic source label
according to visible layers following this example on the
`QGIS documentation <https://docs.qgis.org/latest/en/docs/user_manual/print_composer/composer_items/composer_label.html#id4>`_.
according to visible layers following `this example on the QGIS documentation <https://docs.qgis.org/latest/en/docs/user_manual/print_composer/composer_items/composer_label.html#id4>`_.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
according to visible layers following `this example on the QGIS documentation <https://docs.qgis.org/latest/en/docs/user_manual/print_composer/composer_items/composer_label.html#id4>`_.
according to visible layers following
`this example on the QGIS documentation <https://docs.qgis.org/latest/en/docs/user_manual/print_composer/composer_items/composer_label.html#id4>`_.

The line seems too long ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants