-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
🎉 Automatic alt text for cloudflare images #4261
Conversation
border: 1px solid #ccc; | ||
width: 100%; | ||
&.ImageIndexPage__alt-text-editor--should-autosize { | ||
field-sizing: content; |
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.
This is only supported in chrome as of now, but I think that's the browser most of the authors use.
Quick links (staging server):
Login:
SVG tester:Number of differences (default views): 0 ✅ Edited: 2024-12-06 00:13:41 UTC |
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.
Cool feature! 🙂
I'm not sure how important it is to fix/improve the state synchronization with useEffect
.
console.error( | ||
`Error fetching GPT alt text for image ${imageId}`, | ||
error | ||
) |
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.
Out of scope of this PR, but it's unfortunate if we currently have to effectively log the error twice to surface/log it properly. This shouldn't be necessary, in general.
automatic.alt.text.mp4
Changes: