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

Editing a images alignment in TinyMCE duplicates the image #3910

Closed
pbauer opened this issue Feb 26, 2024 · 6 comments · Fixed by plone/mockup#1369
Closed

Editing a images alignment in TinyMCE duplicates the image #3910

pbauer opened this issue Feb 26, 2024 · 6 comments · Fixed by plone/mockup#1369

Comments

@pbauer
Copy link
Sponsor Member

pbauer commented Feb 26, 2024

Tested in 6.0.9, 6.0 coredev and 6.1 coredev.
For me it only happens in Firefox, not in Chrome

To reproduce:

  • Add a image to a page in TinyMCE and select Align Left
  • Edit the same image (select it with a click and click "Insert Image")
  • Change Align to Right
  • Save
  • You now have two images (one right, one left)

duplicate

@pbauer
Copy link
Sponsor Member Author

pbauer commented Feb 26, 2024

Same issue: Selecting the image via click and pressing "delete" does not remove the image.

Both problems do not happen when selecting the img-tag in status bar before deleting or changing something:
tag

My guess is that the mage-plugin somehow does not select the correct element upon click.

@pbauer
Copy link
Sponsor Member Author

pbauer commented Feb 26, 2024

@petschki maybe this is also up your alley :)

@petschki
Copy link
Member

I remember this behavior some time ago. I think @mauritsvanrees had some similary issues. Thought we fixed that meanwhile. I try to figure out whats going on.

@petschki
Copy link
Member

I fixed an issue here plone/mockup@856b519 but this was there for some reason ;)

@petschki
Copy link
Member

Fixed in

plone.staticresources==2.1.12 (Plone 6.0)
plone.staticresources==2.2.0a5 (Plone 6.1)

@mauritsvanrees
Copy link
Sponsor Member

My earlier report with the same problem was issue #3800.
I confirm this is fixed with the new version. Thanks!
I will include the new version in Plone 6.0.10 today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants