Increase maximum zoom level for image viewer

[throwawaytime]

It would be great to be able to zoom in significantly more than the current limitation. Landscape or portrait, it is currently fairly limited.

Some Lemmy communities are often doing things like trying to read license plate or phone numbers from dash cam footage where you really need to zoom in far, even if the resolution isn’t great.

[micahmo]

@CTalvio Would you know offhand if this is a function of ExtendedImage? We might need to investigate this further.

[CTalvio]

Yes. The max zoom is currently 4x display size.

It would be good to set this number based on the resolution of the image.

[Niranjan-Dorage]

is this issue still open ?
@throwawaytime @CTalvio

[CTalvio]

Yes

[Niranjan-Dorage]

@CTalvio enhancement is done
but what i have to do here in pr description ?

and screenshots are necessary ?

[micahmo]

what i have to do here in pr description ?

For the first one, if you could add a line item in CHANGELOG.md under Unreleased > Changed saying what you changed and adding your username as a contributor, that would help us generate the changelog. You can see the other examples there. However, if you can't, that's ok.

For the second and third ones, most likely you didn't add any UI requiring translation or semantic labels (for screen readers), so don't worry about that.

and screenshots are necessary ?

No, just appreciated if there's any new UI work! See the comment in the PR template.

<!-- This section is optional but highly recommended to show off your changes! -->

[Niranjan-Dorage]

@CTalvio i pulled code before pull request so the code is latest you can merge it

[CTalvio]

It appears this is solved as of #859, closing,