Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Allow sending pictures up to 2560px #222

Closed

Conversation

megapro17
Copy link
Contributor

@megapro17 megapro17 commented Mar 20, 2024

Tittle Here

Better quality photo

Description

DrKLO/Telegram#1790

For reference:
Nekogram/Nekogram@fad3e1a

OwlGramDev/OwlGram@976a1ce

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

@qwq233
Copy link
Owner

qwq233 commented Mar 31, 2024

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

@megapro17
Copy link
Contributor Author

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

@qwq233
Copy link
Owner

qwq233 commented Apr 3, 2024

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

DrKLO/Telegram#1790 (comment)

i think his concerns kinda make sense

@megapro17
Copy link
Contributor Author

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

@qwq233
Copy link
Owner

qwq233 commented Apr 3, 2024

image
well...not really

@qwq233 qwq233 force-pushed the master branch 3 times, most recently from 035d1f0 to c7e941f Compare July 11, 2024 09:20
@qwq233
Copy link
Owner

qwq233 commented Aug 14, 2024

added

@qwq233 qwq233 closed this Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants