Fix camera name label sizing across viewers - #34
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@codex review |
|
Codex Review: Didn't find any major issues. Hooray! Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
Camera name overlays should occupy only the width needed for their text and remain on one line across viewer clients.
Removes the Linux label’s forced 90% tile width, uses wrap-content sizing with single-line ellipsis on Android, and explicitly enables single-line ellipsis on Windows. Windows also renders ampersands literally in camera names.
Validation:
git diff --checkpassed. Local builds could not run because Rust and Java are unavailable; CI will validate the change.