From 32e4564e148d7128f5dbdda15e3c0f33b9432f5c Mon Sep 17 00:00:00 2001 From: Denys Vuika Date: Mon, 16 Jan 2023 15:57:47 +0000 Subject: [PATCH] Revert "[ACS-4051] Copy to clipboard button is now accessible through the keyboard enter earlier which was only accessible through mouse click (#8119)" (#8159) This reverts commit 067d4e9369335268094b2e05c51dcd4e343e9f1c. --- .../lib/clipboard/clipboard.directive.spec.ts | 17 ----------------- .../src/lib/clipboard/clipboard.directive.ts | 6 ------ 2 files changed, 23 deletions(-) diff --git a/lib/core/src/lib/clipboard/clipboard.directive.spec.ts b/lib/core/src/lib/clipboard/clipboard.directive.spec.ts index d19d6951cc..17f84ee1a3 100644 --- a/lib/core/src/lib/clipboard/clipboard.directive.spec.ts +++ b/lib/core/src/lib/clipboard/clipboard.directive.spec.ts @@ -64,14 +64,6 @@ describe('ClipboardDirective', () => { expect(clipboardService.copyToClipboard).toHaveBeenCalled(); }); - - it('should notify copy target value on mouse enter event', () => { - spyOn(clipboardService, 'copyToClipboard'); - fixture.nativeElement.querySelector('input').value = 'some value'; - window.dispatchEvent(new KeyboardEvent('keydown', {code: 'Enter', key: 'Enter'})); - - expect(clipboardService.copyToClipboard).toHaveBeenCalled(); - }); }); describe('CopyClipboardDirective', () => { @@ -136,15 +128,6 @@ describe('CopyClipboardDirective', () => { expect(navigator.clipboard.writeText).toHaveBeenCalledWith('text to copy'); })); - it('should copy the content of element on mouse enter', fakeAsync(() => { - fixture.detectChanges(); - spyOn(navigator.clipboard, 'writeText'); - window.dispatchEvent(new KeyboardEvent('keydown', {code: 'Enter', key: 'Enter'})); - tick(); - fixture.detectChanges(); - expect(navigator.clipboard.writeText).toHaveBeenCalledWith('text to copy'); - })); - it('should not copy the content of element when click it', fakeAsync(() => { const spanHTMLElement = element.querySelector('span'); fixture.detectChanges(); diff --git a/lib/core/src/lib/clipboard/clipboard.directive.ts b/lib/core/src/lib/clipboard/clipboard.directive.ts index 95c0c5da26..ee898cfb4b 100644 --- a/lib/core/src/lib/clipboard/clipboard.directive.ts +++ b/lib/core/src/lib/clipboard/clipboard.directive.ts @@ -46,12 +46,6 @@ export class ClipboardDirective { event.stopPropagation(); this.copyToClipboard(); } - @HostListener('window:keydown.enter', ['$event']) - handleKeyDown(event: KeyboardEvent){ - event.preventDefault(); - event.stopPropagation(); - this.copyToClipboard(); - } @HostListener('mouseenter') showTooltip() {