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

Fixed creation of both a slate block and a (proper) image block on pasting image directly from clipboard #5701

Closed
wants to merge 5 commits into from

Conversation

AnshulKahar2729
Copy link
Member

@AnshulKahar2729 AnshulKahar2729 commented Jan 26, 2024

Fixes #5697

Description : Earlier on the pasting the images in the blocks, rendering two images on the same time.

…sting an image directly from the clipboard in a slate block
Copy link

netlify bot commented Jan 26, 2024

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit 25b965b
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/65b38cb8c470fd00080b32eb

Copy link

netlify bot commented Jan 26, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 25b965b
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/65b38cb8f7fe8a00081a6e0b


return (
<span {...attributes} style={{ display: 'inline-block' }}>
{children}
<img alt="" src={element.url} style={style} />
Copy link
Member

@sneridagh sneridagh Feb 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AnshulKahar2729 We can't do this. This is not the way to fix this issue. We have to dig deeper and find why it's emitting the slate image (which we don't want) and make that it does not emit it. By removing the markup for the slate image is not an acceptable "fix".

Do you agree with me, right?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I got the problem, we need to fix the function which creates extra block rather than removing it manually.

@saloni0419
Copy link

is this issue still open #5701 @sneridagh

@stevepiercy
Copy link
Collaborator

@saloni0419 please read and follow Contributing to Plone and First-time contributors, especially Things not to do.

Using your super power of observation, you may see that the green "Open" icon reflects its status, and that by virtue of its URL and the "Pull requests" tab being selected, this is a pull request, not an issue. See GitHub docs.

@ichim-david
Copy link
Sponsor Member

@AnshulKahar2729 I am closing this pull request, as Victor mentioned this is not as easy as removing the image insertion from Slate. I suggest you study the problem more deeply and raise a pull request only if you've solved it and your work passes the code quality and testing commands.

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