From 60fc34ffac4c2d503a152b7e2f21b4557c0a4ef6 Mon Sep 17 00:00:00 2001
From: Elian Doran <contact@eliandoran.me>
Date: Fri, 12 Dec 2025 21:57:42 +0200
Subject: [PATCH] feat(status_bar): functional attribute toggle button

---
 .../src/stylesheets/theme-next/forms.css      |  9 ++++---
 apps/client/src/widgets/layout/StatusBar.css  |  1 +
 apps/client/src/widgets/layout/StatusBar.tsx  | 26 ++++++++++++-------
 3 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/apps/client/src/stylesheets/theme-next/forms.css b/apps/client/src/stylesheets/theme-next/forms.css
index fb53f167d..de76231be 100644
--- a/apps/client/src/stylesheets/theme-next/forms.css
+++ b/apps/client/src/stylesheets/theme-next/forms.css
@@ -154,7 +154,7 @@ button.btn.btn-success kbd {
     color: var(--button-group-active-button-text-color);
 }
 
-/* 
+/*
  * Input boxes
  */
 
@@ -399,7 +399,8 @@ button.select-button.dropdown-toggle.btn:active {
 select:focus,
 select.form-select:focus,
 select.form-control:focus,
-.select-button.dropdown-toggle.btn:focus {
+.select-button.dropdown-toggle.btn:focus,
+.select-button.focus-outline:focus {
     box-shadow: unset;
     outline: 3px solid var(--input-focus-outline-color);
     outline-offset: 0;
@@ -422,7 +423,7 @@ optgroup {
     line-height: 40px;
 }
 
-/* 
+/*
  * File input
  *
  * <label class="tn-file-input tn-input-field">
@@ -784,4 +785,4 @@ input[type="range"] {
         scrollbar-color: unset;
         scrollbar-width: unset;
     }
-}
\ No newline at end of file
+}
diff --git a/apps/client/src/widgets/layout/StatusBar.css b/apps/client/src/widgets/layout/StatusBar.css
index 1fd845083..c69b0e31e 100644
--- a/apps/client/src/widgets/layout/StatusBar.css
+++ b/apps/client/src/widgets/layout/StatusBar.css
@@ -26,6 +26,7 @@
                 align-items: center;
                 border: 0;
 
+                &.active,
                 &:focus,
                 &:hover {
                     background: var(--input-background-color);
diff --git a/apps/client/src/widgets/layout/StatusBar.tsx b/apps/client/src/widgets/layout/StatusBar.tsx
index 78608c0fe..37a86e390 100644
--- a/apps/client/src/widgets/layout/StatusBar.tsx
+++ b/apps/client/src/widgets/layout/StatusBar.tsx
@@ -36,11 +36,12 @@ interface StatusBarContext {
 
 export default function StatusBar() {
     const { note, noteContext, viewScope } = useActiveNoteContext();
+    const [ attributesShown, setAttributesShown ] = useState(false);
     const context = note && noteContext && { note, noteContext, viewScope } satisfies StatusBarContext;
 
     return (
         <div className="status-bar">
-            {context && <AttributesPane {...context} />}
+            {context && <AttributesPane shown={attributesShown} {...context} />}
 
             <div className="status-bar-main-row">
                 {context && <>
@@ -49,7 +50,7 @@ export default function StatusBar() {
                     </div>
 
                     <div className="actions-row">
-                        <AttributesButton {...context} />
+                        <AttributesButton attributesShown={attributesShown} setAttributesShown={setAttributesShown} {...context} />
                         <AttachmentCount {...context} />
                         <BacklinksBadge {...context} />
                         <LanguageSwitcher {...context} />
@@ -93,12 +94,13 @@ interface StatusBarButtonBaseProps {
     title: string;
     text?: string | number;
     disabled?: boolean;
+    active?: boolean;
 }
 
 type StatusBarButtonWithCommand = StatusBarButtonBaseProps & { triggerCommand: CommandNames; };
 type StatusBarButtonWithClick = StatusBarButtonBaseProps & { onClick: () => void; };
 
-function StatusBarButton({ className, icon, text, title, ...restProps }: StatusBarButtonWithCommand | StatusBarButtonWithClick) {
+function StatusBarButton({ className, icon, text, title, active, ...restProps }: StatusBarButtonWithCommand | StatusBarButtonWithClick) {
     const parentComponent = useContext(ParentComponent);
     const buttonRef = useRef<HTMLButtonElement>(null);
     useStaticTooltip(buttonRef, {
@@ -111,7 +113,7 @@ function StatusBarButton({ className, icon, text, title, ...restProps }: StatusB
     return (
         <button
             ref={buttonRef}
-            className={clsx("btn select-button", className)}
+            className={clsx("btn select-button focus-outline", className, active && "active")}
             type="button"
             onClick={() => {
                 if ("triggerCommand" in restProps) {
@@ -246,7 +248,10 @@ function AttachmentCount({ note }: StatusBarContext) {
 //#endregion
 
 //#region Attributes
-function AttributesButton({ note }: StatusBarContext) {
+function AttributesButton({ note, attributesShown, setAttributesShown }: StatusBarContext & {
+    attributesShown: boolean;
+    setAttributesShown: (shown: boolean) => void;
+}) {
     const [ count, setCount ] = useState(note.attributes.length);
 
     // React to changes in count.
@@ -262,14 +267,15 @@ function AttributesButton({ note }: StatusBarContext) {
             icon="bx bx-list-check"
             title={t("status_bar.attributes_title")}
             text={t("status_bar.attributes", { count })}
-            onClick={() => {
-                alert("Hi");
-            }}
+            active={attributesShown}
+            onClick={() => setAttributesShown(!attributesShown)}
         />
     );
 }
 
-function AttributesPane({ note, noteContext }: StatusBarContext) {
+function AttributesPane({ note, noteContext, shown }: StatusBarContext & {
+    shown: boolean;
+}) {
     const parentComponent = useContext(ParentComponent);
     const api = useRef<AttributeEditorImperativeHandlers>(null);
 
@@ -280,7 +286,7 @@ function AttributesPane({ note, noteContext }: StatusBarContext) {
     };
 
     return (context &&
-        <div className="attribute-list">
+        <div className={clsx("attribute-list", !shown && "hidden-ext")}>
             <InheritedAttributesTab {...context} />
 
             <AttributeEditor