github/penpot
1
0
Fork 0
mirror of https://github.com/penpot/penpot.git synced 2026-07-23 14:38:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

⚡ Skip flex/grid layout reflow for translation-only modifiers

Browse Source
This commit is contained in:
Elena Torro 2026-05-04 17:48:24 +02:00
parent 4ecf93cba6
commit 350d97d069
1 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
common/src/app/common/geom/modifiers.cljc
View File

@ -164,6 +164,17 @@
grid-layout? (ctl/grid-layout? parent)
parent? (or (cfh/group-like-shape? parent) (cfh/frame-shape? parent))
;; Pure-translation modifiers (the typical pointer-drag case)
;; don't change the parent's internal geometry, so children's
;; flex/grid positions relative to the parent are invariant —
;; `set-children-modifiers` already has an `only-move?` fast
;; path that just splats the same translation onto each child.
;; Skipping the flex/grid reflow here removes a per-pointer-
;; move `calc-layout-data` walk over every child in every
;; flex/grid ancestor of the dragged shape; that was the
;; bottleneck for layouts with many component children.
only-move? (ctm/only-move? modifiers)
transformed-parent-bounds (delay (gtr/transform-bounds @(get bounds parent-id) modifiers))
children-modifiers
@ -181,10 +192,10 @@
(and has-modifiers? parent? (not root?))
(set-children-modifiers children-modifiers objects bounds parent transformed-parent-bounds ignore-constraints)
flex-layout?
(and flex-layout? (not only-move?))
(set-flex-layout-modifiers children-layout objects bounds parent transformed-parent-bounds)
grid-layout?
(and grid-layout? (not only-move?))
(set-grid-layout-modifiers objects bounds parent transformed-parent-bounds)))))
(defn propagate-modifiers-constraints