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
+23 -5
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),
@@ -5728,13 +5728,20 @@ fn view(state: &AppState) -> Element<'_, AppMessage> {
.map(format_clip_time)
.unwrap_or_else(|| "--:--".to_string());
let player_bar = container(
row![
container(
row![
text("").size(18).color(color_blue),
text(now_playing_label(listening_peer_name, current_track_name))
.size(13)
.color(color_text)
.color(color_text),
]
.spacing(8)
.align_y(iced::alignment::Vertical::Center)
)
.width(iced::Length::Fill),
button(text("").size(13))
row![
button(text("|◀").size(13))
.on_press(AppMessage::MusicPrev)
.style(b_style(color_surface, color_blue, color_text, 6.0))
.padding(7),
@@ -5742,10 +5749,15 @@ 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),
]
.spacing(8)
.align_y(iced::alignment::Vertical::Center),
container(
row![
text(format!("{elapsed} / {duration}"))
.size(11)
.color(color_subtext),
@@ -5762,6 +5774,12 @@ fn view(state: &AppState) -> Element<'_, AppMessage> {
.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)
)
.style(c_style(color_mantle, color_surface, 8.0))
.padding(10)
.width(iced::Length::Fill)