-
Notifications
You must be signed in to change notification settings - Fork 128
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
launch crash on 0b9d539 #34
Comments
Thanks for reporting! Can you reproduce the crash and is there any relevant information you have been able to debug? Do you use an external (non-Apple) keyboard? |
@kasper |
Okay, have you used this keyboard before or is it new? I presume OS X cannot find a UTF-8 keyboard layout for it... |
I used this keyboard many years. |
Does it crash on each launch? What is the last build that works? |
To troubleshoot the keyboard use Karabiner + it's Event Viewer, it does comprehensive USB/HID checking |
It crash on each launch.
System version: 10.11 (15A284) My latest work phoenix is |
@alswl Thanks! I’ll sort this out as soon as I have time to do so. Sorry for the wait. Are you trying to bind a non-ASCII-key and what layout is your actual physical keyboard (also en-US)? Just to confirm this issue, would you be able to test an another keyboard? |
I'll help to improve Phoenix. Thanks for continue maintain this repo. I use ASCII-key, keyboard layout is en-US(I am a programmer). I will test with Macbook origin keyboard later. On Tue, Oct 20, 2015 at 4:28 PM, Kasper Hirvikoski <notifications@github.com
alswl Mail: alswlx[a]gmailDOTcom |
Thanks, I wonder what the difference is between these versions. I also wonder why nobody else has reported this. Obviously, there is an issue I need to solve. :) |
@alswl So it would seem that your keyboard layout is not Unicode capable (at least OS X thinks so). However, it should be ASCII-cabable. Could you try the |
I complied cc5dafc(origin/fix-for-ascii-keyboard). if I press |
in the same configuration, Version 2.0 (4) works. |
oh, thanks, fix the handlers works for me. |
cc5dafc works for me. Now it do not crash. |
Great! I will keep this open until I have merged the fix. |
Ok, this is now finally merged to 2.0! Thanks for reporting the issue. |
DiagnosticReports:
The text was updated successfully, but these errors were encountered: