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

[Minor Enhancement] Use System Cursor #347

Open
Yaron10 opened this issue Jan 19, 2023 · 4 comments
Open

[Minor Enhancement] Use System Cursor #347

Yaron10 opened this issue Jan 19, 2023 · 4 comments

Comments

@Yaron10
Copy link

Yaron10 commented Jan 19, 2023

Hello Pavel,

notepad-plus-plus/notepad-plus-plus@0742410.

When you have some free time.

Thank you very much.

BR

@pnedev
Copy link
Owner

pnedev commented Jan 20, 2023

Hello Yaron,

I am sorry my friend but I really don't understand what you mean.
Could you please elaborate?
Thank you.

BR

@Yaron10
Copy link
Author

Yaron10 commented Jan 20, 2023

Hello Pavel,

Hovering over a link in the About dialog, you get a custom small cursor (even in 96 DPI).
If you replace the URLCtrl files with the new ones used now by NPP, we would get the system cursor.
Please see the video in notepad-plus-plus/notepad-plus-plus#12773 (comment).

Also, the XORMask can be removed and the code is shorter.

Thank you.

BR

@pnedev
Copy link
Owner

pnedev commented Jan 21, 2023

Hello Yaron,

Oh, OK, now I got it, thank you :)
Good suggestion but since this improvement is not functional and is really minor I'll keep it with lowest priority.

BR

@Yaron10
Copy link
Author

Yaron10 commented Jan 21, 2023

Hello Pavel,

Sure, it's a mini-minor issue. :)

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants