-
Notifications
You must be signed in to change notification settings - Fork 14k
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
feat: sort card view by Alphabetical, Recently Modified, and Least Recently Modified #10601
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10601 +/- ##
==========================================
+ Coverage 60.10% 64.39% +4.28%
==========================================
Files 775 776 +1
Lines 36528 36559 +31
Branches 3459 3464 +5
==========================================
+ Hits 21955 23541 +1586
+ Misses 14382 12909 -1473
+ Partials 191 109 -82
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I like the consistency with filters and columns api.
*/ | ||
import React, { useState } from 'react'; | ||
import { styled, withTheme, SupersetThemeProps } from '@superset-ui/style'; | ||
import { PartialThemeConfig, Select } from 'src/components/Select'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we probably could have gotten away with an antd dropdown as we'll only be rendering a few items and don't need search, but this is good for consistency with the filter components.
Impacts #8976 |
SUMMARY
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
WIP
ADDITIONAL INFORMATION