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

[EuiDataGrid] Sorting and schema issues #2399

Closed
snide opened this issue Oct 3, 2019 · 1 comment
Closed

[EuiDataGrid] Sorting and schema issues #2399

snide opened this issue Oct 3, 2019 · 1 comment
Labels
bug data grid ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible

Comments

@snide
Copy link
Contributor

snide commented Oct 3, 2019

Currency schema is overaggressive

Assume it's looking at the periods in the version

image

Sorting for numbers has trouble

For currency it should be the lowest number to highest.

Boolean ASC should likely be true-false, not false-true

This is minor, and i solved it with labeling, but it feels wrong to ASC by false first.

image

JSON should maybe be off? Or based on size?

image

Email / render content doesn't respect alpha sorting

Assume because it's coming in a a rendered react node and not a string

image

@chandlerprall chandlerprall self-assigned this Oct 4, 2019
@cchaos cchaos changed the title [DATA GRID] Sorting and schema issues [EuiDataGrid] Sorting and schema issues Sep 20, 2020
@daveyholler daveyholler added the ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible label Nov 28, 2022
@cee-chen
Copy link
Contributor

cee-chen commented May 1, 2023

Closing this as a wontfix - listed examples appear to no longer apply to our current data grid docs.

@cee-chen cee-chen closed this as not planned Won't fix, can't repro, duplicate, stale May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug data grid ⚠️ needs validation For bugs that need confirmation as to whether they're reproducible
Projects
None yet
Development

No branches or pull requests

5 participants