You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey again! It sounds like this extension doesn't support inline mode yet, which makes sense given how new it is. I just wanted to flag that the alignment buttons also don't work when inline is enabled on the extension.
I ended up forking the extension to work around this, using the text-align extension's commands for alignment instead, as that works with the paragraph node when the image is inserted inline.
To do this, I ended up swapping out the position controller event handlers to be:
An easier change to support this, if you're interested, would be to expose the paintPositionContoller method as an extension option that can be overridden. If that is a friendly change to you, I'd be happy to make a contribution!
The text was updated successfully, but these errors were encountered:
First, thank you for your interest in my extension! I've reviewed your issues and tested your code. Your solution is promising, but for this inline case, we need to consider potential side effects.
We are planning to refactor the entire logic to better support this inline feature. I apologize for any inconvenience and let you know when the feature is complete.
How exactly can i overwrite the default behavior? i have an vanilla javascript code base and all i do is adding ImageResize to the tiptap extension array right now. I couldn't figure out how you used your own events for the buttons.
Hey again! It sounds like this extension doesn't support inline mode yet, which makes sense given how new it is. I just wanted to flag that the alignment buttons also don't work when
inline
is enabled on the extension.I ended up forking the extension to work around this, using the text-align extension's commands for alignment instead, as that works with the paragraph node when the image is inserted inline.
To do this, I ended up swapping out the position controller event handlers to be:
An easier change to support this, if you're interested, would be to expose the
paintPositionContoller
method as an extension option that can be overridden. If that is a friendly change to you, I'd be happy to make a contribution!The text was updated successfully, but these errors were encountered: