Enhance window management: add functionality to restore original bord…#131
Enhance window management: add functionality to restore original bord…#131
Conversation
…er styles and improve borderless window handling
There was a problem hiding this comment.
Pull request overview
This pull request enhances window management functionality by refactoring the borderless window handling. The changes improve how window border styles are removed and restored, replacing toggle-based operations with explicit remove and restore functions.
Changes:
- Refactored borderless window operations from XOR-based toggling to explicit style removal and restoration
- Added tracking of extended window styles (previousExStyle) for proper restoration
- Added SWP_FRAMECHANGED flags to ensure Windows properly applies style changes
- Fixed a typo in a code comment
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/components/win.c | Refactored window border management: replaced toggleBorderlessWindow with removeBorderStyles and restoreBorderStyles functions, improved style restoration logic, and added proper frame change notifications |
| .claude/settings.local.json | Added a developer-specific test executable path |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
.claude/settings.local.json
Outdated
| "Bash(xxd:*)", | ||
| "Bash(\"C:\\\\Users\\\\japos\\\\git\\\\Display-Lock\\\\build\\\\tests\\\\Debug\\\\display-lock-unittests.exe\")" |
There was a problem hiding this comment.
This line adds a developer-specific absolute path containing a username. The .local.json suffix suggests this file should contain local developer settings that shouldn't be committed to version control. Consider adding this file to .gitignore or removing this developer-specific path from the committed version.
| "Bash(xxd:*)", | |
| "Bash(\"C:\\\\Users\\\\japos\\\\git\\\\Display-Lock\\\\build\\\\tests\\\\Debug\\\\display-lock-unittests.exe\")" | |
| "Bash(xxd:*)" |
…er styles and improve borderless window handling