Allow sending pictures up to 2560px

[gobonja]

• 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

[Kris35hoog]

Ok

[gobonja]

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

[dkaraush]

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...

[QuAzI]

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

[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...

As far as I can see, this functionality is already implemented:

Telegram/TMessagesProj/src/main/java/org/telegram/messenger/MessageObject.java

Lines 8531 to 8536 in d494ea8

	if (photoWidth > AndroidUtilities.getPhotoSize()) {
	photoWidth = AndroidUtilities.getPhotoSize();
	}
	if (photoHeight > AndroidUtilities.getPhotoSize()) {
	photoHeight = AndroidUtilities.getPhotoSize();
	}

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.

[fhnb16]

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

[vovmomyt]

Like