feat(ui): refines tooltip text and placement per feedback

Author: wgordon17

src/app/components/dashboard/PersonalSummaryStrip.tsx

@@ -177,7 +177,7 @@ export default function PersonalSummaryStrip(props: PersonalSummaryStripProps) {
             </>
           )}
         </For>
-        <InfoTooltip content="Click a stat to jump to that tab with filters applied." placement="bottom" />
+        <InfoTooltip content="Click any count to view those items." placement="bottom" />
       </div>
     </Show>
   );

src/app/components/layout/FilterBar.tsx

@@ -2,6 +2,7 @@ import { createMemo, createSignal, createEffect, onCleanup, Show } from "solid-j
 import { Select } from "@kobalte/core/select";
 import { config } from "../../stores/config";
 import { viewState, setGlobalFilter } from "../../stores/view";
+import { Tooltip } from "../shared/Tooltip";
 
 interface FilterBarProps {
   isRefreshing?: boolean;
@@ -118,34 +119,35 @@ export default function FilterBar(props: FilterBarProps) {
         </span>
       </Show>
 
-      <button
-        onClick={props.onRefresh}
-        disabled={props.isRefreshing}
-        class="btn btn-ghost btn-sm"
-        aria-label="Refresh data"
-      >
-        <Show
-          when={props.isRefreshing}
-          fallback={
-            <svg
-              xmlns="http://www.w3.org/2000/svg"
-              class="h-4 w-4 inline-block mr-1"
-              viewBox="0 0 20 20"
-              fill="currentColor"
-              aria-hidden="true"
-            >
-              <path
-                fill-rule="evenodd"
-                d="M4 2a1 1 0 011 1v2.101a7.002 7.002 0 0111.601 2.566 1 1 0 11-1.885.666A5.002 5.002 0 005.999 7H9a1 1 0 010 2H4a1 1 0 01-1-1V3a1 1 0 011-1zm.008 9.057a1 1 0 011.276.61A5.002 5.002 0 0014.001 13H11a1 1 0 110-2h5a1 1 0 011 1v5a1 1 0 11-2 0v-2.101a7.002 7.002 0 01-11.601-2.566 1 1 0 01.61-1.276z"
-                clip-rule="evenodd"
-              />
-            </svg>
-          }
+      <Tooltip content="Refresh data">
+        <button
+          onClick={props.onRefresh}
+          disabled={props.isRefreshing}
+          class="btn btn-ghost btn-sm"
+          aria-label="Refresh data"
         >
-          <span class="loading loading-spinner loading-xs mr-1" />
-        </Show>
-        Refresh
-      </button>
+          <Show
+            when={props.isRefreshing}
+            fallback={
+              <svg
+                xmlns="http://www.w3.org/2000/svg"
+                class="h-4 w-4"
+                viewBox="0 0 20 20"
+                fill="currentColor"
+                aria-hidden="true"
+              >
+                <path
+                  fill-rule="evenodd"
+                  d="M4 2a1 1 0 011 1v2.101a7.002 7.002 0 0111.601 2.566 1 1 0 11-1.885.666A5.002 5.002 0 005.999 7H9a1 1 0 010 2H4a1 1 0 01-1-1V3a1 1 0 011-1zm.008 9.057a1 1 0 011.276.61A5.002 5.002 0 0014.001 13H11a1 1 0 110-2h5a1 1 0 011 1v5a1 1 0 11-2 0v-2.101a7.002 7.002 0 01-11.601-2.566 1 1 0 01.61-1.276z"
+                  clip-rule="evenodd"
+                />
+              </svg>
+            }
+          >
+            <span class="loading loading-spinner loading-xs" />
+          </Show>
+        </button>
+      </Tooltip>
     </div>
   );
 }

src/app/components/shared/ExpandCollapseButtons.tsx

