Skip to content

Conversation

@ManlyMarco
Copy link
Owner

No description provided.

@ManlyMarco ManlyMarco requested a review from Copilot October 20, 2025 22:20
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 refactors and enhances the PatchInspector functionality with improved UI/UX, better code organization, and additional interactive features. The changes focus on modernizing the codebase through consistent naming conventions, better encapsulation, and integration with the broader RuntimeUnityEditor framework.

Key changes:

  • Refactored PatchInspector and ILViewerWindow to inherit from Window<T> base class for consistent window management
  • Enhanced UI with context menus, clickable labels, and better visual feedback
  • Added new reusable GUI styles (MiddleCenterLabelStyle, UpperCenterLabelStyle) to IMGUIUtils

Reviewed Changes

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

Show a summary per file
File Description
RuntimeUnityEditor.Core/Utils/IMGUIUtils.cs Added two new cached GUIStyle properties for centered label alignment
RuntimeUnityEditor.Bepin5/PatchInspector/PatchInspector.cs Major refactor: converted to Window base class, improved UI with interactive elements, better state management, and code cleanup
RuntimeUnityEditor.Bepin5/PatchInspector/PatchInfo.cs Changed from struct to class and renamed MethodName to TargetMethodName, added Patch field
RuntimeUnityEditor.Bepin5/PatchInspector/ILViewerWindow.cs Complete rewrite: converted to Window base class, redesigned UI layout, improved patch management
RuntimeUnityEditor.Bepin5/PatchInspector/ILViewMode.cs Removed obsolete enum that is no longer needed

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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