-
Notifications
You must be signed in to change notification settings - Fork 51
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
#261 Add copy to clipboard functionality to Raw Response #263
Conversation
micmro
commented
Oct 7, 2019
Ah, ok that was the one you meant 😄. For some reason I though about the raw data. What do you think of showing it in both? Copying the raw data is quite a pain too. |
overlay.onClose(overlay.index); | ||
body.removeEventListener("click", onTabDataCopyClick); | ||
}); | ||
body.addEventListener("click", onTabDataCopyClick); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use event delegation as we can't bind callbacks before the tab has been added to the DOM, this also allows us to clean up the event listeners when it gets removed.
Both sounds great :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Woho!!
Released with |