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

Limit attachment sizes #115

Closed
liliakai opened this issue Jan 12, 2015 · 1 comment
Closed

Limit attachment sizes #115

liliakai opened this issue Jan 12, 2015 · 1 comment
Milestone

Comments

@liliakai
Copy link
Contributor

There is currently no limit on attachment sizes or types. This may become problematic, particularly when sending to mobile clients where bandwidth and/or storage may be limited.

On the android side, @mcginty recommends downsizing large images with a target limit of 300KB. On the web, we may just want to handle this at the UX level. My sense is that web users are more used to size limits on file uploads...

@liliakai liliakai modified the milestones: 0.2, On the roadmap, MVP Feb 12, 2015
@liliakai liliakai modified the milestones: MVP, Real soon now Feb 27, 2015
@generalmanager
Copy link

Could we not instead offer to resize it locally before uploading it? Shouldn't be a huge job and would certainly improve the user experience over breaking the workflow. Many people don't even know how to resize a picture or are way too lazy to do it every time they want to send something over TS/Signal to a friend.
Or just inform users that the file is going to be resized automatically because it was bigger than 300KB? I mean there is nothing they can do about a fixed limit. And I suspect the average TS user won't bother to do it manually to be able to use the best possible resizing algorithm.

File size limits are probably going to grow in the future, so people will continue to resize pictures even it's not even necessary any more.

Edit: does this mean you changed your opinion: #132 ?

@liliakai liliakai closed this as completed Mar 3, 2015
jacobogonzalez pushed a commit to ra-app/desktop that referenced this issue Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants