From af90c55a01c9fe5afed14b1d74bc4c9549cfc89b Mon Sep 17 00:00:00 2001
From: minhtuevo <minhtuev@voxel51.com>
Date: Mon, 18 Nov 2024 15:55:57 -0800
Subject: [PATCH 1/3] Added tooltip to actions

---
 .../plugins/SchemaIO/components/ActionsMenu.tsx    | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/app/packages/core/src/plugins/SchemaIO/components/ActionsMenu.tsx b/app/packages/core/src/plugins/SchemaIO/components/ActionsMenu.tsx
index b75c4991f6..08c41781e3 100644
--- a/app/packages/core/src/plugins/SchemaIO/components/ActionsMenu.tsx
+++ b/app/packages/core/src/plugins/SchemaIO/components/ActionsMenu.tsx
@@ -9,6 +9,7 @@ import {
   Menu,
   MenuItem,
   Stack,
+  Tooltip
 } from "@mui/material";
 import React, { useCallback } from "react";
 import { getColorByCode } from "../utils";
@@ -73,7 +74,7 @@ function ActionsOverflowMenu(props: ActionsPropsType) {
 }
 
 function Action(props: ActionPropsType) {
-  const { label, name, onClick, icon, variant, mode, color, size } = props;
+  const { label, name, onClick, icon, variant, mode, color, size, tooltip } = props;
   const resolvedColor = color ? getColorByCode(color) : undefined;
 
   const Icon = icon ? (
@@ -87,7 +88,7 @@ function Action(props: ActionPropsType) {
     [onClick, props]
   );
 
-  return mode === "inline" ? (
+  const content = mode === "inline" ? (
     <Button
       variant={variant}
       startIcon={Icon}
@@ -102,6 +103,14 @@ function Action(props: ActionPropsType) {
       <ListItemText sx={{ color: resolvedColor }}>{label || name}</ListItemText>
     </MenuItem>
   );
+
+  return tooltip ? (
+    <Tooltip title={tooltip}>
+      <span>{content}</span> {/* Use <span> to wrap the child to avoid DOM issues */}
+    </Tooltip>
+  ) : (
+    content
+  );
 }
 
 type SizeType = "small" | "medium";
@@ -121,4 +130,5 @@ type ActionPropsType = {
   mode: "inline" | "menu";
   color?: string;
   size?: SizeType;
+  tooltip: string | null;
 };

From 1cc390f53cb81797cc6fe2af3c768a8b0f6920ad Mon Sep 17 00:00:00 2001
From: minhtuevo <minhtuev@voxel51.com>
Date: Mon, 18 Nov 2024 15:57:45 -0800
Subject: [PATCH 2/3] Added tooltip to Python obj

---
 fiftyone/operators/types.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/fiftyone/operators/types.py b/fiftyone/operators/types.py
index 89f37fccf5..54564654f8 100644
--- a/fiftyone/operators/types.py
+++ b/fiftyone/operators/types.py
@@ -1816,6 +1816,7 @@ class Action(View):
         name: the name of the action
         label (None): the label of the action
         icon (None): the icon of the action
+        tooltip (None): the tooltip of the action
         on_click: the operator to execute when the action is clicked
     """
 
@@ -1851,9 +1852,9 @@ def add_column(self, key, **kwargs):
         self.columns.append(column)
         return column
 
-    def add_row_action(self, name, on_click, label=None, icon=None, **kwargs):
+    def add_row_action(self, name, on_click, label=None, icon=None, tooltip=None, **kwargs):
         row_action = Action(
-            name=name, on_click=on_click, label=label, icon=icon, **kwargs
+            name=name, on_click=on_click, label=label, icon=icon, tooltip=tooltip, **kwargs
         )
         self.row_actions.append(row_action)
         return row_action

From 14bf1717ed3760c510ed6dd99b245bb51cd79214 Mon Sep 17 00:00:00 2001
From: CamronStaley <camron_staley@live.com>
Date: Mon, 18 Nov 2024 18:14:26 -0600
Subject: [PATCH 3/3] left align actions column and fix spacing

---
 .../SchemaIO/components/ActionsMenu.tsx       | 43 +++++++++++--------
 .../plugins/SchemaIO/components/TableView.tsx |  3 +-
 2 files changed, 26 insertions(+), 20 deletions(-)

diff --git a/app/packages/core/src/plugins/SchemaIO/components/ActionsMenu.tsx b/app/packages/core/src/plugins/SchemaIO/components/ActionsMenu.tsx
index 08c41781e3..1c742d2cf1 100644
--- a/app/packages/core/src/plugins/SchemaIO/components/ActionsMenu.tsx
+++ b/app/packages/core/src/plugins/SchemaIO/components/ActionsMenu.tsx
@@ -9,7 +9,7 @@ import {
   Menu,
   MenuItem,
   Stack,
-  Tooltip
+  Tooltip,
 } from "@mui/material";
 import React, { useCallback } from "react";
 import { getColorByCode } from "../utils";
@@ -21,7 +21,7 @@ export default function ActionsMenu(props: ActionsPropsType) {
 
   if (actions.length <= maxInline) {
     return (
-      <Stack direction="row" spacing={0.5} justifyContent="flex-end">
+      <Stack direction="row" spacing={0.5} justifyContent="flex-start">
         {actions.map((action) => (
           <Action {...action} key={action.name} mode="inline" size={size} />
         ))}
@@ -74,7 +74,8 @@ function ActionsOverflowMenu(props: ActionsPropsType) {
 }
 
 function Action(props: ActionPropsType) {
-  const { label, name, onClick, icon, variant, mode, color, size, tooltip } = props;
+  const { label, name, onClick, icon, variant, mode, color, size, tooltip } =
+    props;
   const resolvedColor = color ? getColorByCode(color) : undefined;
 
   const Icon = icon ? (
@@ -88,25 +89,29 @@ function Action(props: ActionPropsType) {
     [onClick, props]
   );
 
-  const content = mode === "inline" ? (
-    <Button
-      variant={variant}
-      startIcon={Icon}
-      onClick={handleClick}
-      sx={{ color: resolvedColor, padding: size === "small" ? 0 : undefined }}
-    >
-      {label}
-    </Button>
-  ) : (
-    <MenuItem onClick={handleClick}>
-      {Icon && <ListItemIcon>{Icon}</ListItemIcon>}
-      <ListItemText sx={{ color: resolvedColor }}>{label || name}</ListItemText>
-    </MenuItem>
-  );
+  const content =
+    mode === "inline" ? (
+      <Button
+        variant={variant}
+        startIcon={Icon}
+        onClick={handleClick}
+        sx={{ color: resolvedColor, padding: size === "small" ? 0 : undefined }}
+      >
+        {label}
+      </Button>
+    ) : (
+      <MenuItem onClick={handleClick}>
+        {Icon && <ListItemIcon>{Icon}</ListItemIcon>}
+        <ListItemText sx={{ color: resolvedColor }}>
+          {label || name}
+        </ListItemText>
+      </MenuItem>
+    );
 
   return tooltip ? (
     <Tooltip title={tooltip}>
-      <span>{content}</span> {/* Use <span> to wrap the child to avoid DOM issues */}
+      <span>{content}</span>{" "}
+      {/* Use <span> to wrap the child to avoid DOM issues */}
     </Tooltip>
   ) : (
     content
diff --git a/app/packages/core/src/plugins/SchemaIO/components/TableView.tsx b/app/packages/core/src/plugins/SchemaIO/components/TableView.tsx
index 7deb8e852e..8567db0760 100644
--- a/app/packages/core/src/plugins/SchemaIO/components/TableView.tsx
+++ b/app/packages/core/src/plugins/SchemaIO/components/TableView.tsx
@@ -141,8 +141,9 @@ export default function TableView(props: ViewPropsType) {
                 {hasRowActions && (
                   <TableCell
                     {...getComponentProps(props, "tableHeadCell", {
-                      sx: { ...headingCellBaseStyles, textAlign: "right" },
+                      sx: { ...headingCellBaseStyles, textAlign: "left" },
                     })}
+                    width={`${max_inline_actions * 5}%`}
                     onClick={() => {
                       handleCellClick(-1, -1);
                     }}