Apply review fixes

- crates/ui-egui/src/app/clipboard.rs
- docs/egui-037-migration-plan.md
This commit is contained in:
autocraft-bot[bot] committed 2026-11-12 05:37:36 +00:00
1 parent 630b5dfc43
commit 6b9c47fdb2
2 files changed
+3 -4

No files matched your search

+1 -1
View File
@@ -59,7 +59,7 @@ impl DesignApp {
if !typing
&& !ours
&& !copied_here
&& text.is_none_or(is_bare_address)
&& text.is_none_or(|t| t.trim().is_empty() || is_bare_address(t))
&& let Some(image) = self.services.clipboard_image.as_mut().and_then(|f| f())
{
return Pasted::Image(image);
+2 -3
View File
@@ -494,9 +494,8 @@ Clipboard:
freshness, owner/seat changes, empty/absent offers, self-paste, canceled/slow transfers,
password exclusion and no unchanged-selection reclaim on wlroots/KWin/Mutter. Native
capability evidence is still required; compile-only is not acceptance.
- `Event::PasteImage` is new but DesignCraft has no image-paste contract. Leave it
unhandled rather than inventing a placement path. Add a test that it does not panic and
does not leak into text paste.
- `Event::PasteImage` (an image on the clipboard with no text) pastes through `edit.pasteImage`: in the
selected empty frame, else centered in the view, and never into text being edited.
Keyboard layouts: after the fork pin, `RawInput::keyboard_layout_is_latin` and
`RawInput::latin_layout_chars` (E19) must preserve tool and command shortcuts on Russian,