Skip to content
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

[Discover] Copy rows as text to clipboard UX / UI enhancements #179731

Closed
Tracked by #191570
renshuki opened this issue Apr 1, 2024 · 3 comments · Fixed by #189512
Closed
Tracked by #191570

[Discover] Copy rows as text to clipboard UX / UI enhancements #179731

renshuki opened this issue Apr 1, 2024 · 3 comments · Fixed by #189512
Assignees
Labels
enhancement New value added to drive a business result Feature:Discover Discover Application impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort Project:OneDiscover Enrich Discover with contextual awareness / Merge with Logs Explorer Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Data, DataViews)

Comments

@renshuki
Copy link
Contributor

renshuki commented Apr 1, 2024

Tested on:
- Kibana 8.11.3 and 8.12.2
- Chrome Version 123.0.6312.59 (Official Build) (arm64)

Describe the feature

The behavior of Discover - Document Explorer compared to the Discover - Classic View (legacy) differ when it comes to copy content.

Extracting content via mouse select / copy

  • Extracting content (rows) from Document Explorer is currently difficult. As you can see below, mouse selection of the rows is not always working as expected.
dicover_doc_explorer_copy_paste.mov

In this example, the first line and second-last line are not selected

  • Copy-pasting the content to a text editor is rendered as:
Dec 18, 2023 @ 14:13:00.000
58U013N
Mexico City
MX
CN
Xi'an
kibana_sample_data_flights


Dec 18, 2023 @ 19:33:28.000
EAYQW69
Naples
IT
IT
Treviso
kibana_sample_data_flights


Dec 19, 2023 @ 02:11:14.000
UFK2WIZ
Venice
IT
IT
Venice
kibana_sample_data_flights

Being able to keep the "row" format would be nice.

Copy documents to clipboard as TXT

  • When selecting rows using checkboxes, only a Copy documents to clipboard (JSON) option is available. I suggest adding a Copy documents to clipboard (TXT) option as well.

Example:
Screen Shot 2024-04-01 at 10 44 04

The Copy documents to clipboard (TXT) would copy the rows as displayed in Discover.

Select all checkboxes feature

  • Checkboxes can be selected one by one but there is no feature to select all rows (as I'm aware of).

Example:
Screen Shot 2024-04-01 at 10 51 15

Shift+click to select multiple rows

  • Capability to do a shift+click to select a specific range of rows

Describe a specific use case for the feature

CSV Reports feature could be used to extract content from Discover but:

  • it takes time to generate
  • the CSV file should be downloaded locally
  • the CSV file should be opened in a dedicated app (i.e. text editor)

Please consider above enhancements for a more seamless method to copy Discover content to clipboard.

@renshuki renshuki added Feature:Discover Discover Application enhancement New value added to drive a business result Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Data, DataViews) labels Apr 1, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-data-discovery (Team:DataDiscovery)

@kertal kertal added impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort labels Apr 12, 2024
@kertal
Copy link
Member

kertal commented Apr 12, 2024

thx for the input and the detailed description, this will be a useful feature. Adding a note by @davismcphee who recently a document comparison mode to the table (#166577), that we might consider capping the number of comparison docs.

@renshuki
Copy link
Contributor Author

Thanks @kertal, I just noticed I uploaded the wrong screenshot for the Copy documents to clipboard (TXT) part, now edited and I hope it makes things clearer :)

@ninoslavmiskovic ninoslavmiskovic added the Project:OneDiscover Enrich Discover with contextual awareness / Merge with Logs Explorer label Jun 13, 2024
@kertal kertal changed the title [Discover] Copy content to clipboard UX / UI enhancements [Discover] Copy rows as text to clipboard UX / UI enhancements Jun 25, 2024
jughosta added a commit that referenced this issue Aug 12, 2024
- Closes #179731
- Related to #177952 
- Related to elastic/eui#6804

## Summary

This PR adds "Copy selection as text" action.

<img width="926" alt="Screenshot 2024-07-31 at 18 09 21"
src="https://github.com/user-attachments/assets/a4fc7456-7cd9-4493-a4dd-45151f845566">


### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/packages/kbn-i18n/README.md)
- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] This was checked for [cross-browser
compatibility](https://www.elastic.co/support/matrix#matrix_browsers)

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Davis McPhee <davismcphee@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Discover Discover Application impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:medium Medium Level of Effort Project:OneDiscover Enrich Discover with contextual awareness / Merge with Logs Explorer Team:DataDiscovery Discover App Team (Document Explorer, Saved Search, Surrounding documents, Data, DataViews)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants