feat: support resolver #1408
feat: support resolver #1408
Conversation
✅ Deploy Preview for rsdoctor ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for the resolver feature by refactoring the ResolverDetailsPanel UI from a collapsible side panel layout to a tab-based interface, improving user experience and simplifying the component structure. The resolver feature is also enabled in the example configuration.
- Replaced collapsible sidebar with Tabs component for better navigation between diff viewer and resolver details
- Removed unused state management for collapse/expand functionality
- Enabled 'resolver' feature in the modern-minimal example configuration
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/components/src/components/Resolver/analysis.tsx | Refactored ResolverDetailsPanel from collapsible side panel to tab-based layout, removed unused imports and collapsed state |
| examples/modern-minimal/modern.config.ts | Added 'resolver' to the features array to enable the resolver functionality |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| </Col> | ||
| </React.Fragment> | ||
| <Col flex={1}> | ||
| <Tabs |
There was a problem hiding this comment.
The Tabs component should include an aria-label or aria-labelledby attribute to provide context for screen reader users navigating between the diff viewer and resolver details tabs.
| <Tabs | |
| <Tabs | |
| aria-label="Diff viewer and resolver details tabs" |
yifancong
force-pushed
the
optimize/resolver
branch
from
844e85e to
1dd821a
Compare
Summary
Related Links