You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Why: a long asset label was cut off inside the fixed-height inspector pill, and
the text label had no hover hint, so the full label was unreadable.
What:
- Keep pill text on one line and end it with an ellipsis.
- Expose the full label text as a hover hint.
- Hold the remove button at its own width beside the clipped text.
Validation: new hosted regression fails on 10a5472 with an empty hint;
InspectorLabelPillsViewTests 3/3, LabelFilter 40/40, ObjectRowLayout 2/2,
ResponsivePaneLayout 9/9, PopoverFocus 11/11; check:gate 16/16.
Refs #312
No actionable comments were generated in the recent review.
Walkthrough
Inspector label pills now truncate long labels with an ellipsis and show the full label in a tooltip. The change also adds an editor test and updates the changelog and label editing documentation.
Changes: Inspector label pill display
Layer / File(s)
Summary
Label pill display and validation Editor/DataVisualizer/Styles/DataVisualizerStyles.uss, Editor/DataVisualizer/UI/InspectorLabelPillsView.cs, Tests/Editor/InspectorLabelPillsViewTests.cs, docs/search-and-filter.md, CHANGELOG.md
Label text stays on one line and uses an ellipsis when it exceeds the pill width. The remove button keeps its intrinsic width, and the label text tooltip shows the full label. The test checks the layout and tooltip. The documentation and changelog describe the behavior.
Issue 312 — The PR identifies this issue as addressed. Its description includes hover hints for clipped or ambiguous labels, which relates to the full-label tooltip added here.
Long labels now remain readable through ellipsis and hover tooltips, with no concrete merge-blocking risk identified.
Pre-merge checks: 4 · 1
Failed checks: 1 warning
Check name
Status
Explanation
Resolution
Docstring Coverage
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed checks: 4 passed
Check name
Status
Explanation
Title check
The title clearly and concisely describes the main change: improving readability for long inspector labels.
Description check
The description explains the problem and lists the three main changes. It references issue #312. It omits the Type of Change and Checklist sections from the template, but the core description is compl…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Description check
Explanation
The description explains the problem and lists the three main changes. It references issue #312. It omits the Type of Change and Checklist sections from the template, but the core description is complete.
Full details: Docstring Coverage
Explanation
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 2 functions across 3 files.
Resolution
Write docstrings for the functions missing them to satisfy the coverage threshold.
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.
CodeRabbit Verify
Run CodeRabbit Verify to test the changes in this pull request.
Recent review details
Context from checks skipped due to timeout: 1
GitHub Check: CodeRabbit
Additional context used
Code guidelines: 1
.llm/context.md — auto-discovered
Path-based instructions: 1
Applicability reference: AGENTS.md: # Repository Guidelines See the [AI Agent Guidelines](./.llm/context.md) for all AI agent guidelines.
CodeRabbit inference engine (.llm/context.md)
Files:
Editor/DataVisualizer/DataVisualizer.cs
Tests/Editor/InspectorLabelPillsViewTests.cs
Learnings: 2
Common learnings
Learnt from: CR
Repo: Ambiguous-Interactive/DxVisualizer PR: 0
File: .llm/context.md:0-0
Timestamp: 2026-10-31T05:33:39.058Z
Learning: Applicability reference:
AGENTS.md: # Repository Guidelines
See the [AI Agent Guidelines](./.llm/context.md) for all AI agent guidelines.
Source excerpt:
# LLM Agent Instructions
## Critical Rules
### C# and Unity
2. Format every modified `.cs` file with CSharpier 1.1.2 (print width 180 in `.csharpierrc`; `dotnet tool run csharpier -- format <paths>`) and every modified `.md` file with
Prettier (print width 180 in `.prettierrc.json`; `npm run format:md`); treat both as gates. Generated and skill Markdown is exempt (`.prettierignore`), and table cells align to
their widest cell, so table lines may exceed the width.
Learnt from: CR
Repo: Ambiguous-Interactive/DxVisualizer PR: 0
File: .llm/context.md:0-0
Timestamp: 2026-11-02T20:14:27.735Z
Learning: Applies to **/*.cs : Applicability reference:
AGENTS.md: # Repository Guidelines
See the [AI Agent Guidelines](./.llm/context.md) for all AI agent guidelines.
Source excerpt:
# LLM Agent Instructions
## Critical Rules
### C# and Unity
2. Format every modified `.cs` file with CSharpier 1.1.2 (print width 180 in `.csharpierrc`; `dotnet tool run csharpier -- format <paths>`) and every modified `.md` file with
Prettier (print width 180 in `.prettierrc.json`; `npm run format:md`); treat both as gates. Generated and skill Markdown is exempt (`.prettierignore`), and table cells align to
their widest cell, so table lines may exceed the width.
Learning: 2026-10-31T05:33:39.058Z
Learnt from: CR
Repo: Ambiguous-Interactive/DxVisualizer PR: 0
File: .llm/context.md:0-0
Timestamp: 2026-10-31T05:33:39.058Z
Learning: Applies to **/*.cs : Applicability reference:
AGENTS.md: # Repository Guidelines
See the [AI Agent Guidelines](./.llm/context.md) for all AI agent guidelines.
Source excerpt:
# LLM Agent Instructions
## Critical Rules
### C# and Unity
29. Use `//` for standalone single-line comments and tool control directives only. Write ordinary multi-line explanations as one indented `/* ... */` block. Keep `///` XML
documentation line-oriented. The C# source lint enforces this rule.
Why: review found that the same paperclip-length labels on the filter pills
still had no hint, so a clipped active label was unreadable.
What:
- Carry the full label on the AND/OR pill container.
- Assert the rendered label reports elision.
Validation: LabelFilter 40/40, InspectorLabelPillsViewTests 3/3;
check:gate 16/16.
Refs #312
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
DISCLOSURE: LLM-GENERATED TEXT
Why: A long asset label was cut off inside the fixed-height inspector pill and had no hover hint, so the full label was unreadable.
What:
Refs #312