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

Add Windows 95/98/ME support to the GDI demo #534

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ccawley2011
Copy link
Contributor

This mostly works, with a few caveats:

  • ANSI clipboard support isn't implemented yet.
  • Text rendering isn't as good as it could be.
  • nk_gdi_rect_multi_color requires msimg32.dll which isn't always available. This can be trivially disabled if it's not needed.

Screenshot:
Nuklear95


HPEN pen = NULL;
if (line_thickness == 1) {
SetDCPenColor(dc, color);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this removal?

ExtTextOutW(dc, x, y, ETO_OPAQUE, NULL, wstr, wsize, NULL);
} else {
/* TODO: Convert from UTF-8 to the active code page */
ExtTextOutA(dc, x, y, ETO_OPAQUE, NULL, text, len, NULL);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wondering if you plan to work on this conversion any time soon.

@dumblob
Copy link
Member

dumblob commented Jan 5, 2023

Interesting. I am open to merge this as increasing platform compatibility is generally a good idea and the changes do not look harmful to me.

Still I wonder what the motivation is as these old systems are super obsolete and I am having hard time imagining anyone targeting them to write new code for. Any insights?

  • ANSI clipboard support isn't implemented yet.

Do you plan to work on it any time soon? Or shall we rather merge this and not wait?

  • Text rendering isn't as good as it could be.

Feel free to consult our wiki in case you missed it: https://github.com/Immediate-Mode-UI/Nuklear/wiki/Complete-font-guide

  • nk_gdi_rect_multi_color requires msimg32.dll which isn't always available. This can be trivially disabled if it's not needed.

Could you please add comment(s) to the relevant part(s) of the source code describing how to disable this dependency?

Thanks!

@RobLoach
Copy link
Contributor

RobLoach commented Apr 4, 2024

Looks like there's a conflict

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.

None yet

3 participants