diff --git a/packages/excalidraw/actions/actionDeleteSelected.tsx b/packages/excalidraw/actions/actionDeleteSelected.tsx index 78a3465689..4fe8643eac 100644 --- a/packages/excalidraw/actions/actionDeleteSelected.tsx +++ b/packages/excalidraw/actions/actionDeleteSelected.tsx @@ -323,7 +323,9 @@ export const actionDeleteSelected = register({ title={t("labels.delete")} aria-label={t("labels.delete")} onClick={() => updateData(null)} - visible={isSomeElementSelected(getNonDeletedElements(elements), appState)} + disabled={ + !isSomeElementSelected(getNonDeletedElements(elements), appState) + } /> ), }); diff --git a/packages/excalidraw/actions/actionDuplicateSelection.tsx b/packages/excalidraw/actions/actionDuplicateSelection.tsx index c1b2a9da42..081980bd9d 100644 --- a/packages/excalidraw/actions/actionDuplicateSelection.tsx +++ b/packages/excalidraw/actions/actionDuplicateSelection.tsx @@ -115,7 +115,9 @@ export const actionDuplicateSelection = register({ )}`} aria-label={t("labels.duplicateSelection")} onClick={() => updateData(null)} - visible={isSomeElementSelected(getNonDeletedElements(elements), appState)} + disabled={ + !isSomeElementSelected(getNonDeletedElements(elements), appState) + } /> ), }); diff --git a/packages/excalidraw/components/Actions.tsx b/packages/excalidraw/components/Actions.tsx index c514b2255f..7d267d2773 100644 --- a/packages/excalidraw/components/Actions.tsx +++ b/packages/excalidraw/components/Actions.tsx @@ -919,10 +919,9 @@ export const MobileShapeActions = ({ style={{ flexDirection: "row", boxShadow: "none", - backgroundColor: "transparent", padding: 0, - margin: "0 0.25rem", zIndex: 2, + backgroundColor: "transparent", height: WIDTH * 1.75, alignItems: "center", gap: GAP, @@ -987,9 +986,6 @@ export const MobileShapeActions = ({ )} - {showDuplicate && renderAction("duplicateSelection")} - {showDelete && renderAction("deleteSelectedElements")} - {/* Combined Other Actions */} - {renderAction("undo")} - {showRedo && renderAction("redo")} +
{renderAction("undo")}
+ {showRedo && ( +
{renderAction("redo")}
+ )} + {showDuplicate && ( +
+ {renderAction("duplicateSelection")} +
+ )} + {showDelete && ( +
+ {renderAction("deleteSelectedElements")} +
+ )} );