Skip to content

fix: restore status indicator symbol default - #2442

Open
abhishekvtangod wants to merge 5 commits into
herdrdev:masterfrom
abhishekvtangod:user/restore-symbol-default
Open

fix: restore status indicator symbol default#2442
abhishekvtangod wants to merge 5 commits into
herdrdev:masterfrom
abhishekvtangod:user/restore-symbol-default

Conversation

@abhishekvtangod

@abhishekvtangod abhishekvtangod commented Aug 6, 2026

Copy link
Copy Markdown

Any reason why the default is dots? I'd prefer to keep symbols as the default.
They're more recognizable at a glance and make it easier to distinguish statuses quickly.

Summary

  • revert the ui.status_indicators default and state/test expectations to symbols
  • keep settings UI selection logic and sample config comments aligned
  • update docs/config reference to show symbols as the default and explain opting into dots
  • add abhishekvtangod to .github/APPROVED_CONTRIBUTORS so I can keep this PR updated

Testing

  • env ZIG="/opt/homebrew/opt/zig@0.15/bin/zig" cargo test status_indicator
  • env ZIG="/opt/homebrew/opt/zig@0.15/bin/zig" just test (fails: cross_area_agent_process_survives_detach_and_reattach panics when reattached frame lacks persisted working status)

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • ai-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 1b3324b0-13d9-4141-a383-4c4d89e6362b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@abhishekvtangod
abhishekvtangod force-pushed the user/restore-symbol-default branch from 2b9f563 to 4c4faf8 Compare August 7, 2026 05:08
@abhishekvtangod abhishekvtangod changed the title Restore status indicator symbol default fix: restore status indicator symbol default Aug 7, 2026
@abhishekvtangod
abhishekvtangod force-pushed the user/restore-symbol-default branch from c02ea94 to 0dbe277 Compare August 7, 2026 06:20
@abhishekvtangod
abhishekvtangod marked this pull request as draft August 7, 2026 06:21
@abhishekvtangod
abhishekvtangod marked this pull request as ready for review August 7, 2026 06:34
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.

1 participant