feature copy results to clipboard#513
Conversation
joewiz
left a comment
There was a problem hiding this comment.
It works great! Just a couple of suggestions in the wording. Thanks for adding tests too! I notice these appear to be failing?
|
@joewiz yes seems like CI test for copying doesn't work in this environment. |
Co-authored-by: Joe Wicentowski <joewiz@gmail.com>
Co-authored-by: Joe Wicentowski <joewiz@gmail.com>
|
@marmoure I also get the same "Browser context management is not supported" error shown in the CI log when running |
|
after updating the test i m still seeing a failure for checking the notification which is there???!! |
|
@marmoure Ah, I'm not sure... Is there a way to keep the tests while marking them as "pending" or "don't run"? Perhaps the situation with browser context management will change in the future, and we could just re-enable your tests. Re: #499, this PR is indeed quite similar, but my original idea was to allow copying on a per-item basis, but yours copies the current 10 results as a whole. I don't suppose it would be hard to add a per-item copy function? I'm happy to merge this without that, but I thought I'd ask in case it helps. |
|
@joewiz can you please elaborate and how that function work When you have multiple results. |
while working with eXide and running eval on some scripts i found that copying the results was not easy at all.
first i can't select it all using ctrl-A as the focus was always on the editor pane and trying to do it with the mouse proven to be a hassle.
i decided to add a copy to clipboard button which copies all the results showing in the results pane.