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

Create AppMenu class #2242

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Create AppMenu class #2242

merged 1 commit into from
Jul 11, 2023

Conversation

danirabbit
Copy link
Member

Just the appmenu part of #2233 since the second commit seems to introduce some issues that need more investigating

@danirabbit danirabbit requested a review from jeremypw July 10, 2023 19:23
Copy link
Collaborator

@jeremypw jeremypw left a comment

Choose a reason for hiding this comment

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

Looks good, no regressions found.

@jeremypw jeremypw merged commit be1f4f8 into main Jul 11, 2023
4 checks passed
@jeremypw jeremypw deleted the danirabbit/appmenu-class branch July 11, 2023 13:14
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