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

Clarify that photos attached to a note are resized #2538

Closed
Helium314 opened this issue Jan 30, 2021 · 5 comments
Closed

Clarify that photos attached to a note are resized #2538

Helium314 opened this issue Jan 30, 2021 · 5 comments
Labels
feedback required more info is needed, issue will be likely closed if it is not provided

Comments

@Helium314
Copy link
Collaborator

Photos attached to notes are resized to max. 1280x1280. This is understandable, but often reduces readability of text.

Since most recent phones have high resolution cameras where text on a small part of an image is still readable, people might be uploading photos where text would be readable in original size, but no after automatic resizing done by SC.

I suggest that the resizing is mentioned when attaching photos to avoid unexpected problems when trying to read text from notes with attached photos.

@westnordost
Copy link
Member

There is soo much text already in the note input form

@westnordost westnordost added the feedback required more info is needed, issue will be likely closed if it is not provided label Jan 30, 2021
@Helium314
Copy link
Collaborator Author

I definitely agree, but sometimes I notice people uploading photos with text that is unreadable because of resizing (also did it myself once or twice)

Once a photo is added, the 'adding a photo will...' text disappears. Here something like 'photo(s) will be resized to 1280 pixels' could be shown.

@westnordost
Copy link
Member

it disappears to make space for the attached photos

@mnalis
Copy link
Member

mnalis commented Jan 30, 2021

Also see related #2497 (comment)

@Helium314
Copy link
Collaborator Author

Right, implementing #2497 should solve this... I guess it would be a better solution anyway, so I'll close this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feedback required more info is needed, issue will be likely closed if it is not provided
Projects
None yet
Development

No branches or pull requests

3 participants