-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Replace with sorting icon #11815
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
Replace with sorting icon #11815
Conversation
started the job as gitpod-build-laushinka-replace-column-sorting-11783.1 because the annotations in the pull request description changed |
22f1baa
to
37eb94d
Compare
da1eacf
to
4e46af5
Compare
@gtsiolis How do I rotate the SVG? 🤣 I tried using |
@@ -204,7 +204,9 @@ function TeamUsage() { | |||
<ItemField className="my-auto cursor-pointer"> | |||
<span onClick={() => setIsStartedTimeDescending(!isStartedTimeDescending)}> | |||
Started | |||
<Arrow direction={isStartedTimeDescending ? "down" : "up"} /> | |||
<SortArrow | |||
className={`h-4 w-4 ${isStartedTimeDescending ? "" : "rotate-180"}`} |
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.
@laushinka Regarding the question about rotation in #11815 (comment), you'll need to add the transform
class here for this to work, right? See relevant docs for 2.x
v. 3.x
.
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.
LOL thank you as always @gtsiolis!
4e46af5
to
a19f7a5
Compare
Description
Uses the correct arrow icon.
Related Issue(s)
Fixes #11783
How to test
Release Notes
Documentation
Werft options: