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

python fixes and new plugin #2637

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

python fixes and new plugin #2637

wants to merge 14 commits into from

Conversation

m32
Copy link
Contributor

@m32 m32 commented Jan 17, 2025

  • removed references to debugpy
  • dialog can be created without borders
  • proper numbering of dialog elements
  • new plugin (WIP) hex editor
  • other small fixes

@unxed
Copy link
Contributor

unxed commented Jan 17, 2025

Hex editor! Cool. Many requests on it

@m32 m32 mentioned this pull request Jan 17, 2025
@m32
Copy link
Contributor Author

m32 commented Jan 17, 2025

I need help with the keybar, I don't know how to read the alt/shift/ctrl key state.
Do you know how to do this in the dialogue function?

@anta999
Copy link
Contributor

anta999 commented Jan 17, 2025

I need help with the keybar, I don't know how to read the alt/shift/ctrl key state. Do you know how to do this in the dialogue function?

DlgProc(HANDLE hDlg, int Msg, int Param1, LONG_PTR Param2)

Msg DM_KEY
Param1 CONTROL ID
Param2 KEY

case DN_KEY: {

	if (Param1 == CONTROL1) {

		if (Param2 == KEY_SHIFT | KEY_CTRL | KEY_ALT ) {

			
		}
	}

}

KEY_SHIFTENTER = KEY_SHIFT | KEY_ENTER,
KEY_ALTSHIFTENTER = KEY_ALT | KEY_SHIFT | KEY_ENTER,
KEY_CTRLALTENTER = KEY_CTRL | KEY_ALT | KEY_ENTER,

@m32
Copy link
Contributor Author

m32 commented Jan 18, 2025

@anta999
It doesn't work and can't work on dialogue - dialogue doesn't have such functionality.
The dialog doesn't inform about the change of key state but about their pressing, i.e. pressing shift+f5 you will see but releasing the shift key you won't.
Far3 introduced DN_CONTROLINPUT (probably for this reason)

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.

3 participants