sambruk/excalidraw
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: buttons jump around on the mobile menu (#5658)

Browse Source 
Update MobileMenu.tsx
This commit is contained in:
zsviczian 2022-09-05 12:30:47 +02:00 committed by GitHub
parent b3052f0178
commit cd61f31116
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/components/MobileMenu.tsx
View File

@ -121,7 +121,6 @@ export const MobileMenu = ({
const renderAppToolbar = () => { const renderAppToolbar = () => {
// Render eraser conditionally in mobile // Render eraser conditionally in mobile
const showEraser = const showEraser =
!appState.viewModeEnabled &&
!appState.editingElement && !appState.editingElement &&
getSelectedElements(elements, appState).length === 0; getSelectedElements(elements, appState).length === 0;
@ -140,9 +139,9 @@ export const MobileMenu = ({
{actionManager.renderAction("undo")} {actionManager.renderAction("undo")}
{actionManager.renderAction("redo")} {actionManager.renderAction("redo")}
{showEraser && actionManager.renderAction("eraser")} {showEraser
? actionManager.renderAction("eraser")
{actionManager.renderAction( : actionManager.renderAction(
appState.multiElement ? "finalize" : "duplicateSelection", appState.multiElement ? "finalize" : "duplicateSelection",
)} )}
{actionManager.renderAction("deleteSelectedElements")} {actionManager.renderAction("deleteSelectedElements")}