@@ -8,10 +8,10 @@ export interface ExpandCollapseButtonsProps {
 export default function ExpandCollapseButtons(props: ExpandCollapseButtonsProps) {
   return (
     <div class="flex items-center gap-1">
-      <Tooltip content="Expand all">
+      <Tooltip content="Expand all repos">
         <button
           class="btn btn-ghost btn-xs"
-          aria-label="Expand all"
+          aria-label="Expand all repos"
           onClick={props.onExpandAll}
         >
           <svg
@@ -30,10 +30,10 @@ export default function ExpandCollapseButtons(props: ExpandCollapseButtonsProps)
           </svg>
         </button>
       </Tooltip>
-      <Tooltip content="Collapse all">
+      <Tooltip content="Collapse all repos">
         <button
           class="btn btn-ghost btn-xs"
-          aria-label="Collapse all"
+          aria-label="Collapse all repos"
           onClick={props.onCollapseAll}
         >
           <svg

src/app/components/shared/RepoLockControls.tsx

@@ -49,7 +49,7 @@ export default function RepoLockControls(props: RepoLockControlsProps) {
             </svg>
           </button>
         </Tooltip>
-        <Tooltip content={lockInfo().isFirst ? "Already at top" : "Move up"}>
+        <Tooltip content={lockInfo().isFirst ? "Already at top of pinned list" : "Move up"}>
           <button
             class="btn btn-ghost btn-xs"
             onClick={() => moveLockedRepo(props.tab, props.repoFullName, "up")}
@@ -62,7 +62,7 @@ export default function RepoLockControls(props: RepoLockControlsProps) {
             </svg>
           </button>
         </Tooltip>
-        <Tooltip content={lockInfo().isLast ? "Already at bottom" : "Move down"}>
+        <Tooltip content={lockInfo().isLast ? "Already at bottom of pinned list" : "Move down"}>
           <button
             class="btn btn-ghost btn-xs"
             onClick={() => moveLockedRepo(props.tab, props.repoFullName, "down")}

src/app/components/shared/StatusDot.tsx

@@ -57,7 +57,7 @@ export default function StatusDot(props: StatusDotProps) {
   );
 
   return (
-    <Tooltip content={cfg().label} focusable={!props.href}>
+    <Tooltip content={cfg().label} focusable={!props.href} placement="right">
       <Show when={props.href} fallback={dot()}>
         {(url) => (
           <a

tests/components/ExpandCollapseButtons.test.tsx

@@ -6,23 +6,23 @@ import ExpandCollapseButtons from "../../src/app/components/shared/ExpandCollaps
 describe("ExpandCollapseButtons", () => {
   it("renders both buttons with correct aria-labels", () => {
     render(() => <ExpandCollapseButtons onExpandAll={() => {}} onCollapseAll={() => {}} />);
-    expect(screen.getByRole("button", { name: "Expand all" })).toBeTruthy();
-    expect(screen.getByRole("button", { name: "Collapse all" })).toBeTruthy();
+    expect(screen.getByRole("button", { name: "Expand all repos" })).toBeTruthy();
+    expect(screen.getByRole("button", { name: "Collapse all repos" })).toBeTruthy();
   });
 
   it("calls onExpandAll when expand button is clicked", async () => {
     const user = userEvent.setup();
     const onExpandAll = vi.fn();
     render(() => <ExpandCollapseButtons onExpandAll={onExpandAll} onCollapseAll={() => {}} />);
-    await user.click(screen.getByRole("button", { name: "Expand all" }));
+    await user.click(screen.getByRole("button", { name: "Expand all repos" }));
     expect(onExpandAll).toHaveBeenCalledTimes(1);
   });
 
   it("calls onCollapseAll when collapse button is clicked", async () => {
     const user = userEvent.setup();
     const onCollapseAll = vi.fn();
     render(() => <ExpandCollapseButtons onExpandAll={() => {}} onCollapseAll={onCollapseAll} />);
-    await user.click(screen.getByRole("button", { name: "Collapse all" }));
+    await user.click(screen.getByRole("button", { name: "Collapse all repos" }));
     expect(onCollapseAll).toHaveBeenCalledTimes(1);
   });
 });

tests/components/IssuesTab.test.tsx

@@ -407,7 +407,7 @@ describe("IssuesTab", () => {
     expect(screen.queryByText("org/repo-b")).toBeNull();
 
     // Collapse all — only affects visible (filtered) repos
-    await user.click(screen.getByLabelText("Collapse all"));
+    await user.click(screen.getByLabelText("Collapse all repos"));
     expect(screen.queryByText("Alice issue")).toBeNull();
 
     // Remove filter — repo-b should still be expanded (was hidden during collapse-all)
@@ -469,7 +469,7 @@ describe("IssuesTab", () => {
     expect(screen.queryByText("Issue A")).toBeNull();
     expect(screen.queryByText("Issue B")).toBeNull();
 
-    await user.click(screen.getByLabelText("Expand all"));
+    await user.click(screen.getByLabelText("Expand all repos"));
     screen.getByText("Issue A");
     screen.getByText("Issue B");
   });
@@ -485,7 +485,7 @@ describe("IssuesTab", () => {
     screen.getByText("Issue A");
     screen.getByText("Issue B");
 
-    await user.click(screen.getByLabelText("Collapse all"));
+    await user.click(screen.getByLabelText("Collapse all repos"));
     expect(screen.queryByText("Issue A")).toBeNull();
     expect(screen.queryByText("Issue B")).toBeNull();
   });
@@ -581,7 +581,7 @@ describe("IssuesTab", () => {
     screen.getByText(/Page 1 of 2/);
 
     // Expand all — affects repos on ALL pages
-    await user.click(screen.getByLabelText("Expand all"));
+    await user.click(screen.getByLabelText("Expand all repos"));
     // Repo-a items visible on page 1
     screen.getByText("Repo A issue 0");
 

tests/components/PullRequestsTab.test.tsx

@@ -509,7 +509,7 @@ describe("PullRequestsTab", () => {
     expect(screen.queryByText("PR in repo A")).toBeNull();
     expect(screen.queryByText("PR in repo B")).toBeNull();
 
-    await user.click(screen.getByLabelText("Expand all"));
+    await user.click(screen.getByLabelText("Expand all repos"));
 
     screen.getByText("PR in repo A");
     screen.getByText("PR in repo B");
@@ -527,7 +527,7 @@ describe("PullRequestsTab", () => {
     screen.getByText("PR in repo A");
     screen.getByText("PR in repo B");
 
-    await user.click(screen.getByLabelText("Collapse all"));
+    await user.click(screen.getByLabelText("Collapse all repos"));
 
     expect(screen.queryByText("PR in repo A")).toBeNull();
     expect(screen.queryByText("PR in repo B")).toBeNull();
@@ -618,7 +618,7 @@ describe("PullRequestsTab", () => {
     screen.getByText(/Page 1 of 2/);
 
     // Expand all — affects repos on ALL pages
-    await user.click(screen.getByLabelText("Expand all"));
+    await user.click(screen.getByLabelText("Expand all repos"));
     // Repo-a items visible on page 1
     screen.getByText("Repo A PR 0");