Skip to content

Commit adc6ad6

Browse files
committed
Bug on double click fixed and esc fixed.
1 parent f1dec24 commit adc6ad6

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

src/PreviewSnippet.tsx

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,11 +59,11 @@ export class Preview<T> extends Widget {
5959

6060
if (Preview.tracker.size > 0) {
6161
const previous = Preview.tracker.currentWidget;
62-
document.getElementsByClassName("drag-hover")[previous._id].classList.remove("drag-hover-clicked");
63-
document.getElementsByClassName("elyra-codeSnippet-item")[previous._id].classList.remove("elyra-codeSnippet-item-clicked");
62+
if (previous._title != this._title) {
63+
document.getElementsByClassName("drag-hover")[previous._id].classList.remove("drag-hover-clicked");
64+
document.getElementsByClassName("elyra-codeSnippet-item")[previous._id].classList.remove("elyra-codeSnippet-item-clicked");
65+
}
6466
if (previous._title === this._title) {
65-
document.getElementsByClassName("drag-hover")[this._id].classList.remove("drag-hover-clicked");
66-
document.getElementsByClassName("elyra-codeSnippet-item")[this._id].classList.remove("elyra-codeSnippet-item-clicked");
6767
if (previous.node.classList.contains('inactive')) {
6868
previous.node.classList.remove('inactive');
6969
this.ready = false;
@@ -160,6 +160,8 @@ export class Preview<T> extends Widget {
160160
// Check for escape key
161161
switch (event.keyCode) {
162162
case 27: // Escape.
163+
document.getElementsByClassName("drag-hover")[this._id].classList.remove("drag-hover-clicked");
164+
document.getElementsByClassName("elyra-codeSnippet-item")[this._id].classList.remove("elyra-codeSnippet-item-clicked");
163165
event.stopPropagation();
164166
event.preventDefault();
165167
this.reject();

0 commit comments

Comments
 (0)