Nullgram icon indicating copy to clipboard operation
Nullgram copied to clipboard

feat: Allow sending pictures up to 2560px

Open megapro17 opened this issue 11 months ago • 5 comments

Tittle Here

Better quality photo

Description

https://github.com/DrKLO/Telegram/pull/1790

For reference: https://github.com/Nekogram/Nekogram/commit/fad3e1a9528e7b229dc1159b59efa83d57709d27

https://github.com/OwlGramDev/OwlGram/commit/976a1ce4bddb38f09711844d4afe0ba8b132afcf

This setting has worked perfectly for over 2 years, there's no point in making an extra button in the settings, it works without a problem. For those who have it enabled, it gives a huge advantage in image quality. Telegram desktop does display photo with higher resolution by default

megapro17 avatar Mar 20 '24 17:03 megapro17

please add the switcher and i guess you missed changes in SendMessagesHelper.java

qwq233 avatar Mar 31 '24 06:03 qwq233

Can you tell me why do you want a switcher? You cannot disable in official telegram web. It doesn't cause any problems, there's no reason to ever disable it, you will only lose quality

megapro17 avatar Apr 03 '24 16:04 megapro17

Can you tell me why do you want a switcher? You cannot disable in official telegram web. It doesn't cause any problems, there's no reason to ever disable it, you will only lose quality

https://github.com/DrKLO/Telegram/pull/1790#issuecomment-1996889948

i think his concerns kinda make sense

qwq233 avatar Apr 03 '24 16:04 qwq233

How it can be verified? Android phones have a lot of memory nowadays

megapro17 avatar Apr 03 '24 16:04 megapro17

image well...not really

qwq233 avatar Apr 03 '24 16:04 qwq233

added

qwq233 avatar Aug 14 '24 18:08 qwq233