feat(topbar): add new look for application drawer trigger button when…

… triggerGroupShortName is set (#282)
AxisCommunications · Jun 13, 2024 · c2ed8cb · c2ed8cb
1 parent bc7fb66
commit c2ed8cb
Show file tree

Hide file tree

Showing 4 changed files with 44 additions and 3 deletions.
diff --git a/components/topbar/src/application-drawer.styles.ts b/components/topbar/src/application-drawer.styles.ts
@@ -1,5 +1,7 @@
 import { makeStyles, shorthands, tokens } from "@fluentui/react-components";
 
+const triggerSpacingWithGroup = tokens.spacingHorizontalS;
+
 export const useApplicationDrawrStyles = makeStyles({
   drawer: {
     ...shorthands.borderRight(0),
@@ -48,6 +50,10 @@ export const useApplicationDrawrStyles = makeStyles({
     ...shorthands.gap(tokens.spacingHorizontalS),
     ...shorthands.padding(tokens.spacingVerticalXS, tokens.spacingHorizontalM),
   },
+  drawerTriggerApplicationWithGroup: {
+    ...shorthands.gap(triggerSpacingWithGroup),
+    ...shorthands.padding(tokens.spacingVerticalXS, triggerSpacingWithGroup),
+  },
   drawerTriggerApplicationIcon: {
     display: "flex",
     color: tokens.colorNeutralForeground2,
@@ -61,6 +67,18 @@ export const useApplicationDrawrStyles = makeStyles({
   drawerTriggerApplicationText: {
     color: tokens.colorNeutralForeground2,
   },
+  drawerTriggerTextWithGroup: {
+    marginLeft: triggerSpacingWithGroup,
+    color: tokens.colorNeutralForeground2,
+  },
+  triggerDividerWithGroup: {
+    ...shorthands.border("0.5px", "solid", tokens.colorNeutralForeground2),
+    ...shorthands.borderRadius("1px"),
+    height: "12px",
+    boxSizing: "border-box",
+    marginTop: "1px",
+    marginLeft: triggerSpacingWithGroup,
+  },
   applicationGroupTitle: {
     display: "flex",
     alignItems: "center",

diff --git a/components/topbar/src/application-drawer.tsx b/components/topbar/src/application-drawer.tsx
@@ -49,10 +49,25 @@ const DrawerTrigger = (
         onMouseLeave={() => setHover(false)}
       >
         {<ApplicationAreaIcon applicationArea={applicationArea} />}
-        <Divider vertical style={{ padding: "0 0 0 12px" }}></Divider>
+        {currentSelection?.triggerGroupShortName
+          ? (
+            <>
+              <Body1Strong className={styles.drawerTriggerTextWithGroup}>
+                {currentSelection?.triggerGroupShortName}
+              </Body1Strong>
+              <div className={styles.triggerDividerWithGroup}></div>
+            </>
+          )
+          : <Divider vertical style={{ padding: "0 0 0 12px" }}></Divider>}
         {currentSelection
           ? (
-            <div className={styles.drawerTriggerApplication}>
+            <div
+              className={mergeClasses(
+                styles.drawerTriggerApplication,
+                currentSelection?.triggerGroupShortName
+                  && styles.drawerTriggerApplicationWithGroup
+              )}
+            >
               <div
                 className={mergeClasses(
                   styles.drawerTriggerApplicationIcon,

diff --git a/components/topbar/src/application-drawer.types.ts b/components/topbar/src/application-drawer.types.ts
@@ -12,11 +12,17 @@ export type SingleApplicationDrawerContent = {
   icon: JSX.Element;
   /** Label to show in opened drawer. */
   label: string;
-  /** Label to show in drawer trigger button when drawer is closed.
+  /**
+   * @deprecated
+   *  Label to show in drawer trigger button when drawer is closed.
    *  If not set, label will be used.
    *  Unused for content with children.
    */
   triggerLabel?: string;
+  /**
+   * Group label to show before triggerLabel
+   */
+  triggerGroupShortName?: string;
 };
 
 export type ApplicationDrawerProps = {

diff --git a/examples/src/components/top-bar.tsx b/examples/src/components/top-bar.tsx
@@ -85,6 +85,7 @@ export const Navbar = () => {
           icon: <FishIcon />,
           label: "Fisk",
           id: "fisk",
+          triggerGroupShortName: "FISH",
         },
         {
           icon: <CatIcon />,
@@ -98,6 +99,7 @@ export const Navbar = () => {
       label: "Add",
       triggerLabel: "ADD trigger",
       id: "add",
+      triggerGroupShortName: "ADD",
     },
   ];