Zoom in on specific pictures does not work at all or not well

[chagai95]

Description

When the length is much longer than the width for example or maybe even big pictures and in general you can only zoom in to a specific size and then it will zoom out again making the GUI bigger but not the picture

Pictures

Steps to reproduce

try these pictures:

Logs being sent: no

Version information

• Platform: web (in-browser) or desktop? web

For the web app:

• Browser: Chrome, Safari, Firefox? which version? Chrome
• OS: Windows, macOS, Ubuntu, Arch Linux, etc? Windows
• URL: develop.element.io / app.element.io / somewhere else? If a private server, what version of Element Web? app.element.io

[jryans]

That does not seem like a picture of Element Web...

[t3chguy]

@jryans those are example images to reproduce the issue

try these pictures:

[SimonBrandner]

@chagai95, I am not sure I understand what exactly is the issue about... Are the problems solved by the new image view? If not, could you perhaps try to explain this in more detail/add screenshots/add a recording?

[SimonBrandner]

I will close this as there doesn't seem to be a response from the author or anyone else. Feel free to reopen or ping me if this is still a problem.

[chagai95]

Sorry I missed this, I guess this could turn into a feature request to add more zoom capability because it still seems limited, also the intuitive CRTL mouse scroll is not regarded and only the mouse scroll is mapped to zooming, any specific reason for that? @SimonBrandner Thanks so much for pinging and the zoom is amazing otherwise!

[SimonBrandner]

Also the intuitive CRTL mouse scroll is not regarded and only the mouse scroll is mapped to zooming, any specific reason for that?

It works for me both with and without pressing Ctrl 🤔 If this isn't the case for you, please open an issue and please include browser info.

The zoom behavior originates from #17049. There has been a suggestion to change this (#17114) but that quite conflicts with the expected behavior in the original issue. That is why I suggested this.

Feel free to comment in the relevant places