Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate Item details to Direct2D #1120

Merged
merged 3 commits into from
Feb 4, 2025
Merged

Migrate Item details to Direct2D #1120

merged 3 commits into from
Feb 4, 2025

Conversation

reupen
Copy link
Owner

@reupen reupen commented Feb 3, 2025

#954

This migrates Item details from GDI to Direct2D. Text rendering appears to be identical in all modes.

This also adds support for SVG glyphs, in theory from later Windows 10 versions onwards (but I'm only aware of Windows 11 actually having SVG glyphs in Segoe UI Emoji).

Additionally, a couple of small bugs with padding and font-related title formatting fields were fixed.

@reupen reupen changed the title Item details d2d Migrate Item details to Direct2D Feb 3, 2025
@reupen reupen force-pushed the item-details-d2d branch 4 times, most recently from 99d4208 to d0b1a85 Compare February 4, 2025 19:59
This migrates Item details from GDI to Direct2D. Text rendering appears to be identical in all modes.

This also adds support for SVG glyphs, in theory from later Windows 10 versions onwards (but I'm only aware of Windows 11 actually having SVG glyphs in Segoe UI Emoji).
This fixes a small bug in Item details where left and top padding where missing when there was no horizontal and vertical scroll bar respectively.
This fixes a small bug where `%default_font_face%` (deprecated) and `%default_font_size%` did not update in Item details until some other event triggered a content update.
@reupen reupen marked this pull request as ready for review February 4, 2025 20:01
@reupen reupen merged commit 731bdd9 into main Feb 4, 2025
8 checks passed
@reupen reupen deleted the item-details-d2d branch February 4, 2025 20:26
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.

1 participant