Skip to content

Conversation

@ManlyMarco
Copy link
Collaborator

No description provided.

@ManlyMarco ManlyMarco requested a review from Copilot October 30, 2025 01:46
Copy link

Copilot AI left a 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 enhances the sideloader viewer UI with a resizable split container layout and improves property display for mod metadata. The key changes include restructuring the UI for better flexibility, adding a new column for content type, and improving the browsability of manifest data in property grids.

  • Replaces fixed layout with nested split containers for resizable panels
  • Adds ContentsKind column to display mod content types
  • Enhances property grid display with TypeConverters and ToString methods

Reviewed Changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
SideloaderViewer.resx Updates UI layout metadata to use split containers instead of fixed docking
SideloaderViewer.Designer.cs Adds splitContainer1 and splitContainer2 controls with proper initialization
SideloaderModsWindow.resx Adds ContentsKind column metadata and reorders existing column properties
SideloaderModsWindow.Designer.cs Implements the new olvColumnContentType column in the object list view
SideloaderModInfo.cs Removes [Browsable(false)] from Manifest property to show it in property grid
MigrationInfo.cs Adds ReadOnly attribute and ToString method for better property grid display
Manifest.cs Adds TypeConverter attributes and [Browsable(false)] to ManifestDocumentRoot
Files not reviewed (2)
  • src/KKManager/Windows/Content/SideloaderModsWindow.Designer.cs: Language not supported
  • src/KKManager/Windows/ToolWindows/Properties/Viewers/SideloaderViewer.Designer.cs: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ManlyMarco ManlyMarco merged commit 43f29e0 into master Oct 30, 2025
8 checks passed
@ManlyMarco ManlyMarco deleted the cards branch October 30, 2025 02:19
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.

2 participants