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

feat: add default mac menu bar items #85

Merged
merged 1 commit into from
Nov 2, 2021
Merged

Conversation

leo-benz
Copy link
Contributor

@leo-benz leo-benz commented Nov 1, 2021

  • Add default 'Window' menu
  • Add help menu with links to github issues and wiki

Closes #83

- Add default 'Window' menu
- Add help menu with links to github issues and wiki

Closes Xiphe#83
@Xiphe
Copy link
Owner

Xiphe commented Nov 2, 2021

Cool! Thanks for the PR.

(And also sorry for not really working on this lately :/)

@Xiphe Xiphe merged commit e63de8e into Xiphe:beta Nov 2, 2021
@Xiphe
Copy link
Owner

Xiphe commented Nov 2, 2021

the release did not went through :/

Will look into that within the next days

@leo-benz
Copy link
Contributor Author

leo-benz commented Nov 2, 2021

I checked the build log and noticed that I faced the same issue locally when packaging a build. I traced it down to an incompatibility between ts-loader and webpack (see TypeStrong/ts-loader#595). By updating CRA to the next version (see facebook/create-react-app#11278) it was fixed. I did not commit these changes because I thought it is a local issue and I don't know if updating CRA has any side effects.

@Xiphe
Copy link
Owner

Xiphe commented Nov 8, 2021

Ah awesome! Thanks for sharing. Will try that later today

@github-actions
Copy link

github-actions bot commented Nov 9, 2021

🎉 This PR is included in version 1.0.0-beta.18 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Request: Support for macOS Keyboard Shortcuts (e.g., Cmd+M for minimizing)
2 participants