volto icon indicating copy to clipboard operation
volto copied to clipboard

onFocus function to handle block selection

Open Wagner3UB opened this issue 2 years ago • 7 comments

Having focus on the block now is equivalent to clicking on it. Now the block becomes active when focused

To solve this issue: https://github.com/plone/volto/issues/5204

https://github.com/plone/volto/assets/60133113/5410aa64-c4e9-42b0-bb09-190704477322

Wagner3UB avatar Oct 04 '23 14:10 Wagner3UB

Deploy Preview for volto canceled.

Name Link
Latest commit 7b635fdf156db4d4629199eebeb72cbea41a5419
Latest deploy log https://app.netlify.com/sites/volto/deploys/65c1de27a78b7e0009e583b7

netlify[bot] avatar Oct 04 '23 14:10 netlify[bot]

@Wagner3UB @giuliaghisini did you have tested this thoroughly in your projects? No side effects? Also in other "bigger" blocks.

sneridagh avatar Nov 04 '23 08:11 sneridagh

@sneridagh @giuliaghisini I just ran some tests on our product, and the only behavior I observed was the expected one, specifically the activation of the blocks, I did not find side effects.

Wagner3UB avatar Nov 07 '23 11:11 Wagner3UB

@sneridagh I'm having an issue with a cypress test on the "Blocks-copypaste.js" component. Looks like the test misses "#toolbar-delete-blocks". I tried to find this element on the page but, nothing.

To be honest, I don't remember a toolbar for canceling blocks, but I might be missing something.

Wagner3UB avatar Nov 10 '23 10:11 Wagner3UB

"#toolbar-delete-blocks

@Wagner3UB the delete button is visible when you have multiple blocks selected, have a look at this screenshot toolbar-delete-blocks

ichim-david avatar Jan 11 '24 12:01 ichim-david

Deploy Preview for plone-components canceled.

Name Link
Latest commit 4fc3fe3934cab0baf530fb775b918e3d418bd824
Latest deploy log https://app.netlify.com/sites/plone-components/deploys/669cb3d0766976000894e8e2

netlify[bot] avatar Jan 11 '24 14:01 netlify[bot]

@sneridagh @ichim-david Piero and I ran some tests, and locally, we didn't find any issues. The Cypress tests didn't show us any problems. Besides that, I did not change the test 'Blocks-copypaste.js' or the component 'BlocksToolbar.jsx'.

We've run out of possibilities.

cc:/ @pnicolli

Wagner3UB avatar Jan 12 '24 11:01 Wagner3UB