fix(component): rework tags list collapsing (#5072)

Before:

![CleanShot 2023-11-27 at 16.39.55@2x.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/T2klNLEk0wxLh4NRDzhk/2ac2b8e3-6c30-41f7-a9b2-7a9c81b250fa.png)

After:
![CleanShot 2023-11-27 at 16.38.50@2x.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/T2klNLEk0wxLh4NRDzhk/12eac806-e641-45be-9215-d166f8733db9.png)

packages/frontend/component/src/components/page-list/page-tags.css.ts

@@ -1,4 +1,6 @@
-import { style } from '@vanilla-extract/css';
+import { createVar, style } from '@vanilla-extract/css';
+
+export const hoverMaxWidth = createVar();
 
 export const root = style({
   position: 'relative',
@@ -15,7 +17,8 @@ export const tagsContainer = style({
 export const tagsScrollContainer = style([
   tagsContainer,
   {
-    overflow: 'auto',
+    overflowX: 'hidden',
+    position: 'relative',
     height: '100%',
     gap: '8px',
   },
@@ -41,7 +44,7 @@ export const innerContainer = style({
   transition: 'all 0.2s 0.3s ease-in-out',
   selectors: {
     [`${root}:hover &`]: {
-      maxWidth: 'var(--hover-max-width)',
+      maxWidth: hoverMaxWidth,
     },
   },
 });
@@ -66,6 +69,16 @@ export const innerBackdrop = style({
 
 export const tag = style({
   height: '20px',
+  display: 'flex',
+  minWidth: 0,
+  alignItems: 'center',
+  justifyContent: 'space-between',
+  ':last-child': {
+    minWidth: 'max-content',
+  },
+});
+
+export const tagInnerWrapper = style({
   display: 'flex',
   alignItems: 'center',
   justifyContent: 'space-between',
@@ -74,7 +87,7 @@ export const tag = style({
 });
 
 export const tagSticky = style([
-  tag,
+  tagInnerWrapper,
   {
     fontSize: 'var(--affine-font-xs)',
     borderRadius: '10px',
@@ -82,10 +95,8 @@ export const tagSticky = style([
     border: '1px solid var(--affine-border-color)',
     background: 'var(--affine-background-primary-color)',
     maxWidth: '128px',
-    position: 'sticky',
     textOverflow: 'ellipsis',
     whiteSpace: 'nowrap',
-    left: 0,
   },
 ]);
 

packages/frontend/component/src/components/page-list/page-tags.tsx

@@ -1,8 +1,9 @@
 import type { Tag } from '@affine/env/filter';
 import { MoreHorizontalIcon } from '@blocksuite/icons';
 import { Menu } from '@toeverything/components/menu';
+import { assignInlineVars } from '@vanilla-extract/dynamic';
 import clsx from 'clsx';
-import { useEffect, useMemo, useRef } from 'react';
+import { useMemo } from 'react';
 
 import * as styles from './page-tags.css';
 import { stopPropagation } from './utils';
@@ -42,10 +43,13 @@ const TagItem = ({ tag, idx, mode, style }: TagItemProps) => {
   return (
     <div
       data-testid="page-tag"
-      className={mode === 'sticky' ? styles.tagSticky : styles.tagListItem}
+      className={styles.tag}
       data-idx={idx}
       title={tag.value}
       style={style}
+    >
+      <div
+        className={mode === 'sticky' ? styles.tagSticky : styles.tagListItem}
       >
         <div
           className={styles.tagIndicator}
@@ -55,6 +59,7 @@ const TagItem = ({ tag, idx, mode, style }: TagItemProps) => {
         />
         <div className={styles.tagLabel}>{tag.value}</div>
       </div>
+    </div>
   );
 };
 
@@ -69,26 +74,6 @@ export const PageTags = ({
       ? widthOnHover
       : `${widthOnHover}px`
     : 'auto';
-  const tagsContainerRef = useRef<HTMLDivElement>(null);
-
-  useEffect(() => {
-    if (tagsContainerRef.current) {
-      const tagsContainer = tagsContainerRef.current;
-      const listener = () => {
-        // on mouseleave, reset scroll position to the hoverExpandDirection
-        tagsContainer.scrollTo({
-          left: hoverExpandDirection === 'left' ? Number.MAX_SAFE_INTEGER : 0,
-          behavior: 'smooth',
-        });
-      };
-      listener();
-      tagsContainerRef.current.addEventListener('mouseleave', listener);
-      return () => {
-        tagsContainer.removeEventListener('mouseleave', listener);
-      };
-    }
-    return;
-  }, [hoverExpandDirection]);
 
   const tagsInPopover = useMemo(() => {
     const lastTags = tags.slice(maxItems);
@@ -107,36 +92,17 @@ export const PageTags = ({
     // sort tags by length
     nTags.sort((a, b) => a.value.length - b.value.length);
 
-    const tagRightCharLength = nTags.reduceRight<number[]>(
-      (acc, tag) => {
-        const curr = acc[0] + Math.min(tag.value.length, 10);
-        return [curr, ...acc];
-      },
-      [0]
-    );
-
-    tagRightCharLength.shift();
-
     return nTags.map((tag, idx) => (
-      <TagItem
-        key={tag.id}
-        tag={tag}
-        idx={idx}
-        mode="sticky"
-        style={{
-          right: `calc(${tagRightCharLength[idx]}em)`,
-        }}
-      />
+      <TagItem key={tag.id} tag={tag} idx={idx} mode="sticky" />
     ));
   }, [maxItems, tags]);
   return (
     <div
       data-testid="page-tags"
       className={styles.root}
-      style={{
-        // @ts-expect-error it's fine
-        '--hover-max-width': sanitizedWidthOnHover,
-      }}
+      style={assignInlineVars({
+        [styles.hoverMaxWidth]: sanitizedWidthOnHover,
+      })}
     >
       <div
         style={{
@@ -146,9 +112,7 @@ export const PageTags = ({
         className={clsx(styles.innerContainer)}
       >
         <div className={styles.innerBackdrop} />
-        <div className={styles.tagsScrollContainer} ref={tagsContainerRef}>
-          {tagsNormal}
-        </div>
+        <div className={styles.tagsScrollContainer}>{tagsNormal}</div>
         {maxItems && tags.length > maxItems ? (
           <Menu
             items={tagsInPopover}