ImageProcessing-ElectronicPublications / photoquick

Light-weight image viewer with crop,resize,collage, photogrid and filters
GNU General Public License v3.0
4 stars 0 forks source link

ReOpen? #6

Closed zvezdochiot closed 4 years ago

zvezdochiot commented 4 years ago

:question: Maybe add "ReLoad" current image?

ksharindam commented 4 years ago

As it is a quick opening image viewer. One can just close window and double click to reopen image. But if you want to undo changes, an undo option can be added.

On Fri, 19 Jun, 2020, 10:50 AM звездочёт, notifications@github.com wrote:

❓ Maybe add "ReLoad" current image?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ImageProcessing-ElectronicPublications/photoquick/issues/6, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEKWXPRNBQPNOE2WYVOQ6QDRXLYRFANCNFSM4OCM4WYQ .

ksharindam commented 4 years ago

Most of the image viewers dont require reload button.

On Fri, 19 Jun, 2020, 10:54 AM Arindam Chaudhuri, ksharindam@gmail.com wrote:

As it is a quick opening image viewer. One can just close window and double click to reopen image. But if you want to undo changes, an undo option can be added.

On Fri, 19 Jun, 2020, 10:50 AM звездочёт, notifications@github.com wrote:

❓ Maybe add "ReLoad" current image?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ImageProcessing-ElectronicPublications/photoquick/issues/6, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEKWXPRNBQPNOE2WYVOQ6QDRXLYRFANCNFSM4OCM4WYQ .

zvezdochiot commented 4 years ago

@ksharindam say:

an undo option can be added.

:warning: Too much trouble and hardly a good result. ReOpen is simpler and more reliable.

ksharindam commented 4 years ago

what about adding a keyboard shortcut, such as press "R" to reload?

zvezdochiot commented 4 years ago

Hi @ksharindam .

This is a pretty good idea.

zvezdochiot commented 4 years ago

Hi @ksharindam .

:warning: Reload not work of " default image".

But this is not critical at all.

ksharindam commented 4 years ago

Yes, the default image is just for demo. nobody will work with the default image.