Merge pull request #7136 from penpot/niwinz-develop-fix-inconsistencies-on-text-shortcuts

🐛 Fix several inconsistencies and duplicated shortcuts
This commit is contained in:
Alejandro Alonso
2025-08-18 12:57:18 +02:00
committed by GitHub
2 changed files with 7 additions and 11 deletions

View File

@@ -213,12 +213,12 @@
:fn #(emit-when-no-readonly (dw/vertical-order-selected :bottom))}
:move-fast-up {:tooltip (ds/shift ds/up-arrow)
:command ["shift+up" "shift+alt+up" "alt+shift+up"]
:command ["shift+up"]
:subsections [:modify-layers]
:fn #(emit-when-no-readonly (dwt/move-selected :up true))}
:move-fast-down {:tooltip (ds/shift ds/down-arrow)
:command ["shift+down" "shift+alt+down" "alt+shift+down"]
:command ["shift+down"]
:subsections [:modify-layers]
:fn #(emit-when-no-readonly (dwt/move-selected :down true))}
@@ -233,12 +233,12 @@
:fn #(emit-when-no-readonly (dwt/move-selected :left true))}
:move-unit-up {:tooltip ds/up-arrow
:command ["up" "alt+up"]
:command ["up"]
:subsections [:modify-layers]
:fn #(emit-when-no-readonly (dwt/move-selected :up false))}
:move-unit-down {:tooltip ds/down-arrow
:command ["down" "alt+down"]
:command ["down"]
:subsections [:modify-layers]
:fn #(emit-when-no-readonly (dwt/move-selected :down false))}

View File

@@ -206,17 +206,13 @@
(filter cfh/text-shape?)
(not-empty))
;; Check if we're actually editing text content (not just selecting text shapes)
;; Handle both text-editor/v1 and text-editor/v2
editing-text? (if (features/active-feature? @st/state "text-editor/v2")
(some? (deref refs/workspace-v2-editor-state))
(some? (deref refs/workspace-editor)))
props (if (> (count text-shapes) 1)
(blend-props text-shapes props)
props)]
(when (and can-edit? (not read-only?) text-shapes editing-text?)
(when (and can-edit?
(not read-only?)
(some? text-shapes))
(st/emit! (dwu/start-undo-transaction undo-id))
(run! #(update-attrs % props) text-shapes)
(st/emit! (dwu/commit-undo-transaction undo-id)))))