Conversation
- Increase ItemHeight from 60 to 70 pixels for better spacing - Increase NEW badge font size from 7pt to 8pt and height from 18 to 20 pixels - Increase detail font size from 8pt to 9pt for better readability - Improve color contrast: change detail text from Color.Gray to Color.FromArgb(80, 80, 80) - Increase title truncation from 60 to 70 characters to show more context - Increase vertical spacing between text lines from 18 to 20 pixels Co-authored-by: sunzhuoshi <592211+sunzhuoshi@users.noreply.github.com>
sunzhuoshi
left a comment
There was a problem hiding this comment.
@copilot revert the code and support font selection
- Reverted previous hardcoded font size and spacing changes - Added FontFamily and FontSize properties to Configuration - Added font selection UI in SettingsForm with FontDialog - Updated MainWindow to use configurable font from settings - Dynamic item height calculation based on selected font size - Dynamic badge and spacing adjustment based on font size - Font settings are persisted in Windows Registry Co-authored-by: sunzhuoshi <592211+sunzhuoshi@users.noreply.github.com>
I've reverted the hardcoded text clarity changes and implemented a font selection feature instead. Changes in commit 93af8cb: ✅ Reverted previous fixed font size/spacing changes How to use:
The review request list will automatically adjust to your selected font with appropriate spacing and layout. UI Preview: |
sunzhuoshi
left a comment
There was a problem hiding this comment.
@copilot revert the code, support high DPI better
- Reverted font selection feature from Configuration and SettingsForm - Added application manifest for Windows 10/11 compatibility - Enabled PerMonitorV2 high DPI mode in Program.cs - Added DPI scale factor detection and helper methods in MainWindow - Applied DPI-aware scaling to all UI dimensions and drawing code - Form size, padding, button dimensions scale with DPI - ListBox item height scales with DPI - Badge, text positioning, and spacing scale with DPI Co-authored-by: sunzhuoshi <592211+sunzhuoshi@users.noreply.github.com>
I've reverted the font selection feature and implemented comprehensive high DPI support instead. Changes in commit 2b60c0b: ✅ Reverted font selection feature High DPI Features:
How it works: // Detects DPI at startup
_dpiScale = g.DpiX / 96f;
// Scales all dimensions
ItemHeight = ScaleDpi(60); // e.g., 90px at 150% scalingThe application now provides optimal visual quality on high DPI displays and multi-monitor setups with different scaling levels. |
Addresses text clarity issues in review request list box items by implementing comprehensive high DPI support for modern displays.
Changes
High DPI Support
Dynamic DPI Scaling
ScaleDpi(value) = value * (currentDPI / 96)Implementation Details
Application.SetHighDpiMode(HighDpiMode.PerMonitorV2)callDPI Scaling Examples
Benefits
Compatibility
Windows 10 1703+: Full PerMonitorV2 support
Windows 10 (earlier): Falls back to PerMonitor mode
Windows 8.1: Basic per-monitor support
Windows 7: System-aware mode
Fixes Texts in review request list box items are not clear #95
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.