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.

Sign up for GitHub

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

Jump to bottom

[Bug]: ExtendedMessageBox does not support CancelTryContinue when built for "ANY" of the supported TFM's #378

Closed
Smurf-IV opened this issue Dec 11, 2022 · 2 comments
Closed

[Bug]: ExtendedMessageBox does not support CancelTryContinue when built for "ANY" of the supported TFM's #378

Smurf-IV opened this issue Dec 11, 2022 · 2 comments
Labels
bug Something isn't working fixed This issue has been fixed.
Milestone
Version 8

Comments

@Smurf-IV
Copy link
Member

The following code is incorrect and will not build because the enum is not named liked that:

#if NET60_OR_GREATER
                case ExtendedMessageBoxButtons.CANCELTRYCONTINUE:

Also the idea of using that is to make it work for any TFM.
@Smurf-IV Smurf-IV added the bug Something isn't working label Dec 11, 2022
@Smurf-IV Smurf-IV changed the title [Bug]: ExtendedMessageBox does not support CancelTryContinue when built for "ANY" TFM [Bug]: ExtendedMessageBox does not support CancelTryContinue when built for "ANY" of the supported TFM's Dec 11, 2022
@Wagnerp Wagnerp self-assigned this Dec 15, 2022
@Wagnerp Wagnerp added the under investigation This bug/issue is currently under investigation. label Dec 15, 2022
@Wagnerp Wagnerp added this to the Version 8 milestone Dec 15, 2022
@Wagnerp
Copy link
Contributor

Wagnerp commented Dec 15, 2022

Pushing a nightly build tomorrow (16/12/2022) that includes modifications carried over from Krypton-Suite/Standard-Toolkit#869

@Wagnerp Wagnerp mentioned this issue Dec 19, 2022
Merged
@Wagnerp Wagnerp changed the title [Bug]: ExtendedMessageBox does not support CancelTryContinue when built for "ANY" of the supported TFM's [Bug]: ExtendedMessageBox does not support CancelTryContinue when built for "ANY" of the supported TFM's Dec 20, 2022
@Wagnerp Wagnerp removed their assignment Dec 20, 2022
@Wagnerp Wagnerp added fixed This issue has been fixed. and removed under investigation This bug/issue is currently under investigation. labels Dec 20, 2022
@Wagnerp
Copy link
Contributor

Wagnerp commented Dec 20, 2022

Resolved 20/12/2022

@Wagnerp Wagnerp closed this as completed Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working fixed This issue has been fixed.
Projects
None yet
Milestone
Version 8
Development

No branches or pull requests
2 participants
@Wagnerp @Smurf-IV