mirror of
https://github.com/kusti8/proton-native.git
synced 2026-05-15 14:15:50 -06:00
[GH-ISSUE #18] Menu.Item type="Quit" throwing error #10
Labels
No labels
bug
documentation
enhancement
libui issue
pull-request
question
wait for libui implementation
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/proton-native#10
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @kaseyhinton on GitHub (Feb 18, 2018).
Original GitHub issue: https://github.com/kusti8/proton-native/issues/18
The quit menu item is throwing an error. Using windows 10.
Here is the error
@kusti8 commented on GitHub (Feb 19, 2018):
What's the full output ?
@kusti8 commented on GitHub (Mar 8, 2018):
Closing due to inactivity. Feel free to open it if the issue persists.
@mischnic commented on GitHub (Mar 8, 2018):
@kusti8
That really is the full output:
I'm guessing there is some error in native code (hence no error details).
log:
code:
@mischnic commented on GitHub (Mar 17, 2018):
@parro-it This looks like some crash in native code (libui-node) ? My attempts at using a debugger on Windows weren't successful.
@parro-it commented on GitHub (Mar 17, 2018):
It seems. Did you already try it with the 0_1_0 branch? I will otherwise...
@mischnic commented on GitHub (Mar 17, 2018):
Yes, exact same error.
@mischnic commented on GitHub (Mar 17, 2018):
proton-native calls
UiMenuItem.onClickedalso ontype="Quit"items. This causes the crash.@parro-it commented on GitHub (Mar 17, 2018):
I will try it tonight... I developed the new version on seven, I didn't yet tested it on 10.
@mischnic commented on GitHub (Mar 17, 2018):
Proposed fix: Removing line
7741dd2fcc/src/components/MenuBar.js (L83)Would this have other side effects?
@kusti8 commented on GitHub (Mar 17, 2018):
It shouldn't. Cause then when it's iterating through the props, it just
won't set that.
On Sat, Mar 17, 2018, 12:25 PM Niklas Mischkulnig notifications@github.com
wrote:
@mischnic commented on GitHub (Mar 18, 2018):
On macOS, libui prints an error before it quits:
Not on Windows, I created an issue: https://github.com/andlabs/libui/issues/301.
(Any idea why proton-native works fine on macOS?)
@andlabs commented on GitHub (Mar 18, 2018):
As I said in that other issue, on Windows libui prints things to the debug log, which is your only real course of action on a
/subsystem:windowsprogram; you can use SysInternals DebugView if you aren't running in an actual debugger. I could make it print to stderr in addition to that, maybe... or provide a custom reporting option, maybe...