fix(ui): tighten oversized padding and add visible borders to dropdown triggers

[Tryanks]

Feedback addressed

Many components had unnecessarily large padding, hurting visual density:

1. ChatView: gap between the message stream and the composer — timeline py_6 → py_4, composer wrapper pt_2 → pt_1 (32px → 20px combined)
2. Composer → worktree/branch row: removed a pt_2 stacking on the parent's gap_2; wrapper pb_3 → pb_2 (16px → 8px)
3. Top-right Commit / Open menus: shared popover panel padding p_3 → p_1 (16px → 8px inset)
4. Model / reasoning-effort / permission popovers: covered by the same shared popover change
5. Settings select menus: same shared change; the two prose-card popovers (update notices) explicitly keep p_3
6. Dropdown triggers lacked a visible border: Ghost buttons now use theme.border as their border color — still zero-width by default, visible only with .outline(), which was applied to every value+chevron select trigger (settings selects, composer model/effort/permission pickers, branch/workspace pickers, diff turn/base selects, provider model picker, orchestrate child-approval dropdown)
7. Shared context menu (markdown link right-click, sidebar, terminal): dropped the gap_1 between items; separators keep my_1 breathing room

Deliberately not bordered: split-button chevrons (already inside a bordered group) and bare icon-only ghost buttons.

Verification

• cargo fmt --all --check — clean
• cargo clippy --workspace --all-targets --locked -- -D warnings — clean
• cargo test --workspace --locked — 878 passed, 0 failed

[Tryanks]

Feedback addressed

Many components had unnecessarily large padding, hurting visual density:

1. ChatView: gap between the message stream and the composer — timeline py_6 → py_4, composer wrapper pt_2 → pt_1 (32px → 20px combined)
2. Composer → worktree/branch row: removed a pt_2 stacking on the parent's gap_2; wrapper pb_3 → pb_2 (16px → 8px)
3. Top-right Commit / Open menus: shared popover panel padding p_3 → p_1 (16px → 8px inset)
4. Model / reasoning-effort / permission popovers: covered by the same shared popover change
5. Settings select menus: same shared change; the two prose-card popovers (update notices) explicitly keep p_3
6. Dropdown triggers lacked a visible border: Ghost buttons now use theme.border as their border color — still zero-width by default, visible only with .outline(), which was applied to every value+chevron select trigger (settings selects, composer model/effort/permission pickers, branch/workspace pickers, diff turn/base selects, provider model picker, orchestrate child-approval dropdown)
7. Shared context menu (markdown link right-click, sidebar, terminal): dropped the gap_1 between items; separators keep my_1 breathing room

Deliberately not bordered: split-button chevrons (already inside a bordered group) and bare icon-only ghost buttons.

Verification

• cargo fmt --all --check — clean
• cargo clippy --workspace --all-targets --locked -- -D warnings — clean
• cargo test --workspace --locked — 878 passed, 0 failed

[Tryanks]

Feedback addressed

Many components had unnecessarily large padding, hurting visual density:

1. ChatView: gap between the message stream and the composer — timeline py_6 → py_4, composer wrapper pt_2 → pt_1 (32px → 20px combined)
2. Composer → worktree/branch row: removed a pt_2 stacking on the parent's gap_2; wrapper pb_3 → pb_2 (16px → 8px)
3. Top-right Commit / Open menus: shared popover panel padding p_3 → p_1 (16px → 8px inset)
4. Model / reasoning-effort / permission popovers: covered by the same shared popover change
5. Settings select menus: same shared change; the two prose-card popovers (update notices) explicitly keep p_3
6. Dropdown triggers lacked a visible border: Ghost buttons now use theme.border as their border color — still zero-width by default, visible only with .outline(), which was applied to every value+chevron select trigger (settings selects, composer model/effort/permission pickers, branch/workspace pickers, diff turn/base selects, provider model picker, orchestrate child-approval dropdown)
7. Shared context menu (markdown link right-click, sidebar, terminal): dropped the gap_1 between items; separators keep my_1 breathing room

Deliberately not bordered: split-button chevrons (already inside a bordered group) and bare icon-only ghost buttons.

Verification

• cargo fmt --all --check — clean
• cargo clippy --workspace --all-targets --locked -- -D warnings — clean
• cargo test --workspace --locked — 878 passed, 0 failed

[Tryanks]

Feedback addressed

Many components had unnecessarily large padding, hurting visual density:

1. ChatView: gap between the message stream and the composer — timeline py_6 → py_4, composer wrapper pt_2 → pt_1 (32px → 20px combined)
2. Composer → worktree/branch row: removed a pt_2 stacking on the parent's gap_2; wrapper pb_3 → pb_2 (16px → 8px)
3. Top-right Commit / Open menus: shared popover panel padding p_3 → p_1 (16px → 8px inset)
4. Model / reasoning-effort / permission popovers: covered by the same shared popover change
5. Settings select menus: same shared change; the two prose-card popovers (update notices) explicitly keep p_3
6. Dropdown triggers lacked a visible border: Ghost buttons now use theme.border as their border color — still zero-width by default, visible only with .outline(), which was applied to every value+chevron select trigger (settings selects, composer model/effort/permission pickers, branch/workspace pickers, diff turn/base selects, provider model picker, orchestrate child-approval dropdown)
7. Shared context menu (markdown link right-click, sidebar, terminal): dropped the gap_1 between items; separators keep my_1 breathing room

Deliberately not bordered: split-button chevrons (already inside a bordered group) and bare icon-only ghost buttons.

Verification

• cargo fmt --all --check — clean
• cargo clippy --workspace --all-targets --locked -- -D warnings — clean
• cargo test --workspace --locked — 878 passed, 0 failed

