Skip to content

Name Clementine's computer as Clementine does - #325

Merged
hatstand merged 2 commits into
masterfrom
claude/nifty-ramanujan-c2i3zy
Oct 5, 2026
Merged

hatstand merged 2 commits into
masterfrom
claude/nifty-ramanujan-c2i3zy

Conversation

@hatstand

@hatstand hatstand commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

"Play on" called Clementine's own output "This computer", which reads as the phone. Clementine now sends its computer's host name (or a --remote-name override), in clementine-player/Clementine#7586.

  • Play on: shows the name Clementine sends, or "Clementine" with the computer icon when it sends none.
  • Connection chip and connection sheet heading: show the same name when Clementine sends it, and otherwise fall back to what they showed before (the network's name for the address, or the address).

Same change on iOS: clementine-player/iOS-Remote, branch claude/nifty-ramanujan-c2i3zy.

A Clementine from before clementine-player/Clementine#7586 still sends "This computer", and that's what shows until it's updated.

Tests: OutputSheetTest (name and fallback) and RemoteRepositoryTest (computerName). Not built here: the session had no Android SDK, so CI is the first build.

The "(this tablet)" wording is a separate PR from claude/android-tablet-output-label; the two touch the same lines of OutputSheet.kt, so whichever merges second needs a small conflict resolved.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CTf1Z6GAbDbQeNJ5BkewC1


Generated by Claude Code

claude added 2 commits October 5, 2026 16:17
The output sheet called Clementine's own output "This computer", which
reads as the phone. Clementine now sends its computer's host name, so the
sheet shows that, or "Clementine" when it doesn't say.

Release-note: "Play on" names Clementine's computer instead of saying "This computer".
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CTf1Z6GAbDbQeNJ5BkewC1
…connection sheet

The connection chip and the connection sheet's heading showed the name
the network gave Clementine's address, or the address itself, which
needn't match the name "Play on" shows. They now show the name Clementine
sends, when it sends one.

Release-note: The top of the screen and the connection details name Clementine's computer as Clementine does.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CTf1Z6GAbDbQeNJ5BkewC1
@hatstand
hatstand enabled auto-merge (rebase) October 5, 2026 16:29
@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

Store screenshots

From run 37340822106, against clementine-it. Left: the store listing of the latest release, v13.7, which shows the dark theme. Right: this pull request, dark and light.

Screen v13.7 This PR, dark This PR
1_player
2_library
2_library_album
3_playlist
4_search
5_connection
6_connect
10_internet –
11_internet_radio –
settings –

hatstand commented Oct 5, 2026

Copy link
Copy Markdown
Contributor Author

device failed before any test ran. The emulator never came up (adb: device offline while waiting for boot), so connectedFdroidDebugAndroidTest couldn't even uninstall or install the APK. No test result came from this PR's code; device (Android 17), build, integration and screenshots all passed on the same commit.

I've re-run the failed job once.


Generated by Claude Code

@hatstand
hatstand merged commit b5be943 into master Oct 5, 2026
9 of 10 checks passed
@hatstand
hatstand deleted the claude/nifty-ramanujan-c2i3zy branch October 5, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants