Dev 260103 improve auto detect screen aspect ratio#96
Merged
kevinzjpeng merged 3 commits intomainfrom Jan 4, 2026
Merged
Conversation
…abilities - Removed the settings export format documentation as it is no longer needed. - Simplified the creation and management of the "View" menu in AppDelegate. - Added conditional logging for various components (CH32V208ControlChipset, HALIntegrationManager, VideoChipsetHAL) based on new logging flags. - Introduced a new `HalPrint` logging flag in LoggerManager to control HAL-related logs. - Updated MouseManager and SerialPortManager to log events conditionally based on the new logging flags. - Enhanced VideoManager to handle zoom functionality, including zooming to height and width, and added corresponding menu actions. - Implemented zoom level handling in PlayerViewModel and PlayerView, allowing for dynamic zoom adjustments in the video preview. - Added zoom controls to the app's command menu for user accessibility.
- Introduced ZoomSource enum to track the origin of zoom changes (manual, menu, autoResize). - Updated zoom methods (zoomIn, zoomOut, resetZoom, zoomToHeight, zoomToWidth) to accept a source parameter. - Enhanced setZoomLevel method to log the source of zoom changes and post notifications with zoom level and source. - Modified VideoOutputDelegate to utilize VideoManager for active rectangle detection, improving performance and clarity. - Added AppInfoOverlayView to display the active aspect ratio alongside video rect dimensions. - Created AspectRatioSettingsView for user to select aspect ratio and scaling options, replacing previous popup implementation. - Updated PlayerView to handle auto-zoom after window resizing based on active video rect. - Adjusted AdvancedDebugSettingsView and AudioVideoSettingsView to reflect new aspect ratio settings structure. - Removed deprecated methods and variables related to aspect ratio handling in WindowUtils.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.