-
Notifications
You must be signed in to change notification settings - Fork 20
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
[WiP] Add top menu to app #557
base: main
Are you sure you want to change the base?
Conversation
guergana
commented
Sep 22, 2024
- fixes ODE - Top menu options #417
Deploying opendataeditor with Cloudflare Pages
|
I guess to get actual implementation of this you need to pass all the actions and store changes through the electron bridge which is quite a task |
@roll yes, and I am stuck. I would appreciate your help since the code I have currently in this PR is supposed to work but it isn't working 😭 do you have any suggestions of what I might be doing wrong? |
submenu: [ | ||
{ role: 'about' }, | ||
{ role: 'quit' } | ||
] as MenuItemConstructorOptions[] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's better to use type validation instead of type inforcement like:
const template: Electron.MenuItemConstructorOptions[] // or what is correct here
Otherwise using as
will silently hide all configuration errors
@guergana |