Telegram icon indicating copy to clipboard operation
Telegram copied to clipboard

Allow sending pictures up to 2560px

Open gobonja opened this issue 2 years ago • 6 comments

  • This PR will allow sending pictures up to 2560px.

  • Telegram API already allows to send photos with that size.

  • Telegram Desktop and some android client forks (e.g. Nekogram) also supports this.

  • There is also bug report for this: https://bugs.telegram.org/c/15370

gobonja avatar Nov 04 '23 22:11 gobonja

Ok

Kris35hoog avatar Nov 05 '23 17:11 Kris35hoog

@dkaraush @DrKLO @xaxtix any news about this PR?

gobonja avatar Jan 23 '24 10:01 gobonja

This increases the initial texture buffer allocation by significant amount in painting screen. You might want to apply this increase only for photos that are already that size or larger. Considering reports of OutOfMemoryException, it would be also great to do that only for HIGH and AVERAGE devices...

dkaraush avatar Mar 14 '24 08:03 dkaraush

Waiting for this feature as currently photos quality in TG is terrible and TG gallery works ugly with photos which uploaded as files

QuAzI avatar May 24 '24 11:05 QuAzI

This increases the initial texture buffer allocation by significant amount in painting screen. You might want to apply this increase only for photos that are already that size or larger. Considering reports of OutOfMemoryException, it would be also great to do that only for HIGH and AVERAGE devices...

As far as I can see, this functionality is already implemented: https://github.com/DrKLO/Telegram/blob/d494ea8cb59f6ff1af5faca9f5f651abcec618c0/TMessagesProj/src/main/java/org/telegram/messenger/MessageObject.java#L8531-L8536

It will use actual image size if size is not bigger then 2560px, if it is bigger it will downscale image to 2560px. So it will never increase image size, just allow images up to 2560px.

gobonja avatar May 24 '24 12:05 gobonja

Still waiting for this feature and suffering with overcompressed images..

fhnb16 avatar Jul 17 '24 10:07 fhnb16