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

Minor improvements of z order feature #5145

Merged
merged 4 commits into from
Oct 21, 2022
Merged

Conversation

bsekachev
Copy link
Member

@bsekachev bsekachev commented Oct 18, 2022

Motivation and context

Resolved #5142
Resolved #2180
Related #3942

  • Fixed issue when changing an object causes changing of the current z layer
  • Added sorting by z order
  • If the sorting is applied, additional indication added to the sidebar

image

How has this been tested?

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@bsekachev bsekachev added bug Something isn't working enhancement New feature or request labels Oct 18, 2022
@bsekachev bsekachev changed the title [WIP] Minor improvements of z order feature Minor improvements of z order feature Oct 18, 2022
@bsekachev
Copy link
Member Author

@klakhov
Could you please take a look?

@bsekachev
Copy link
Member Author

/check

@github-actions
Copy link
Contributor

github-actions bot commented Oct 18, 2022

✔️ All checks completed successfully
📄 See logs here

@klakhov
Copy link
Contributor

klakhov commented Oct 21, 2022

@bsekachev Improvements seem nice to me. I noticed that shortcuts for brightness/saturation ... mentioned in #5142 are not working for me eigther. We have some description about them in docs
image
Should we add them or remove notes from docs?

@bsekachev
Copy link
Member Author

@klakhov

We do not have these shortcuts anymore for a long time. Since these image settings moved from CVAT settings (F2).
Yes, I will remove them from docs.

@bsekachev bsekachev merged commit dfcc69c into develop Oct 21, 2022
@bsekachev
Copy link
Member Author

#5154

@bsekachev bsekachev deleted the bs/fixed_issue_5142 branch October 24, 2022 07:21
@nmanovic nmanovic mentioned this pull request Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix z-axis z-order: showing and sorting
2 participants