Skip to content

feat(file): add per-file statusText with accessible live region#4157

Merged
Kiarokh merged 1 commit into
mainfrom
feat/file-status-text
Jul 6, 2026
Merged

feat(file): add per-file statusText with accessible live region#4157
Kiarokh merged 1 commit into
mainfrom
feat/file-status-text

Conversation

@Befkadu1

@Befkadu1 Befkadu1 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add FileInfo.statusText — a short label (e.g. Uploading…, failed, 45%) that replaces the size badge on the file chip when set.
  • Announce the current status to assistive tech via a visually-hidden role="status" aria-live="polite" region.
  • Refactor badge resolution out of getChipArray() into a getBadge() helper using early returns.

Test plan

  • Set value.statusText on a limel-file and confirm the chip badge shows the text instead of the file size.
  • Clear statusText and confirm the size badge returns.
  • With a screen reader, confirm status changes are announced politely.

Note

Stacked on top of #4154 (file-loading-progress). Base should be retargeted to main once that PR merges.

🤖 Generated with Claude Code

@Befkadu1 Befkadu1 requested a review from a team as a code owner July 2, 2026 12:38
@coderabbitai

coderabbitai Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@Kiarokh, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 57 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 52e2e450-460b-4e95-a9f6-8660d849ad97

📥 Commits

Reviewing files that changed from the base of the PR and between d6ae225 and 2ac8b66.

⛔ Files ignored due to path filters (1)
  • etc/lime-elements.api.md is excluded by !etc/lime-elements.api.md
📒 Files selected for processing (3)
  • src/components/file/examples/file-per-file-status.tsx
  • src/components/file/file.tsx
  • src/global/shared-types/file.types.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/file-status-text

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

Documentation has been published to https://lundalogik.github.io/lime-elements/versions/PR-4157/

@Kiarokh Kiarokh force-pushed the file-loading-progress branch from 78fc20a to f45838f Compare July 2, 2026 14:13
Base automatically changed from file-loading-progress to main July 2, 2026 14:18
@Kiarokh Kiarokh force-pushed the feat/file-status-text branch 2 times, most recently from 4cafeb9 to 2111bb4 Compare July 6, 2026 09:26
Add an optional `statusText` field to `FileInfo` that replaces the file
size badge on the chip and is announced to assistive technologies via a
polite `role="status"` live region. Extract badge resolution into a
`getBadge()` helper.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Kiarokh Kiarokh force-pushed the feat/file-status-text branch from 2111bb4 to 2ac8b66 Compare July 6, 2026 09:28
@Kiarokh Kiarokh enabled auto-merge (rebase) July 6, 2026 09:32
@Kiarokh Kiarokh disabled auto-merge July 6, 2026 09:33
@Kiarokh Kiarokh merged commit 7b1b305 into main Jul 6, 2026
18 checks passed
@Kiarokh Kiarokh deleted the feat/file-status-text branch July 6, 2026 10:59
@lime-opensource

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 39.40.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants