Skip to content

Added new Open Both Folders button #102

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 1 commit into from
Jun 18, 2020

Conversation

inger147
Copy link
Contributor

  • Added new button next to duplicate image results (next to the Delete Both Images button), with tooltip text "Open both containing folders". Clicking this will open a separate windows explorer window for both of the displayed images;
  • Added new PNG image of two open folders;
  • Modified logic in Images.Get to enable other types of images to be loaded in addition to the default .IMG format.

Notes:

  • I don't speak Russian so the Russian tooltip text I added is from a Google translation.
  • I used a PNG image for the new button, because my graphics editing software (XnView) was exporting a .IMG file in a format that AntiDupl couldn't read properly (I got an OutOfMemoryException when I tried).

…Both Images button), with tooltip text "Open both containing folders". Clicking this will open a separate windows explorer window for both of the displayed images;

Added new PNG image of two open folders;
Modified logic in Images.Get to enable other types of images to be loaded in addition to the default .IMG format.
@ermig1979
Copy link
Owner

The folder which contains the image can be open from context menu of the image. Are you sure that opening of two folders is so often used operation that we need to create new button?

@inger147
Copy link
Contributor Author

In my experience at least, I found that I was opening both folders so often that I found it to be quite useful to have a button available that would open both at once. For large image collections, it saves time + clicks versus using the context menu on each image, plus it lessens the chance of clicking something unintended in the context menu, such as "add to ignored folders" which I found myself clicking accidentally on occasion.

@ermig1979
Copy link
Owner

Ok. I will merge your pull request.

@ermig1979 ermig1979 merged commit 847b9bc into ermig1979:master Jun 18, 2020
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.

2 participants