Skip to content

Sales Grid export to CSV not working as expected. #105

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.

Sign up for GitHub

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

Jump to bottom
Closed
skrustev opened this issue Apr 9, 2025 · 1 comment · Fixed by IgniteUI/grid-demos-react#3, IgniteUI/grid-demos-wc#9 or #106
Closed

Sales Grid export to CSV not working as expected. #105

skrustev opened this issue Apr 9, 2025 · 1 comment · Fixed by IgniteUI/grid-demos-react#3, IgniteUI/grid-demos-wc#9 or #106
Assignees
@skrustev @yradoeva
Labels
design:in-review ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.

Comments

@skrustev
Copy link
Member

skrustev commented Apr 9, 2025

Due CSV currently not supported for the Pivot Grid, the Export to CSV is pretty much useless in this sample so it needs to be removed.
Here's how it is looking at the moment:

Image

I think it should be just a single button in this case and loose the dropdown in general.
@sdimchevski sdimchevski assigned yradoeva and unassigned sdimchevski Apr 11, 2025
@yradoeva
Copy link

From design and UX point of view, if CSV export isn't supported for the Pivot Grid, the better approach is to show only the one option "Export to Excel" in an outlined button, without the dropdown arrow on the right.

Image

This was referenced Apr 23, 2025
Merged
Merged
@skrustev skrustev added 🛠️ status: in-development Issues and PRs with active development on them and removed 👀 status: in-review labels Apr 23, 2025
@skrustev skrustev mentioned this issue Apr 23, 2025
Merged
@skrustev skrustev added ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged. and removed 🛠️ status: in-development Issues and PRs with active development on them labels Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skrustev skrustev

@yradoeva yradoeva

Labels
design:in-review ✅ status: resolved Applies to issues that have pending PRs resolving them, or PRs that have already merged.
Projects
None yet
Milestone
No milestone
3 participants
@skrustev @sdimchevski @yradoeva