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

Fix #1617. #1619

Merged
merged 1 commit into from
Oct 14, 2024
Merged

Fix #1617. #1619

merged 1 commit into from
Oct 14, 2024

Conversation

Hoikas
Copy link
Member

@Hoikas Hoikas commented Oct 12, 2024

Fixes #1617.

This invalidates the UI when the clothing material combo box selections are changed. By invalidating the UI, the texture pickers update appropriately. I also added some extra safety valves to reduce spurious updates that the debugger showed coming through.

This invalidates the UI when the clothing material combo box selections
are changed. By invalidating the UI, the texture pickers update
appropriately. I also added some extra safety valves to reduce spurious
updates that the debugger showed coming through.
@Hoikas Hoikas added the Plugin label Oct 12, 2024
Copy link
Member

@dpogue dpogue left a comment

Choose a reason for hiding this comment

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

The code looks good to me, I can see about trying to build and test it in Max

@Hoikas
Copy link
Member Author

Hoikas commented Oct 12, 2024

It seems to fix the problem for me in Max 2022. I'm a little unsure as to why this is needed though - it feels like the code should have simply worked as is. I wonder if this was a problem in Max 7?

@Deledrius
Copy link
Member

That makes me wonder about regressions on Max 7. It doesn't look like it should be a problem, but it wouldn't be the first time that a seemingly-innocuous change around the WinAPI breaks something in older Max versions.

@Hoikas
Copy link
Member Author

Hoikas commented Oct 12, 2024

This isn't actually a WinAPI regression. What's happening here is that, in Max 2022, the Update method on ClothingBasicDlgProc isn't being called when values on the param bock are being changed during the DlgProc() method.

@dpogue
Copy link
Member

dpogue commented Oct 14, 2024

Built and tested this and it's working in Max 2022 👍

@dpogue dpogue merged commit d9ee8e9 into H-uru:master Oct 14, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Changing the layer dropdown in a Clothing Material does not re-render the texture selection
3 participants