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

Add field for image description #751

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add field for image description #751

wants to merge 1 commit into from

Conversation

jcoyne
Copy link
Collaborator

@jcoyne jcoyne commented Nov 20, 2024

Why was this change made? 🤔

This helps support accessibility and AI feature extraction for indexing

This helps support accessibility and AI feature extraction for indexing
@edsu
Copy link
Contributor

edsu commented Nov 20, 2024

Is there a reason why this isn't added a level up next to label?

@jcoyne
Copy link
Collaborator Author

jcoyne commented Nov 20, 2024

@edsu because it's only on images and it's for presentation.

@justinlittman
Copy link
Contributor

How is this different than the file label field?

@andrewjbtw
Copy link

  1. Won't alt-text be needed on other types of content?
  2. How will someone edit this field? I think it's a good idea to add fields that users will need to edit only in conjunction with some way for a user to edit them.

@jcoyne
Copy link
Collaborator Author

jcoyne commented Nov 20, 2024

How is this different than the file label field?

The label field is user provided text, but it is not intended to be used as alt text.

Won't alt-text be needed on other types of content?

The alt attribute is only supported on images (also <area> and <input> but we don't use these to present SDR objects)

How will someone edit this field? I think it's a good idea to add fields that users will need to edit only in conjunction with some way for a user to edit them.

Currently undetermined.

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.

4 participants