From d24009f4941514d7cf69cce4914812fa3c374c61 Mon Sep 17 00:00:00 2001
From: 3720 <zuozijian1994@gmail.com>
Date: Fri, 30 Jun 2023 23:38:47 +0800
Subject: [PATCH] chore: adjust icon and style (#2949)

---
 .../workspace-slider-bar/collections/page.tsx  |  7 ++++---
 .../page-list/view/create-collection.tsx       | 18 +++++++++---------
 2 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/apps/web/src/components/pure/workspace-slider-bar/collections/page.tsx b/apps/web/src/components/pure/workspace-slider-bar/collections/page.tsx
index fe8c47c543..63c4c42715 100644
--- a/apps/web/src/components/pure/workspace-slider-bar/collections/page.tsx
+++ b/apps/web/src/components/pure/workspace-slider-bar/collections/page.tsx
@@ -4,7 +4,8 @@ import { useAFFiNEI18N } from '@affine/i18n/hooks';
 import {
   DeleteIcon,
   EdgelessIcon,
-  FilterIcon,
+  FilterMinusIcon,
+  FilterUndoIcon,
   MoreHorizontalIcon,
   PageIcon,
 } from '@blocksuite/icons';
@@ -56,7 +57,7 @@ export const PageOperations = ({
       ...(inAllowList
         ? [
             {
-              icon: <FilterIcon />,
+              icon: <FilterMinusIcon />,
               name: 'Remove special filter',
               click: () => removeFromAllowList(page.id),
             },
@@ -65,7 +66,7 @@ export const PageOperations = ({
       ...(!inExcludeList
         ? [
             {
-              icon: <FilterIcon />,
+              icon: <FilterUndoIcon />,
               name: 'Exclude from filter',
               click: () => addToExcludeList(page.id),
             },
diff --git a/packages/component/src/components/page-list/view/create-collection.tsx b/packages/component/src/components/page-list/view/create-collection.tsx
index f8b166a287..cbf754cf40 100644
--- a/packages/component/src/components/page-list/view/create-collection.tsx
+++ b/packages/component/src/components/page-list/view/create-collection.tsx
@@ -157,11 +157,11 @@ export const EditCollection = ({
         className={styles.scrollContainer}
         viewPortClassName={styles.container}
       >
-        <div className={styles.excludeList}>
-          <div className={styles.excludeTitle}>
-            Exclude from this collection
-          </div>
-          {value.excludeList ? (
+        {value.excludeList?.length ? (
+          <div className={styles.excludeList}>
+            <div className={styles.excludeTitle}>
+              Exclude from this collection
+            </div>
             <div className={styles.excludeListContent}>
               {value.excludeList.map(id => {
                 return (
@@ -174,11 +174,11 @@ export const EditCollection = ({
                 );
               })}
             </div>
-          ) : null}
-          <div className={styles.excludeTip}>
-            These pages will never appear in the current collection
+            <div className={styles.excludeTip}>
+              These pages will never appear in the current collection
+            </div>
           </div>
-        </div>
+        ) : null}
         <div
           style={{
             backgroundColor: 'var(--affine-hover-color)',