Compare commits

..
Author SHA1 Message Date
molluskandClaude Opus 4.8 c0c1969332 style(music): theme-colored skip glyphs in player bar + drawer
windows-build / windows-build (push) Has been cancelled
cargo-deny / cargo-deny (push) Has been cancelled
The skip-back/forward buttons rendered orange in every theme because the
emoji glyphs ⏮/⏭ (U+23EE/U+23ED) are drawn by the system color-emoji font,
which ignores the button's text color. Replace them with |◀ / ▶| built from
the text-presentation triangles ◀/▶ (U+25C0/U+25B6) — the same family the
play button already uses — so they honor .color() and follow the active
theme like the play button does. Applies to both the now-playing player bar
and the full music drawer panel. Pure visual change; no behavior change.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-28 17:33:32 -04:00
d155091eed style(music): center the transport controls on the player bar
The transport buttons were pushed to the far right because the
now-playing label had width(Fill). Regroup the bar into three sections
— left(Fill) identity+label, centered transport, right(Fill) position +
expand — so the controls sit in the middle. Pure widget regrouping; no
message, config, or behavior change.

Implemented by Codex (gpt-5.5), reviewed + gates re-run by Claude.

Co-Authored-By: Codex <codex@openai.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-28 17:07:59 -04:00
+49 -31
View File
@@ -5567,7 +5567,7 @@ fn view(state: &AppState) -> Element<'_, AppMessage> {
.unwrap_or_else(|| "--:--".to_string());
column![
row![
button(text("").size(13))
button(text("|◀").size(13))
.on_press(AppMessage::MusicPrev)
.style(b_style(color_surface, color_blue, color_text, 6.0))
.padding(7),
@@ -5575,7 +5575,7 @@ fn view(state: &AppState) -> Element<'_, AppMessage> {
.on_press(AppMessage::MusicPlayPause)
.style(b_style(color_blue, color_lavender, color_crust, 6.0))
.padding(7),
button(text("").size(13))
button(text("▶|").size(13))
.on_press(AppMessage::MusicNext)
.style(b_style(color_surface, color_blue, color_text, 6.0))
.padding(7),
@@ -5729,35 +5729,53 @@ fn view(state: &AppState) -> Element<'_, AppMessage> {
.unwrap_or_else(|| "--:--".to_string());
let player_bar = container(
row![
text("").size(18).color(color_blue),
text(now_playing_label(listening_peer_name, current_track_name))
.size(13)
.color(color_text)
.width(iced::Length::Fill),
button(text("").size(13))
.on_press(AppMessage::MusicPrev)
.style(b_style(color_surface, color_blue, color_text, 6.0))
.padding(7),
button(text(music_bar_play_label).size(13))
.on_press(AppMessage::MusicPlayPause)
.style(b_style(color_blue, color_lavender, color_crust, 6.0))
.padding(7),
button(text("").size(13))
.on_press(AppMessage::MusicNext)
.style(b_style(color_surface, color_blue, color_text, 6.0))
.padding(7),
text(format!("{elapsed} / {duration}"))
.size(11)
.color(color_subtext),
button(text(if state.playlist_drawer_open { "" } else { "" }).size(13))
.on_press(AppMessage::TogglePlaylistDrawer)
.style(b_style(
if state.playlist_drawer_open { color_blue } else { color_surface },
color_blue,
if state.playlist_drawer_open { color_crust } else { color_text },
6.0,
))
.padding(7),
container(
row![
text("").size(18).color(color_blue),
text(now_playing_label(listening_peer_name, current_track_name))
.size(13)
.color(color_text),
]
.spacing(8)
.align_y(iced::alignment::Vertical::Center)
)
.width(iced::Length::Fill),
row![
button(text("|◀").size(13))
.on_press(AppMessage::MusicPrev)
.style(b_style(color_surface, color_blue, color_text, 6.0))
.padding(7),
button(text(music_bar_play_label).size(13))
.on_press(AppMessage::MusicPlayPause)
.style(b_style(color_blue, color_lavender, color_crust, 6.0))
.padding(7),
button(text("▶|").size(13))
.on_press(AppMessage::MusicNext)
.style(b_style(color_surface, color_blue, color_text, 6.0))
.padding(7),
]
.spacing(8)
.align_y(iced::alignment::Vertical::Center),
container(
row![
text(format!("{elapsed} / {duration}"))
.size(11)
.color(color_subtext),
button(text(if state.playlist_drawer_open { "" } else { "" }).size(13))
.on_press(AppMessage::TogglePlaylistDrawer)
.style(b_style(
if state.playlist_drawer_open { color_blue } else { color_surface },
color_blue,
if state.playlist_drawer_open { color_crust } else { color_text },
6.0,
))
.padding(7),
]
.spacing(8)
.align_y(iced::alignment::Vertical::Center)
)
.width(iced::Length::Fill)
.align_x(iced::alignment::Horizontal::Right),
]
.spacing(8)
.align_y(iced::alignment::Vertical::Center)