Skip to content

Fix: cut a tile label at the end of its own name - #174

Merged
SandObserver merged 1 commit into
mainfrom
fix/label-truncation-direction
Aug 30, 2026
Merged

SandObserver merged 1 commit into
mainfrom
fix/label-truncation-direction

Conversation

@SandObserver

@SandObserver SandObserver commented Aug 30, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Set dir="auto" on .ilabel, .dyn-mob-label, .dyn-fold-label and .dyn-fold-inner-label at each creation site, so a label takes its direction from the name it holds and ellipsises at that name's end.
  • unicode-bidi:plaintext moves the cut in Blink but is ignored by WebKit, where the label was clipped at both ends. The labels are centred, so dir cannot move them off their icons.
  • Covers tile labels, folder labels and names inside an open folder, on the desktop and phone layouts.
  • Add a check that every one of those labels carries the attribute, and a check that all four rules stay centred, which is what makes the attribute safe.

Testing
All green, plus the Docker build. Added 2 tests, verified to fail without the change, and confirmed the rendering in WebKit and Blink against a container.

Checklist

  • Stays within the project constraints: one container, no runtime dependencies, no build step (see CONTRIBUTING.md).
  • Tests added or updated for new logic, and the full suite, lint, and typechecks pass.
  • Docs updated if behavior or configuration changed.
  • CHANGELOG.md updated under Unreleased if the change is user-visible.

A tile label took its direction from the page, so a Latin name on a
right-to-left dashboard was truncated at its start. The labels are
centred, so taking the direction from the name moves no text.
@SandObserver SandObserver added bug Something isn't working ui/ux UI, UX, and accessibility only fix labels Aug 30, 2026
@SandObserver
SandObserver merged commit f5fadea into main Aug 30, 2026
4 checks passed
@SandObserver
SandObserver deleted the fix/label-truncation-direction branch August 30, 2026 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ui/ux UI, UX, and accessibility only fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant