Fix for the fact that sometimes too much accordion items were added (#…

…1851)

* Fix for the fact that sometimes too much accordion items were added

* add contributor

Co-authored-by: AnthonyLedesma <anthonymledesma@gmail.com>

CONTRIBUTORS.md

@@ -46,3 +46,4 @@ This list is manually curated to include valuable contributions by volunteers th
 | @WunderBart       |                        |
 | @sirreal          | @jonsurrell            |
 | @kishanjasani     | @kishanjasani          |
+| @olafleur         |                        |

src/blocks/accordion/edit.js

@@ -113,7 +113,7 @@ class AccordionEdit extends Component {
 								<Button
 									label={ __( 'Add accordion item', 'coblocks' ) }
 									className="block-editor-button-block-appender"
-									onMouseUp={ handleEvent }
+									onMouseDown={ handleEvent }
 								>
 									<Icon icon={ plus } />
 								</Button>

src/blocks/accordion/test/accordion.cypress.js

@@ -48,8 +48,7 @@ describe( 'Block: Accordion', () => {
 	it( 'can add multiple accordion item blocks', () => {
 		cy.get( '[data-type="coblocks/accordion"]' ).click( 'top', { force: true } );
 
-		cy.get( '.coblocks-block-appender button' ).trigger( 'mouseup' );
-		// Using trigger here instead of click to more accurately represent behavior of event bubbling from user interaction within the editor.
+		cy.get( '.coblocks-block-appender button' ).click();
 
 		cy.get( '[data-type="coblocks/accordion"]' ).find( '[data-type="coblocks/accordion-item"]' ).should( 'have.length', 2 );