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

[ui] ImageGallery: Add "Remove All Images" menu to clear all images #2221

Merged
merged 2 commits into from
Oct 11, 2023

Conversation

cbentejac
Copy link
Contributor

Description

In addition to the existing "Remove" menu action in the Image Gallery that deletes the currently selected image, this PR adds a "Remove All Images" menu action that performs the same action as "Clear Images" from the "File" menu.

The Del key was used as a shortcut to remove the currently selected image when the Image Gallery has the focus, Shift+Del can now also be used to remove all the images at once.

In addition to the existing "Remove" menu action in the Image Gallery that
deletes the currently selected image, add a "Remove All Images" menu
action that performs the same action as "Clear Images" from the "File"
menu.

The "Del" key was used as a shortcut to remove the currently selected
image when the Image Gallery has the focus, "Shift+Del" can now also be
used to remove all the images at once.
@cbentejac cbentejac added the UI label Oct 11, 2023
@cbentejac cbentejac added this to the Meshroom 2023.3.0 milestone Oct 11, 2023
@cbentejac cbentejac self-assigned this Oct 11, 2023
Update the names of the commands that remove images in the "File"
menu to match with those in the Image Gallery menu.

"Clear Images" becomes "Remove All Images" and "Clear All Images" becomes
"Remove Images From All CameraInit Nodes".
@fabiencastan fabiencastan merged commit 5ccbfe4 into develop Oct 11, 2023
4 checks passed
@fabiencastan fabiencastan deleted the dev/clearImagesGallery branch October 11, 2023 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants