Browse Source

fix: align arrows bound to elements excalidraw#8833 (#8998)

pull/9052/head
Alplune 11 months ago committed by GitHub
parent
commit
807b3c59f2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
  1. 1
      packages/excalidraw/actions/actionAlign.tsx
  2. 19
      packages/excalidraw/align.ts

1
packages/excalidraw/actions/actionAlign.tsx

@ -48,6 +48,7 @@ const alignSelectedElements = ( @@ -48,6 +48,7 @@ const alignSelectedElements = (
selectedElements,
elementsMap,
alignment,
app.scene,
);
const updatedElementsMap = arrayToMap(updatedElements);

19
packages/excalidraw/align.ts

@ -1,8 +1,10 @@ @@ -1,8 +1,10 @@
import type { ElementsMap, ExcalidrawElement } from "./element/types";
import { newElementWith } from "./element/mutateElement";
import { mutateElement } from "./element/mutateElement";
import type { BoundingBox } from "./element/bounds";
import { getCommonBoundingBox } from "./element/bounds";
import { getMaximumGroups } from "./groups";
import { updateBoundElements } from "./element/binding";
import type Scene from "./scene/Scene";
export interface Alignment {
position: "start" | "center" | "end";
@ -13,6 +15,7 @@ export const alignElements = ( @@ -13,6 +15,7 @@ export const alignElements = (
selectedElements: ExcalidrawElement[],
elementsMap: ElementsMap,
alignment: Alignment,
scene: Scene,
): ExcalidrawElement[] => {
const groups: ExcalidrawElement[][] = getMaximumGroups(
selectedElements,
@ -26,12 +29,18 @@ export const alignElements = ( @@ -26,12 +29,18 @@ export const alignElements = (
selectionBoundingBox,
alignment,
);
return group.map((element) =>
newElementWith(element, {
return group.map((element) => {
// update element
const updatedEle = mutateElement(element, {
x: element.x + translation.x,
y: element.y + translation.y,
}),
);
});
// update bound elements
updateBoundElements(element, scene.getNonDeletedElementsMap(), {
simultaneouslyUpdated: group,
});
return updatedEle;
});
});
};

Loading…
Cancel
Save