[Schnee111]

Fixes #80

What this changes

Visual only. Selection by row index, Esc/cancel handling and name validation are unchanged. The only script changes are the row markup and the footer text.

This replaces #81, which was closed without merging: the icon-tile grid was not the direction we wanted. This one keeps the row list and redesigns it, with no icons and no emoji.

common/rofi/rice-switch.rasi, rewritten:

• Panel: neutral frosted glass, rgba(30, 30, 34, 66%) with a 1 px rgba(255, 255, 255, 14%) border and a 24 px radius, 540 px wide. It no longer imports colors.rasi; the palette accent is not used.
• Header: "Switch Desktop" title (Inter Display Semi-Bold 16) on the left and a rounded search pill on the right, followed by a hairline. Typing still filters, and matches are shown in bold rather than underlined.
• Rows: two lines each, the name in semibold, then compositor · shell at 55 % alpha. Rows use 9px 14px padding with a 12 px radius, and the selection is a soft rgba(255, 255, 255, 10%) pill.
• Footer: behind a hairline, centred and dim: Switching logs you out · ↵ switch · esc cancel.

bin/rice-switch:

• A leading column holds a dot (80 %) for the current rice. Both lines of every row start with the same monospace ● glyph, invisible where unused, so the name and the subtitle stay aligned in a proportional font.
• The footer text is updated; the current rice is shown by the dot instead.

tests/rice-switch.sh: exactly one visible dot, on the end4 row.

Verification

• bash tests/run.sh: all test files pass. shellcheck -S warning is clean. rofi -dump-theme parses the theme.
• Negative control: with bin/rice-switch from main, the test fails with expected one visible dot, for the current rice.
• Live on end4 (1920x1080), running the real bin/rice-switch -n. Screenshots were checked by hand:
  • all 8 rows are visible with nothing cut off, and names line up with their subtitles;
  • the dot sits on end4, which is preselected;
  • Down moves the selection pill to ii-p3drovfx;
  • Escape exits 0 with no output.

Risk

Low. On rices without a blur rule for the rofi layer, the panel is a 66 % dark translucent box.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

[Schnee111]

Autonomous Code Review

Verdict: Changes Requested
Target Commit: dc7a317

Summary

The redesign is clean and well-structured, but the title textbox in the rofi theme uses an invalid property name that prevents the header text from rendering.

Logic & Invariants

• common/rofi/rice-switch.rasi:53: Rofi textbox widgets do not support the CSS-style content property. Static text must be declared using str: "Switch Desktop"; instead of content: "Switch Desktop";. Because content is ignored by rofi's theme parser, textbox-title renders as an empty widget without the title text.

Recommendations

• In tests/rice-switch.sh:86, consider using integer comparison (-eq 1) rather than string comparison (== 1) for wc -l output to avoid fragility across environments where wc emits leading whitespace.
• In bin/rice-switch:97, alpha="0%" or fgcolor="transparent" would be more idiomatic than alpha="1%" for a fully invisible placeholder gutter.

---

Shorekeeper Sentinel · Autonomous Quality Gate

[Schnee111]

Follow-ups in 4e399f4:

• textbox-title now uses str:. For the record, content: was not broken. rofi 2.0.0 documents both (man rofi-theme: "str/content: the string to display by this textbox"), and live screenshots showed the title with content. The switch only makes the review pass; the title still renders with str (checked live again).
• The dot count is compared with -eq 1.
• alpha="1%" stays. Pango's alpha range starts at 1, so 0% is not a valid value. The line now has a comment saying so.

[Schnee111]

Autonomous Code Review

Verdict: Approved
Target Commit: 4e399f4

Summary

The redesign of the rice-switcher menu as a frosted row list is clean and well-structured, with previous review feedback on theme properties and test assertions properly resolved.

---

Shorekeeper Sentinel · Autonomous Quality Gate