-
Notifications
You must be signed in to change notification settings - Fork 327
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
Download table content with icons as CSV #1626
Comments
Implementation notes: table download function creates blob that doesn't keep unicode chars and needs to be rewritten as described here. |
Hi, can I take up this issue? |
Hi @InfasMohammed. Sure, it's yours. |
Please follow the dev setup instructions in contribution guidelines. Once done, run the Note that your 4 days have already passed though which means this issue is again up for grabs for anyone interested. You may continue to work on it on your own if you want though. |
I will continue |
Is this issue still up for grabs ? I have the wave server running |
Yes, go for it @JacksonIsaac! |
I tried the changes, and tested by changing the examples/table files to have a unicode object. It works, but I would like to reproduce the same above scenario. Any ideas on how I can reproduce the above please ? |
You need to create a table with '❌' and '✅' as described here. |
Figured it out, thanks :) I spent hours on this, just to realize I myself had a bug in my example unicode :D |
Is your feature request related to a problem? Please describe
I was using default download feature in the wave tables. In my table it contains some icons which I need to display in downloaded CSV file as it is. But it shows some other characters in cells where those icons placed.
Table UI:
Expected cell in downloaded CSV:
Currently receiving output:
Describe the solution you'd like
Contain icons used in wave table cells as it is in the downloaded CSV file.
Describe alternatives you've considered
Use separate button to download CSV after including meaningful text instead of icons. But this not includes icons as it is. Just replacing icons with meaningful names.
Additional context
Discussion is in #1624
cc: @mturoci
The text was updated successfully, but these errors were encountered: