Skip to content
This repository has been archived by the owner on Dec 23, 2023. It is now read-only.

Image tool now functioning with new features on Strapi V4 #20

Merged
merged 21 commits into from
Mar 13, 2022
Merged

Image tool now functioning with new features on Strapi V4 #20

merged 21 commits into from
Mar 13, 2022

Conversation

jaskipper
Copy link
Contributor

@jaskipper jaskipper commented Feb 25, 2022

I now have the image tool working. I've changed some code and added some comparing this code with the following tutorial: https://docs.strapi.io/developer-docs/latest/guides/registering-a-field-in-admin.html#creating-the-wysiwyg.

…ary, and automatically shuts the media library window after saving.
This fixes an issue with the link and image tools not working with yarn/npm installed packages.
This feature adds in the width, height, size, mime, and the responsive image data to the image tool. It saves all of that information to the database and is returned so that the user can use that information in their frontend.

This addresses several open and closed issues:
- #6
- #16
- #22 (duplicate of closed 6)
@jaskipper jaskipper changed the title Image tool now functioning on Strapi V4 Image tool now functioning with new features on Strapi V4 Feb 26, 2022
@jaskipper
Copy link
Contributor Author

jaskipper commented Feb 26, 2022

Commit e07548c adds in width, height, size, mime and responsive data to the image tool. Commits f318525 and 5949de0 cleaned up the responses. Here is a screenshot of the returned image data after being turned into a json object.

Screen Shot 2022-02-26 at 6 32 58 AM

@jaskipper
Copy link
Contributor Author

@melishev - For the most part, I believe that I am done with commits to this. Please check 6223702, as this a complete refactoring of Wysiwyg > index.js. I've removed the usage of @buffetjs.

1f2d9a7 fixes issue #5

From everything that I can tell, with all of these changes, EditorJS is working perfectly in v4 with the following exceptions:

  1. Link tool doesn't work properly in strapi develop --watch-admin mode
  2. Link tool doesn't work properly when @strapi/mailgun-js is installed.

@jaskipper jaskipper mentioned this pull request Feb 28, 2022
@melishev melishev self-requested a review March 11, 2022 19:56
admin/src/components/Wysiwyg/wrapper.js Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
admin/src/components/Wysiwyg/wrapper.js Outdated Show resolved Hide resolved
admin/src/components/Wysiwyg/wrapper.js Outdated Show resolved Hide resolved
@jaskipper
Copy link
Contributor Author

@melishev I believe that all changes that you requested have been completed.

@melishev melishev merged commit 01e5a04 into melishev:beta Mar 13, 2022
@melishev
Copy link
Owner

🎉 This PR is included in version 2.0.0-beta.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@jaskipper jaskipper deleted the beta-v4 branch March 16, 2022 17:23
@melishev
Copy link
Owner

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.