[PR #3] [MERGED] Fix DPI scaling issues: dynamic status bar height and parts sizing #17

Closed
opened 2026-03-03 12:01:45 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ForLoopCodes/legacy-notepad/pull/3
Author: @radami2006
Created: 1/29/2026
Status: Merged
Merged: 1/30/2026
Merged by: @ForLoopCodes

Base: mainHead: fix/dpi-statusbar


📝 Commits (2)

  • 59bda7d Fix DPI scaling issues: dynamic status bar height and parts sizing
  • 25f5979 Fix DPI scaling issues: dynamic status bar height and parts sizing and kept original icons

📊 Changes

1 file changed (+49 additions, -11 deletions)

View changed files

📝 src/main.cpp (+49 -11)

📄 Description

Problem

On high-DPI systems, the status bar was partially clipped:

  • Hardcoded status bar height (22px) caused bottom truncation.
  • Fixed part widths caused rightmost panels (encoding / line endings / zoom) to be cut.

Solution

  • Compute the actual status bar height at runtime instead of using a hardcoded value.
  • Dynamically size status bar parts based on text width and current DPI.

Result

  • No more clipping on high-DPI displays.
  • Correct layout across different DPI scales and system themes.

How to test

  1. Run the app on a system with DPI scaling >100%.
  2. Toggle the status bar.
  3. Resize the window and verify the bottom/right areas are fully visible.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ForLoopCodes/legacy-notepad/pull/3 **Author:** [@radami2006](https://github.com/radami2006) **Created:** 1/29/2026 **Status:** ✅ Merged **Merged:** 1/30/2026 **Merged by:** [@ForLoopCodes](https://github.com/ForLoopCodes) **Base:** `main` ← **Head:** `fix/dpi-statusbar` --- ### 📝 Commits (2) - [`59bda7d`](https://github.com/ForLoopCodes/legacy-notepad/commit/59bda7dd945c371b4bf9cbce7e7f0564cead8d11) Fix DPI scaling issues: dynamic status bar height and parts sizing - [`25f5979`](https://github.com/ForLoopCodes/legacy-notepad/commit/25f59790f90b7fb0b3603eb28050b521be733320) Fix DPI scaling issues: dynamic status bar height and parts sizing and kept original icons ### 📊 Changes **1 file changed** (+49 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `src/main.cpp` (+49 -11) </details> ### 📄 Description ### Problem On high-DPI systems, the status bar was partially clipped: - Hardcoded status bar height (22px) caused bottom truncation. - Fixed part widths caused rightmost panels (encoding / line endings / zoom) to be cut. ### Solution - Compute the actual status bar height at runtime instead of using a hardcoded value. - Dynamically size status bar parts based on text width and current DPI. ### Result - No more clipping on high-DPI displays. - Correct layout across different DPI scales and system themes. ### How to test 1. Run the app on a system with DPI scaling >100%. 2. Toggle the status bar. 3. Resize the window and verify the bottom/right areas are fully visible. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-03 12:01:45 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/legacy-notepad#17
No description provided.