tdesktop icon indicating copy to clipboard operation
tdesktop copied to clipboard

[Feature request] Ability to disable integrated photo viewer

Open matejdro opened this issue 10 years ago • 24 comments

When you receive images from chat (either sent as image or as file), they are displayed in integrated photo viewer.

This viewer lacks zoom and other functionality that is present in other photo viewers. So if I want this functionality I must right click on the image, select "View in folder" and then open file manually from file browser.

There should be an option to disable integrated viewer and open my default image viewer when I click on the image in chat.

matejdro avatar Sep 21 '15 10:09 matejdro

Zoom with ctrl and +/-

auchri avatar Sep 21 '15 10:09 auchri

Oh wow I didn't even tried it. Why wouldn't you support mouse scroll wheel zoom?

matejdro avatar Sep 21 '15 10:09 matejdro

Agreed. Ctrl +/- works, but mouse-wheel zoom is much more intuitive, especially when people usually open images in the viewer with a mouse click, not keyboard input. Since we use the mouse to open and close images, should use the mouse for most image interaction.

joshsleeper avatar Nov 17 '15 03:11 joshsleeper

you can do mouse wheel zoom now.

Aokromes avatar Nov 08 '16 10:11 Aokromes

now we have https://github.com/telegramdesktop/tdesktop/commit/54c409d967f9e823afdb8d9f94d775074c284aa0 maybe it can be improved to include photo viewers / audio players, etc without being hidden setting.

Aokromes avatar Jan 27 '20 16:01 Aokromes

now we have 54c409d maybe it can be improved to include photo viewers / audio players, etc without being hidden setting.

This simple PR works only with downloaded (on disk as video file) video. In theory same must go with photos, but in practice in TD there are no downloaded (in same therms as video) photo :( This button: image

always download photo and save it with new name. So, with photo it's of course doable, but not so simple as with video...

kirsan31 avatar Jan 27 '20 16:01 kirsan31

Hey there!

This issue will be automatically closed in 7 days if there would be no activity. We therefore assume that the user has lost interest or resolved the problem on their own.

Don't worry though; if this is an error, let us know with a comment and we'll be happy to reopen the issue.

Thanks!

stale[bot] avatar Oct 24 '20 02:10 stale[bot]

We are still interested in this.

matejdro avatar Oct 24 '20 06:10 matejdro

Hey there!

This issue was inactive for a long time and will be automatically closed in 30 days if there isn't any further activity. We therefore assume that the user has lost interest or resolved the problem on their own.

Don't worry though; if this is an error, let us know with a comment and we'll be happy to reopen the issue.

Thanks!

stale[bot] avatar Apr 22 '21 06:04 stale[bot]

Still actual.

kirsan31 avatar Apr 22 '21 06:04 kirsan31

Zoom works but there is no way to scroll the image (e.g. to see a detail in a corner) as far as I can tell. Saving the picture and opening it in an external viewer calls for a bunch of clicks and context-switching.

qsmodo avatar Sep 18 '21 18:09 qsmodo

Other file types have an "Open with" and "Show in folder" options in the context menu, even images which have been sent as a file do. It seems like a minor thing to add these options to the context menu for images as well. Would be great, if this could be fixed! :smiley:

Photon89 avatar Oct 24 '21 07:10 Photon89

@qsmodo You can zoom and then drag the image by mouse, looking to all parts of it with zoom in.

john-preston avatar Oct 24 '21 08:10 john-preston

Just for completeness, in my case I need to view the image and use the chat simultaneously which is not possible because the integrated image viewer blocks the chat.

Photon89 avatar Oct 24 '21 09:10 Photon89

Just for completeness, in my case I need to view the image and use the chat simultaneously which is not possible because the integrated image viewer blocks the chat.

This is a bit different issue: https://github.com/telegramdesktop/tdesktop/issues/1710

kirsan31 avatar Oct 24 '21 09:10 kirsan31

@Photon89 It's not that simple, because when photos are downloaded they're not saved as a regular files on disk (so nothing can be opened in the external viewer). They're saved in an encrypted local storage only. And only after manually saving them to system Downloads they're materialized as real jpg files on disk.

john-preston avatar Oct 24 '21 09:10 john-preston

@kirsan31 Not sure, even if the viewer wouldn't go to full screen, which is the issue #1710, it most probably would still block the chat portion of the Telegram window. So using the chat and viewing the image simultaneously would still be impossible.

@john-preston I see, thanks for the explanation! But I don't quite understand: This seems to be some security related feature, right? So the sender of the image chooses that the image should not be saved on the receiver's disk but only in an encrypted local storage. But this feature doesn't really work, if the receiver can still select to save the file on disk.

What would make more sense to me, would be: Give the sender two options: "Send encrypted" and "Send without encryption". For the first option, remove the possibility to save the screenshot on disk completely; for the second option save the file on disk without encryption (so it can be opened with an external viewer).

Photon89 avatar Oct 24 '21 09:10 Photon89

IMHO, the best resolution here is to implement https://github.com/telegramdesktop/tdesktop/issues/1710 with ability to use chat in ||...

kirsan31 avatar Oct 24 '21 09:10 kirsan31

Well, an option to open the viewer in a new window, like the call dialogue, would mostly solve it for me (just would need to get used to the controls for zoom and shift in the integrated viewer). :smiley:

Photon89 avatar Oct 24 '21 09:10 Photon89

It's not that simple, because when photos are downloaded they're not saved as a regular files on disk (so nothing can be opened in the external viewer). They're saved in an encrypted local storage only. And only after manually saving them to system Downloads they're materialized as real jpg files on disk.

I don't really see why would this be such an issue. When "Do not use internal photo viewer" option is enabled, Telegram would just use existing download code to materialize jpg immediately into OS's tmp folder and use also existing code to open that file with default application.

matejdro avatar Oct 24 '21 09:10 matejdro

As far as I understand, the intention is that an encrypted image should not be decrypted (to protect sensitive information maybe) but this is not really given if the image can still be decrypted and saved on disk using the context menu...

Photon89 avatar Oct 24 '21 09:10 Photon89

Right, but by enabling "Do not use internal photo viewer" I'm giving telegram explicit permission to decrypt my images (you could add a warning next to the setting I guess, but I don't really see this as a big security issue).

matejdro avatar Oct 24 '21 10:10 matejdro

What @john-preston meant (I guess) is that it's implemented in a way that it's not saved to disk and it's hard to change that (read like "too much effort for such a low priority issue"), not that it's some security feature that require some question to decrypt images, it's not a secret chat or something, you can still save the file by a button in the viewer. Also, if a feature requires an option, it's a blocker in tdesktop. Adding options is not something @john-preston likes.

ilya-fedin avatar Oct 31 '21 00:10 ilya-fedin

The same request

aljustiet avatar Sep 19 '24 11:09 aljustiet