Skip to content

Commit

Permalink
[WEB-2520] fix-Sorted Icon Not Updating Dynamically in Spreadsheet Vi…
Browse files Browse the repository at this point in the history
…ew (#5688)

* Updated conditional rendering of sorting icons

* Removed unused imports
  • Loading branch information
Jimmycutie authored Oct 3, 2024
1 parent b7ee7e1 commit ee0dce4
Showing 1 changed file with 6 additions and 10 deletions.
Original file line number Diff line number Diff line change
@@ -1,15 +1,7 @@
"use client";

//ui
import {
ArrowDownWideNarrow,
ArrowUpNarrowWide,
CheckIcon,
ChevronDownIcon,
Eraser,
ListFilter,
MoveRight,
} from "lucide-react";
import { ArrowDownWideNarrow, ArrowUpNarrowWide, CheckIcon, ChevronDownIcon, Eraser, MoveRight } from "lucide-react";
import { IIssueDisplayFilterOptions, IIssueDisplayProperties, TIssueOrderByOptions } from "@plane/types";
import { CustomMenu, Row } from "@plane/ui";
//hooks
Expand Down Expand Up @@ -59,7 +51,11 @@ export const HeaderColumn = (props: Props) => {
<div className="ml-3 flex">
{activeSortingProperty === property && (
<div className="flex h-3.5 w-3.5 items-center justify-center rounded-full">
<ListFilter className="h-3 w-3" />
{propertyDetails.ascendingOrderKey === displayFilters.order_by ? (
<ArrowDownWideNarrow className="h-3 w-3" />
) : (
<ArrowUpNarrowWide className="h-3 w-3" />
)}
</div>
)}
<ChevronDownIcon className="h-3 w-3" aria-hidden="true" />
Expand Down

0 comments on commit ee0dce4

Please sign in to comment.