[Tryanks]

Feedback addressed

Many components had unnecessarily large padding, hurting visual density:

1. ChatView: gap between the message stream and the composer — timeline py_6 → py_4, composer wrapper pt_2 → pt_1 (32px → 20px combined)
2. Composer → worktree/branch row: removed a pt_2 stacking on the parent's gap_2; wrapper pb_3 → pb_2 (16px → 8px)
3. Top-right Commit / Open menus: shared popover panel padding p_3 → p_1 (16px → 8px inset)
4. Model / reasoning-effort / permission popovers: covered by the same shared popover change
5. Settings select menus: same shared change; the two prose-card popovers (update notices) explicitly keep p_3
6. Dropdown triggers lacked a visible border: Ghost buttons now use theme.border as their border color — still zero-width by default, visible only with .outline(), which was applied to every value+chevron select trigger (settings selects, composer model/effort/permission pickers, branch/workspace pickers, diff turn/base selects, provider model picker, orchestrate child-approval dropdown)
7. Shared context menu (markdown link right-click, sidebar, terminal): dropped the gap_1 between items; separators keep my_1 breathing room

Deliberately not bordered: split-button chevrons (already inside a bordered group) and bare icon-only ghost buttons.

Verification

• cargo fmt --all --check — clean
• cargo clippy --workspace --all-targets --locked -- -D warnings — clean
• cargo test --workspace --locked — 878 passed, 0 failed

[Tryanks]

Feedback addressed

Many components had unnecessarily large padding, hurting visual density:

1. ChatView: gap between the message stream and the composer — timeline py_6 → py_4, composer wrapper pt_2 → pt_1 (32px → 20px combined)
2. Composer → worktree/branch row: removed a pt_2 stacking on the parent's gap_2; wrapper pb_3 → pb_2 (16px → 8px)
3. Top-right Commit / Open menus: shared popover panel padding p_3 → p_1 (16px → 8px inset)
4. Model / reasoning-effort / permission popovers: covered by the same shared popover change
5. Settings select menus: same shared change; the two prose-card popovers (update notices) explicitly keep p_3
6. Dropdown triggers lacked a visible border: Ghost buttons now use theme.border as their border color — still zero-width by default, visible only with .outline(), which was applied to every value+chevron select trigger (settings selects, composer model/effort/permission pickers, branch/workspace pickers, diff turn/base selects, provider model picker, orchestrate child-approval dropdown)
7. Shared context menu (markdown link right-click, sidebar, terminal): dropped the gap_1 between items; separators keep my_1 breathing room

Deliberately not bordered: split-button chevrons (already inside a bordered group) and bare icon-only ghost buttons.

Verification

• cargo fmt --all --check — clean
• cargo clippy --workspace --all-targets --locked -- -D warnings — clean
• cargo test --workspace --locked — 878 passed, 0 failed

[Tryanks]

Feedback addressed

Many components had unnecessarily large padding, hurting visual density:

1. ChatView: gap between the message stream and the composer — timeline py_6 → py_4, composer wrapper pt_2 → pt_1 (32px → 20px combined)
2. Composer → worktree/branch row: removed a pt_2 stacking on the parent's gap_2; wrapper pb_3 → pb_2 (16px → 8px)
3. Top-right Commit / Open menus: shared popover panel padding p_3 → p_1 (16px → 8px inset)
4. Model / reasoning-effort / permission popovers: covered by the same shared popover change
5. Settings select menus: same shared change; the two prose-card popovers (update notices) explicitly keep p_3
6. Dropdown triggers lacked a visible border: Ghost buttons now use theme.border as their border color — still zero-width by default, visible only with .outline(), which was applied to every value+chevron select trigger (settings selects, composer model/effort/permission pickers, branch/workspace pickers, diff turn/base selects, provider model picker, orchestrate child-approval dropdown)
7. Shared context menu (markdown link right-click, sidebar, terminal): dropped the gap_1 between items; separators keep my_1 breathing room

Deliberately not bordered: split-button chevrons (already inside a bordered group) and bare icon-only ghost buttons.

Verification

• cargo fmt --all --check — clean
• cargo clippy --workspace --all-targets --locked -- -D warnings — clean
• cargo test --workspace --locked — 878 passed, 0 failed

[Tryanks]

Feedback addressed

Many components had unnecessarily large padding, hurting visual density:

1. ChatView: gap between the message stream and the composer — timeline py_6 → py_4, composer wrapper pt_2 → pt_1 (32px → 20px combined)
2. Composer → worktree/branch row: removed a pt_2 stacking on the parent's gap_2; wrapper pb_3 → pb_2 (16px → 8px)
3. Top-right Commit / Open menus: shared popover panel padding p_3 → p_1 (16px → 8px inset)
4. Model / reasoning-effort / permission popovers: covered by the same shared popover change
5. Settings select menus: same shared change; the two prose-card popovers (update notices) explicitly keep p_3
6. Dropdown triggers lacked a visible border: Ghost buttons now use theme.border as their border color — still zero-width by default, visible only with .outline(), which was applied to every value+chevron select trigger (settings selects, composer model/effort/permission pickers, branch/workspace pickers, diff turn/base selects, provider model picker, orchestrate child-approval dropdown)
7. Shared context menu (markdown link right-click, sidebar, terminal): dropped the gap_1 between items; separators keep my_1 breathing room

Deliberately not bordered: split-button chevrons (already inside a bordered group) and bare icon-only ghost buttons.

Verification

• cargo fmt --all --check — clean
• cargo clippy --workspace --all-targets --locked -- -D warnings — clean
• cargo test --workspace --locked — 878 passed, 0 failed