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

GDI+ (gdip), Text Input Copy/Paste/Ctrl+A not working #572

Closed
nyaruku opened this issue Aug 17, 2023 · 5 comments
Closed

GDI+ (gdip), Text Input Copy/Paste/Ctrl+A not working #572

nyaruku opened this issue Aug 17, 2023 · 5 comments

Comments

@nyaruku
Copy link
Contributor

nyaruku commented Aug 17, 2023

image
So I'm working on an Application with Nuklear and i realised following things:

  • Ctrl+C (copy) only works partially, when i paste into notepad.exe, it works, while pasting into windows search, it doesnt.
  • Ctrl+V (paste) doesn't do anything as well as Ctrl+A (select entire text).
    image
@a1ive
Copy link

a1ive commented Sep 8, 2023

if (!IsClipboardFormatAvailable(CF_UNICODETEXT) && OpenClipboard(NULL))

=> if (!IsClipboardFormatAvailable(CF_UNICODETEXT) || !OpenClipboard(NULL))
It's working for me. (#126)

@nyaruku
Copy link
Contributor Author

nyaruku commented Sep 8, 2023

if (!IsClipboardFormatAvailable(CF_UNICODETEXT) && OpenClipboard(NULL))

=> if (!IsClipboardFormatAvailable(CF_UNICODETEXT) || !OpenClipboard(NULL))
It's working for me. (#126)

gonna try the paste fix, but for me ctrl+a is also not working, and i tried to switch to GDI and it works, so I'll have to stick with GDI for now, gonna try to compare the gdi+ header with the gdi header if i can find a fix.

@nyaruku
Copy link
Contributor Author

nyaruku commented Sep 8, 2023

Alright i fixed copy/paste by just using the code from the gdi header, now i have to fix the ctrl+a im pretty sure it has something to do with the key events, gonna do a PR once i fixed that too

@nyaruku
Copy link
Contributor Author

nyaruku commented Sep 8, 2023

image
after comparing both headers, i realised that gdip header doesnt even have a switch statement to select all the text bruh

@nyaruku
Copy link
Contributor Author

nyaruku commented Sep 8, 2023

Fix is there, updated my PR: 0435477
#576

@nyaruku nyaruku closed this as completed Sep 8, 2023
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

No branches or pull requests

2 participants