smig/excalidraw
1
0
Fork 0
mirror of https://github.com/excalidraw/excalidraw.git synced 2025-05-03 10:00:07 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix selecting text containers and containing frames together

Browse source
This commit is contained in:
Ryan Di 2025-04-03 15:52:00 +11:00
parent 57b4a098f3
commit 7a9df73e0b
1 changed files with 15 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

24
packages/excalidraw/lasso/index.ts
View file

@ -96,15 +96,6 @@ export class LassoTrail extends AnimatedTrail {
for (const [id] of Object.entries(nextSelectedElementIds)) { for (const [id] of Object.entries(nextSelectedElementIds)) {
const element = this.app.scene.getNonDeletedElement(id); const element = this.app.scene.getNonDeletedElement(id);
if (element && isFrameLikeElement(element)) {
const elementsInFrame = getFrameChildren(
this.app.scene.getNonDeletedElementsMap(),
element.id,
);
for (const child of elementsInFrame) {
delete nextSelectedElementIds[child.id];
}
}
if (element && isTextElement(element)) { if (element && isTextElement(element)) {
const container = getContainerElement( const container = getContainerElement(
@ -118,6 +109,21 @@ export class LassoTrail extends AnimatedTrail {
} }
} }
// remove all children of selected frames
for (const [id] of Object.entries(nextSelectedElementIds)) {
const element = this.app.scene.getNonDeletedElement(id);
if (element && isFrameLikeElement(element)) {
const elementsInFrame = getFrameChildren(
this.app.scene.getNonDeletedElementsMap(),
element.id,
);
for (const child of elementsInFrame) {
delete nextSelectedElementIds[child.id];
}
}
}
const nextSelection = selectGroupsForSelectedElements( const nextSelection = selectGroupsForSelectedElements(
{ {
editingGroupId: prevState.editingGroupId, editingGroupId: prevState.editingGroupId,