fix(app-iced): M77 r1 —— 菜单项 trailing 槽位靠右(视觉 r5 P1)

`item_row` 的 Row 早先是 `leading → label → trailing`,中间没有撑开的空间,于是
trailing(快捷键 / 子菜单箭头 / 勾)**紧贴 label 右边**,一列菜单里读不出一条竖线,
也与 M3「trailing 落在行右内边距上」不符。

改法(只动 `menu.rs` 的行装配):
- label 与 trailing 之间插一个 `space::horizontal()`(宽度 Fill),**只在有 trailing 时插**——
  没有 trailing 的项仍然左边起排,不被居中/右对齐;
- 把 Row 包进 `container(..).width(Fill).height(Fill).align_y(Center)`:Fill 空间要有宽度可撑
  (Q8 的教训:撑开靠容器,不靠子元素),顺带把内容垂直居中(`button` 只把内容放左上角);
- trailing 的右缘现在落在 item 右内边距上,与 leading 左内边距对称(12 = 4 内缩 + 8 button pad);
- 分组分隔线 / 禁用项的行为未动(同一函数的其它分支没改)。

渲染层断言(`probe/menu.rs`):带 trailing 的项,其 trailing 墨迹右缘 = `menu_right − 17`
(= 4 指示器内缩 + 12 内边距 + `Check` 字形在 24 viewBox 里约 5 的边界)±1。
变异(删掉那行 `space::horizontal()`)后实测 `menu_right=559、墨迹右缘=438 ⇒ 偏 121`,断言红;
修复后偏 **17**。重出 `menu-{light,dark}.png`,勾已贴行右缘成一条竖线。
This commit is contained in:
AskaEth committed 2026-11-02 07:45:39 +08:00
1 parent 21f2734981
commit 05fefc8a7b
2 files changed
+35 -1

No files matched your search

+12 -1
View File
@@ -287,6 +287,10 @@ fn item_row<'a, M: Clone + 'a>(
}
row = row.push(label);
if let Some(kind) = item.trailing {
// **撑开**:label 与 trailing 之间插一个水平 Fill,trailing 才会被推到行右内边距上
// (M3 里 trailing 是「快捷键 / 子菜单箭头 / 勾」,必须在一列菜单里成一条竖线)。
// 没有 trailing 时**不插**:label 仍在左边,不会被居中或右对齐。
row = row.push(iced::widget::space::horizontal());
row = row.push(layout::fixed(
geometry.icon,
geometry.icon,
@@ -294,6 +298,13 @@ fn item_row<'a, M: Clone + 'a>(
));
}
// 行必须撑满 item 宽度:上面的 Fill 空间要有宽度可撑(Q8 的教训:撑开靠容器,不靠子元素)。
// 顺带把内容垂直居中——`button` 只把内容放在 padding-box 的左上角。
let content = iced::widget::container(row)
.width(Length::Fill)
.height(Length::Fill)
.align_y(Alignment::Center);
let selected = item.selected;
let style = move |_theme: &iced::Theme, status: button::Status| {
let state = forced.unwrap_or_else(|| Interaction::from_status(status));
@@ -311,7 +322,7 @@ fn item_row<'a, M: Clone + 'a>(
};
let inset = ui.space(Space::X1);
let button = button(row)
let button = button(content)
.width(Length::Fill)
.height(Length::Fixed(geometry.item_height))
.padding(Padding {