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

(fix): Paste button disappearing while coping from nested blocks #4505

Merged
merged 5 commits into from
Mar 15, 2023

Conversation

dobri1408
Copy link
Contributor

screen-capture.1.webm

I discovered that if you try to copy from a block with other blocks in it, like EEA volto-accordion-block or EEA volto-columns-block and paste outside, the paste button disappears. This is due to the Plug component not rendering. I saw that setting the "dependencies" property, to selectedBlock, makes the Plug rerender. This can't be tested with a clean volto without addons, because there is not a pre-built block that has other blocks in it

@dobri1408 dobri1408 requested a review from avoinea March 12, 2023 09:30
@netlify
Copy link

netlify bot commented Mar 12, 2023

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit 9e3fd2b
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/6411640047494900089e9c3e

@cypress
Copy link

cypress bot commented Mar 12, 2023

Passing run #4372 ↗︎

0 485 20 0 Flakiness 0

Details:

Merge branch 'master' into paste-button-disappearing
Project: Volto Commit: 9e3fd2b653
Status: Passed Duration: 14:38 💡
Started: Mar 15, 2023 6:25 AM Ended: Mar 15, 2023 6:40 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@sneridagh sneridagh merged commit 45ecca3 into master Mar 15, 2023
@sneridagh sneridagh deleted the paste-button-disappearing branch March 15, 2023 10:26
sneridagh pushed a commit that referenced this pull request Mar 15, 2023
Co-authored-by: Tiberiu Ichim <tiberiuichim@users.noreply.github.com>
sneridagh added a commit that referenced this pull request Mar 15, 2023
…) (#4530)

Co-authored-by: dobri1408 <50819975+dobri1408@users.noreply.github.com>
Co-authored-by: Tiberiu Ichim <tiberiuichim@users.noreply.github.com>
erral pushed a commit that referenced this pull request Mar 18, 2023
Co-authored-by: Tiberiu Ichim <tiberiuichim@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants