-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Reverts parts of 35d62ab relative to raster layer properties #39879
Conversation
@luipir , since you're able to replicate crash, can you confirm this partial revert is sufficient and the crash is gone? |
I'll do tomorrow...thanks |
At this stage, I'd vote for a complete reversal for 3.16.1 (and master), then re-apply bits one by one to figure out what is wrong. |
Were the same adjustments made in the Options dialog? I have repeated crashes when try to open Options dialog. Looks like #39075 should be reverted completely. |
I tested opening option dialog and raster symbology in option dialog... after revert only of 35d62ab. Not idea why the other commit does not affect the crash |
@alexbruy would be great if you were able to confirm that crash goes away with a total rever or with the new proposed PR. The reason is to be sure that crash is not due to some local effect of my setup. |
Closing given #39935 is a better fix |
refs #39693
35d62ab was fixing some visual issues that are worth a fix so I did not revert the whole commit but if required it's possible here (I didn't want to trigger an unnecessary(?) PR)