-
-
Notifications
You must be signed in to change notification settings - Fork 11
feat: ability to add alt text to images #421
base: main
Are you sure you want to change the base?
Conversation
So currently there are a view problems:
CC: @sidemt and @ojeytonwilliams |
@Sembauke Thank you for catching this, I didn't know Strapi saves I think we can ignore the value in Ghost doesn't have a separate attribute for alt text either so it shouldn't be a problem (it has feature_image_alt on the latest version, but not on v3, the version we are using.) The modal to add/edit image is looking awesome 👍 |
One more thing that need to be solved : There is one other issue in the editor but I will have to do more research before I can comment on that. I have removed the logic that uploads the alternative text and captions to the database. Hopefully we will not need to re-add this in the future. This makes things a lot easier. There won't be any need to auto save as well! CC: @sidemt |
I believe this is OK :)
Is it possible to disable the save button until the upload completes? If it's complicated, we can leave it to another PR. |
apps/frontend/src/components/tip-tap-extensions/image-extension.jsx
Outdated
Show resolved
Hide resolved
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.
Overall, it's looking good!
Currently, the preview doesn't seem to render <figure>
and <figcaption>
elements even if the image has a caption. It is rendered properly in the editor. Could you make it appear in the preview too?
Edit: Oliver's PR #427 may fix this issue.
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.
LGTM 👍 Sorry for keeping you waiting!
@Sembauke Sorry, do you mind resolving the conflicts again? I tried doing it myself, but my attempt seems to erase some code in the main branch (the line using |
When adding a new image:

When editing a new image:

Closes #392