chore: issue and properties filter dropdown improvement (#4733)

makeplane · Jun 10, 2024 · 0fa703a · 0fa703a
1 parent 36cb36a
commit 0fa703a
Show file tree

Hide file tree

Showing 14 changed files with 46 additions and 17 deletions.
diff --git a/web/app/[workspaceSlug]/@header/profile/[userId]/mobile-header.tsx b/web/app/[workspaceSlug]/@header/profile/[userId]/mobile-header.tsx
@@ -148,6 +148,8 @@ const ProfileIssuesMobileHeader = observer(() => {
             }
             filters={issueFilters?.filters ?? {}}
             handleFiltersUpdate={handleFiltersUpdate}
+            displayFilters={issueFilters?.displayFilters ?? {}}
+            handleDisplayFiltersUpdate={handleDisplayFilters}
             states={states}
             labels={workspaceLabels}
             memberIds={members}

diff --git a/web/app/[workspaceSlug]/@header/projects/[projectId]/issues/header.tsx b/web/app/[workspaceSlug]/@header/projects/[projectId]/issues/header.tsx
@@ -182,6 +182,8 @@ const ProjectIssuesHeader: React.FC = observer(() => {
             <FilterSelection
               filters={issueFilters?.filters ?? {}}
               handleFiltersUpdate={handleFiltersUpdate}
+              displayFilters={issueFilters?.displayFilters ?? {}}
+              handleDisplayFiltersUpdate={handleDisplayFilters}
               layoutDisplayFiltersOptions={
                 activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.issues[activeLayout] : undefined
               }

diff --git a/web/app/[workspaceSlug]/@header/projects/[projectId]/issues/mobile-header.tsx b/web/app/[workspaceSlug]/@header/projects/[projectId]/issues/mobile-header.tsx
@@ -132,6 +132,8 @@ const ProjectIssuesMobileHeader = observer(() => {
             <FilterSelection
               filters={issueFilters?.filters ?? {}}
               handleFiltersUpdate={handleFiltersUpdate}
+              displayFilters={issueFilters?.displayFilters ?? {}}
+              handleDisplayFiltersUpdate={handleDisplayFilters}
               layoutDisplayFiltersOptions={
                 activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.issues[activeLayout] : undefined
               }

diff --git a/web/app/[workspaceSlug]/@header/workspace-views/header.tsx b/web/app/[workspaceSlug]/@header/workspace-views/header.tsx
@@ -118,6 +118,8 @@ const GlobalIssuesHeader: React.FC = observer(() => {
                 layoutDisplayFiltersOptions={ISSUE_DISPLAY_FILTERS_BY_LAYOUT.my_issues.spreadsheet}
                 filters={issueFilters?.filters ?? {}}
                 handleFiltersUpdate={handleFiltersUpdate}
+                displayFilters={issueFilters?.displayFilters ?? {}}
+                handleDisplayFiltersUpdate={handleDisplayFilters}
                 labels={workspaceLabels ?? undefined}
                 memberIds={workspaceMemberIds ?? undefined}
               />

diff --git a/web/components/cycles/cycle-mobile-header.tsx b/web/components/cycles/cycle-mobile-header.tsx
@@ -156,6 +156,8 @@ export const CycleMobileHeader = () => {
               layoutDisplayFiltersOptions={
                 activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.issues[activeLayout] : undefined
               }
+              displayFilters={issueFilters?.displayFilters ?? {}}
+              handleDisplayFiltersUpdate={handleDisplayFilters}
               labels={projectLabels}
               memberIds={projectMemberIds ?? undefined}
               states={projectStates}

diff --git a/web/components/headers/cycle-issues.tsx b/web/components/headers/cycle-issues.tsx
@@ -240,6 +240,8 @@ export const CycleIssuesHeader: React.FC = observer(() => {
                 layoutDisplayFiltersOptions={
                   activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.issues[activeLayout] : undefined
                 }
+                displayFilters={issueFilters?.displayFilters ?? {}}
+                handleDisplayFiltersUpdate={handleDisplayFilters}
                 labels={projectLabels}
                 memberIds={projectMemberIds ?? undefined}
                 states={projectStates}

diff --git a/web/components/headers/module-issues.tsx b/web/components/headers/module-issues.tsx
@@ -238,6 +238,8 @@ export const ModuleIssuesHeader: React.FC = observer(() => {
                 <FilterSelection
                   filters={issueFilters?.filters ?? {}}
                   handleFiltersUpdate={handleFiltersUpdate}
+                  displayFilters={issueFilters?.displayFilters ?? {}}
+                  handleDisplayFiltersUpdate={handleDisplayFilters}
                   layoutDisplayFiltersOptions={
                     activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.issues[activeLayout] : undefined
                   }

diff --git a/web/components/headers/project-draft-issues.tsx b/web/components/headers/project-draft-issues.tsx
@@ -137,6 +137,8 @@ export const ProjectDraftIssueHeader: FC = observer(() => {
             <FilterSelection
               filters={issueFilters?.filters ?? {}}
               handleFiltersUpdate={handleFiltersUpdate}
+              displayFilters={issueFilters?.displayFilters ?? {}}
+              handleDisplayFiltersUpdate={handleDisplayFilters}
               layoutDisplayFiltersOptions={
                 activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.issues[activeLayout] : undefined
               }

diff --git a/web/components/headers/project-view-issues.tsx b/web/components/headers/project-view-issues.tsx
@@ -215,6 +215,8 @@ export const ProjectViewIssuesHeader: React.FC = observer(() => {
           <FilterSelection
             filters={issueFilters?.filters ?? {}}
             handleFiltersUpdate={handleFiltersUpdate}
+            displayFilters={issueFilters?.displayFilters ?? {}}
+            handleDisplayFiltersUpdate={handleDisplayFilters}
             layoutDisplayFiltersOptions={
               activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.issues[activeLayout] : undefined
             }

diff --git a/web/components/issues/archived-issues-header.tsx b/web/components/issues/archived-issues-header.tsx
@@ -76,6 +76,8 @@ export const ArchivedIssuesHeader: FC = observer(() => {
           <FilterSelection
             filters={issueFilters?.filters || {}}
             handleFiltersUpdate={handleFiltersUpdate}
+            displayFilters={issueFilters?.displayFilters ?? {}}
+            handleDisplayFiltersUpdate={handleDisplayFiltersUpdate}
             layoutDisplayFiltersOptions={
               activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.archived_issues[activeLayout] : undefined
             }

diff --git a/...ponents/issues/issue-layouts/filters/header/display-filters/display-filters-selection.tsx b/...ponents/issues/issue-layouts/filters/header/display-filters/display-filters-selection.tsx
@@ -7,7 +7,6 @@ import {
   FilterDisplayProperties,
   FilterExtraOptions,
   FilterGroupBy,
-  FilterIssueType,
   FilterOrderBy,
   FilterSubGroupBy,
 } from "@/components/issues";
@@ -111,20 +110,6 @@ export const DisplayFiltersSelection: React.FC<Props> = observer((props) => {
         </div>
       )}
 
-      {/* issue type */}
-      {isDisplayFilterEnabled("type") && (
-        <div className="py-2">
-          <FilterIssueType
-            selectedIssueType={displayFilters?.type}
-            handleUpdate={(val) =>
-              handleDisplayFiltersUpdate({
-                type: val,
-              })
-            }
-          />
-        </div>
-      )}
-
       {/* Options */}
       {layoutDisplayFiltersOptions?.extra_options.access && (
         <div className="py-2">

diff --git a/web/components/issues/issue-layouts/filters/header/filters/filters-selection.tsx b/web/components/issues/issue-layouts/filters/header/filters/filters-selection.tsx
@@ -1,7 +1,7 @@
 import { useState } from "react";
 import { observer } from "mobx-react";
 import { Search, X } from "lucide-react";
-import { IIssueFilterOptions, IIssueLabel, IState } from "@plane/types";
+import { IIssueDisplayFilterOptions, IIssueFilterOptions, IIssueLabel, IState } from "@plane/types";
 // hooks
 import {
   FilterAssignees,
@@ -16,6 +16,7 @@ import {
   FilterTargetDate,
   FilterCycle,
   FilterModule,
+  FilterIssueType,
 } from "@/components/issues";
 import { ILayoutDisplayFiltersOptions } from "@/constants/issue";
 import { useAppRouter } from "@/hooks/store";
@@ -25,6 +26,8 @@ import { useAppRouter } from "@/hooks/store";
 
 type Props = {
   filters: IIssueFilterOptions;
+  displayFilters?: IIssueDisplayFilterOptions | undefined;
+  handleDisplayFiltersUpdate?: (updatedDisplayFilter: Partial<IIssueDisplayFilterOptions>) => void;
   handleFiltersUpdate: (key: keyof IIssueFilterOptions, value: string | string[]) => void;
   layoutDisplayFiltersOptions: ILayoutDisplayFiltersOptions | undefined;
   labels?: IIssueLabel[] | undefined;
@@ -37,6 +40,8 @@ type Props = {
 export const FilterSelection: React.FC<Props> = observer((props) => {
   const {
     filters,
+    displayFilters,
+    handleDisplayFiltersUpdate,
     handleFiltersUpdate,
     layoutDisplayFiltersOptions,
     labels,
@@ -52,6 +57,9 @@ export const FilterSelection: React.FC<Props> = observer((props) => {
 
   const isFilterEnabled = (filter: keyof IIssueFilterOptions) => layoutDisplayFiltersOptions?.filters.includes(filter);
 
+  const isDisplayFilterEnabled = (displayFilter: keyof IIssueDisplayFilterOptions) =>
+    Object.keys(layoutDisplayFiltersOptions?.display_filters ?? {}).includes(displayFilter);
+
   return (
     <div className="flex h-full w-full flex-col overflow-hidden">
       <div className="bg-custom-background-100 p-2.5 pb-0">
@@ -187,7 +195,19 @@ export const FilterSelection: React.FC<Props> = observer((props) => {
             />
           </div>
         )}
-
+        {/* issue type */}
+        {isDisplayFilterEnabled("type") && displayFilters && handleDisplayFiltersUpdate && (
+          <div className="py-2">
+            <FilterIssueType
+              selectedIssueType={displayFilters.type}
+              handleUpdate={(val) =>
+                handleDisplayFiltersUpdate({
+                  type: val,
+                })
+              }
+            />
+          </div>
+        )}
         {/* start_date */}
         {isFilterEnabled("start_date") && (
           <div className="py-2">

diff --git a/web/components/modules/module-mobile-header.tsx b/web/components/modules/module-mobile-header.tsx
@@ -134,6 +134,8 @@ export const ModuleMobileHeader = observer(() => {
             <FilterSelection
               filters={issueFilters?.filters ?? {}}
               handleFiltersUpdate={handleFiltersUpdate}
+              displayFilters={issueFilters?.displayFilters ?? {}}
+              handleDisplayFiltersUpdate={handleDisplayFilters}
               layoutDisplayFiltersOptions={
                 activeLayout ? ISSUE_DISPLAY_FILTERS_BY_LAYOUT.issues[activeLayout] : undefined
               }

diff --git a/web/components/profile/profile-issues-filter.tsx b/web/components/profile/profile-issues-filter.tsx
@@ -112,6 +112,8 @@ export const ProfileIssuesFilter = observer(() => {
           }
           filters={issueFilters?.filters ?? {}}
           handleFiltersUpdate={handleFiltersUpdate}
+          displayFilters={issueFilters?.displayFilters ?? {}}
+          handleDisplayFiltersUpdate={handleDisplayFilters}
           states={states}
           labels={workspaceLabels}
           memberIds={members}