Skip to content

Fix warnings gap in properties panel (#29)#42

Merged
erikdarlingdata merged 1 commit intodevfrom
fix/issue-29-warnings-gap
Mar 5, 2026
Merged

Fix warnings gap in properties panel (#29)#42
erikdarlingdata merged 1 commit intodevfrom
fix/issue-29-warnings-gap

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Warning items were being added outside their Expander container (directly to PropertiesContent), leaving an empty Expander header with a visible gap below it
  • Both "Plan Warnings" (statement-level) and "Warnings" (operator-level) now create their own Expander with a StackPanel content, so items render inside the collapsible section with no gap

Test plan

  • Open a plan with warnings, click a node with warnings — no gap between header and first warning
  • Plan-level warnings (root node properties) also render without gap
  • Warnings sections are still collapsible via the Expander

Closes #29

🤖 Generated with Claude Code

Warning items were added outside their Expander (directly to PropertiesContent),
leaving an empty Expander header with a visible gap. Now both Plan Warnings and
operator Warnings create their own Expander with a StackPanel content so items
render inside the collapsible section with no gap.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 259a101 into dev Mar 5, 2026
@erikdarlingdata erikdarlingdata mentioned this pull request Mar 5, 2026
4 tasks
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