Skip to content

Comments

BUG-T16: Tool Distribution (Pie) widget is cropped at medium widths#99

Merged
SoundBlaster merged 8 commits intomainfrom
feature/BUG-T16-tool-distribution-pie-responsive-crop
Feb 20, 2026
Merged

BUG-T16: Tool Distribution (Pie) widget is cropped at medium widths#99
SoundBlaster merged 8 commits intomainfrom
feature/BUG-T16-tool-distribution-pie-responsive-crop

Conversation

@SoundBlaster
Copy link
Owner

@SoundBlaster SoundBlaster commented Feb 20, 2026

Description

Fixes BUG-T16 by preventing medium-width cropping of the Tool Distribution (Pie) widget in the Web UI dashboard.

Implemented changes:

  • Updated chart container CSS to avoid clipping at medium breakpoints.
  • Added responsive doughnut legend positioning logic:
    • right on wide layouts
    • bottom on medium/smaller layouts (<= 1280px)
  • Added regression coverage for responsive doughnut legend behavior.
  • Archived FLOW artifacts for BUG-T16 (plan, validation, review).

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • CI/CD improvement

Quality Gates

Before submitting, ensure all quality gates pass:

make check

Or run individually:

  • make test - All tests pass with ≥90% coverage
  • make lint - No linting errors
  • make format - Code is properly formatted
  • make typecheck - Type checking passes
  • make doccheck - Documentation is synced with DocC (if docs changed)

Documentation Sync

If you modified files in docs/, ensure corresponding DocC files are also updated:

docs/ file DocC file
docs/installation.md mcpbridge-wrapper.docc/Installation.md
docs/cursor-setup.md mcpbridge-wrapper.docc/CursorSetup.md
docs/claude-setup.md mcpbridge-wrapper.docc/ClaudeCodeSetup.md
docs/codex-setup.md mcpbridge-wrapper.docc/CodexCLISetup.md
docs/troubleshooting.md mcpbridge-wrapper.docc/Troubleshooting.md
docs/architecture.md mcpbridge-wrapper.docc/Architecture.md
docs/environment-variables.md mcpbridge-wrapper.docc/EnvironmentVariables.md
README.md mcpbridge-wrapper.docc/mcpbridge-wrapper.md
  • Documentation changes are synced with DocC catalog (or N/A)

Testing

  • Added/updated tests for new functionality
  • All tests pass locally
  • Manually tested the changes

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • Comments added for complex code
  • Documentation updated (if needed)
  • No new warnings generated
  • PR title is descriptive

Validation Output

  • pytest -q => 629 passed, 5 skipped
  • ruff check src/ => pass
  • mypy src/ => pass
  • pytest --cov => 91.33%

Task Artifacts

  • /Users/egor/Development/GitHub/XcodeMCPWrapper/SPECS/ARCHIVE/BUG-T16_Tool_Distribution_Pie_widget_is_cropped_at_medium_widths/BUG-T16_Tool_Distribution_Pie_widget_is_cropped_at_medium_widths.md
  • /Users/egor/Development/GitHub/XcodeMCPWrapper/SPECS/ARCHIVE/BUG-T16_Tool_Distribution_Pie_widget_is_cropped_at_medium_widths/BUG-T16_Validation_Report.md
  • /Users/egor/Development/GitHub/XcodeMCPWrapper/SPECS/ARCHIVE/_Historical/REVIEW_bug_t16_pie_responsive.md

@SoundBlaster SoundBlaster merged commit fdb53c8 into main Feb 20, 2026
10 checks passed
@SoundBlaster SoundBlaster deleted the feature/BUG-T16-tool-distribution-pie-responsive-crop branch February 20, 2026 20:40
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