aleleba/deckcraft
1
0
Fork 0
mirror of https://github.com/storytold/deckcraft.git synced 2026-10-25 19:17:06 -06:00
Code Issues Packages Projects Releases Wiki Activity

Fix up merged PRs: the palette test skips UI commands that need a value (#440 hides them)

Browse Source 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Brandon Thomas
2026-10-14 14:05:00 -04:00
co-authored by Claude Opus 5.5
parent 7266b30018
commit 31e22977f7
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/ui-egui/src/menus.rs
+5 -1
View File
@@ -305,7 +305,11 @@ mod tests {
assert_eq!(exports[0].0, "app.exportDialog");
}
// Open…, Save As…, Audio/Video from File… and Notes had the same duplicates: every UI command is the only entry with its label.
for (id, label, _, _) in crate::UI_COMMANDS {
// Commands that need a value (Set Zoom's percent) are left out of the palette.
for (id, label, _, params) in crate::UI_COMMANDS {
if deckcraft_engine::cmd::needs_arguments(params) {
continue;
}
let same: Vec<_> = palette_items(&session, &label.to_lowercase()).into_iter().filter(|(_, l, _)| l == label).collect();
assert_eq!(same.len(), 1, "{label}");
assert_eq!(same[0].0, *id, "{label}");