-
-
Notifications
You must be signed in to change notification settings - Fork 859
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
Increase alt text size #4724
Increase alt text size #4724
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is good. I'm gonna wait for some of the other maintainers' opinions on this before merging.
For reference here is a text with 300 characters:
And here 1500:
That seems way too long to describe an image, and I bet many apps/frontends wont have space to display it properly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Useful for images that contain text
Maybe 500 would be better?
Perhaps 750?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I forget why I chose 300, but ya I agree it makes sense especially for image posts with a LOT of text to have this larger. Maybe not 10k
like the post body, but this seems fine for now.
300 characters isn't enough, this increases it to be the same as Mastodon.