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
Added move to bottom in layertreeview context menu #35531
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one!
I could not find tests for |
There is tests/src/core/testqgslayertree.cpp . If it doesn't fit in there add a new one 👍 |
While it is the closest match, I don't think it fits, no |
In this case just add a new file (in there as cpp test or if you prefer you can also do it on python side, your call). |
You were right, |
|
||
int depth() const; | ||
%Docstring | ||
Returns the depth of this node, i.e. the number of it's ancestors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HMm - sipify test should be rejecting this as out-of-sync...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh I see, I don't think you've pushed your "it's" fix yet?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I missed that one, should be ok now
Please add |
@pigreco please apply for labelling power here: https://github.com/orgs/qgis/teams/community/members |
Description
Moving around layers on big projects was made easier with Move to top but still one had to move manually his basemaps to the bottom of the layer list, a cumbersome task in big layer trees. This PR adds a Move to bottom option on the layer tree context menu that works in the same way as Move to top.
Also fixes #35530