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

Use a block to determine the enablement state of a menu item that is created based on a command #1604

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

koendehondt
Copy link
Contributor

Code changes for #1603.

I did not add a test, nor an example, because it is not clear to me how to test this. If one can provide me some guidance, I am happy to add the test.

…created based on a command, otherwise the menu item is not enabled/disabled correctly when its parent menu opens
@koendehondt
Copy link
Contributor Author

This change is important for the Spec book. Please merge the PR to Pharo 12 as well.

@jecisc jecisc merged commit 5b6ba6a into pharo-spec:Pharo13 Sep 17, 2024
1 of 2 checks passed
@jecisc
Copy link
Member

jecisc commented Sep 17, 2024

Hi @koendehondt
Would it be possible to produce the PR to P12 and ping me on it to merge it?

@koendehondt
Copy link
Contributor Author

Hi @koendehondt Would it be possible to produce the PR to P12 and ping me on it to merge it?

@jecisc Here it is: #1605

Thank you!

@koendehondt koendehondt deleted the gh1603-menu-item-enablement branch September 19, 2024 04:51
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.

2 participants