-
Notifications
You must be signed in to change notification settings - Fork 3
Minor misc changes #41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces mobile-responsive UI enhancements across several views, adds a searchable sidebar navigation, and includes a new standard column (organizationid) in the generator metadata.
- Introduce a search bar with entity filtering and text highlighting in the sidebar
- Update responsive styling and typography for Relationships, Keys, and Attributes components
- Add
organizationidfield to default metadata descriptions and adjust release workflow to commit version changes
Reviewed Changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Website/components/datamodelview/SidebarDatamodelView.tsx | Added a search input, group auto-expansion, and highlighting |
| Website/components/datamodelview/Relationships.tsx | Updated filter bar and table header styling for mobile |
| Website/components/datamodelview/Keys.tsx | Updated key search bar and table column styling |
| Website/components/datamodelview/Attributes.tsx | Updated attribute search/filter bar and table styling |
| Website/components/attributes/*.tsx | Standardized typography and responsive classes throughout |
| Generator/MetadataExtensions.cs | Added organizationid to default English and localized fields |
| .github/workflows/release.yml | Switched from PR-based to commit-based manual release workflow |
Comments suppressed due to low confidence (2)
Website/components/datamodelview/SidebarDatamodelView.tsx:36
- New search filtering logic in the sidebar isn't covered by existing tests. Consider adding unit or integration tests to verify search term handling, group expansion, and highlighting behavior.
const handleSearch = (term: string) => {
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…/DataModelViewer into patch/misc-changes-01
Summary of changes