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

Add Neo 2 Layout data #8753

Closed
wants to merge 1 commit into from
Closed

Conversation

kreibaum
Copy link

This is for gitpod-io/gitpod#1363 where I have executed the generate-layout script on an Ubuntu 20.10. (Windows 10 does not support Neo 2 layouts afaik)

Since I just followed instructions and uploaded a generated file, I don't think the Eclipse Contributor Agreement (ECA) is required. If you think otherwise I can of course do that step.

What it does

Adds the result of

yarn --cwd packages/core generate-layout --output src/common/keyboard/layouts/de-Neo_2-pc.json

How to test

Choose Neo 2 Keyboard in (linux) keyboard settings. Then check if shortcuts work. (e.g. Copy + Paste) This is of course hard to do if you don't have the layout memorized :-/

Review checklist

No. I have just uploaded the generated file.

Reminder for reviewers

This is for gitpod-io/gitpod#1363 where I have executed the generate-layout script on an Ubuntu 20.10. (Windows 10 does not support Neo 2 layouts afaik)
@kreibaum
Copy link
Author

Looks like the ECA check is included in your CI so I'll make sure to sign that.

@kreibaum
Copy link
Author

I have now signed the ECA but I don't know how to retrigger the check.

@DucNgn
Copy link
Contributor

DucNgn commented Nov 14, 2020

@kreibaum Hi, in order to pass the ECA check, you will need to sign off your work in your git commit message (sign your work)

@@ -0,0 +1 @@
{"info":{"model":"pc105","layout":"de,de,us","variant":"neo,,","options":"","rules":"evdev"},"mapping":{"KeyA":{"value":"u","withShift":"U","withAltGr":"\\"},"KeyB":{"value":"z","withShift":"Z","withAltGr":"`","withShiftAltGr":"ζ"},"KeyC":{"value":"ä","withShift":"Ä","withAltGr":"|","withShiftAltGr":"η"},"KeyD":{"value":"a","withShift":"A","withAltGr":"{","withShiftAltGr":"α"},"KeyE":{"value":"l","withShift":"L","withAltGr":"[","withShiftAltGr":"λ"},"KeyF":{"value":"e","withShift":"E","withAltGr":"}","withShiftAltGr":"ε"},"KeyG":{"value":"o","withShift":"O","withAltGr":"*","withShiftAltGr":"ο"},"KeyH":{"value":"s","withShift":"S","withAltGr":"?","withShiftAltGr":"σ"},"KeyI":{"value":"g","withShift":"G","withAltGr":">","withShiftAltGr":"γ"},"KeyJ":{"value":"n","withShift":"N","withAltGr":"(","withShiftAltGr":"ν"},"KeyK":{"value":"r","withShift":"R","withAltGr":")","withShiftAltGr":"ρ"},"KeyL":{"value":"t","withShift":"T","withAltGr":"-","withShiftAltGr":"τ"},"KeyM":{"value":"m","withShift":"M","withAltGr":"%","withShiftAltGr":"μ"},"KeyN":{"value":"b","withShift":"B","withAltGr":"+","withShiftAltGr":"β"},"KeyO":{"value":"f","withShift":"F","withAltGr":"=","withShiftAltGr":"φ"},"KeyP":{"value":"q","withShift":"Q","withAltGr":"&","withShiftAltGr":"ϕ"},"KeyQ":{"value":"x","withShift":"X","withAltGr":"…","withShiftAltGr":"ξ"},"KeyR":{"value":"c","withShift":"C","withAltGr":"]","withShiftAltGr":"χ"},"KeyS":{"value":"i","withShift":"I","withAltGr":"/","withShiftAltGr":"ι"},"KeyT":{"value":"w","withShift":"W","withAltGr":"^","withShiftAltGr":"ω"},"KeyU":{"value":"h","withShift":"H","withAltGr":"<","withShiftAltGr":"ψ"},"KeyV":{"value":"p","withShift":"P","withAltGr":"~","withShiftAltGr":"π"},"KeyW":{"value":"v","withShift":"V","withAltGr":"_"},"KeyX":{"value":"ö","withShift":"Ö","withAltGr":"$","withShiftAltGr":"ϵ"},"KeyY":{"value":"k","withShift":"K","withAltGr":"!","withShiftAltGr":"κ"},"KeyZ":{"value":"ü","withShift":"Ü","withAltGr":"#"},"Digit1":{"value":"1","withShift":"°","withAltGr":"¹","withShiftAltGr":"₁"},"Digit2":{"value":"2","withShift":"§","withAltGr":"²","withShiftAltGr":"₂"},"Digit3":{"value":"3","withShift":"ℓ","withAltGr":"³","withShiftAltGr":"₃"},"Digit4":{"value":"4","withShift":"»","withAltGr":"›","withShiftAltGr":"♀"},"Digit5":{"value":"5","withShift":"«","withAltGr":"‹","withShiftAltGr":"♂"},"Digit6":{"value":"6","withShift":"$","withAltGr":"¢","withShiftAltGr":"⚥"},"Digit7":{"value":"7","withShift":"€","withAltGr":"¥","withShiftAltGr":"ϰ"},"Digit8":{"value":"8","withShift":"„","withAltGr":"‚","withShiftAltGr":"⟨"},"Digit9":{"value":"9","withShift":"“","withAltGr":"‘","withShiftAltGr":"⟩"},"Digit0":{"value":"0","withShift":"”","withAltGr":"’","withShiftAltGr":"₀"},"Minus":{"value":"-","withShift":"—","withShiftAltGr":"‑"},"Equal":{"value":"̀","withShift":"̧","withAltGr":"̊","withShiftAltGr":"̔"},"BracketLeft":{"value":"ß","withShift":"ẞ","withAltGr":"ſ","withShiftAltGr":"ς"},"BracketRight":{"value":"́","withShift":"̃","withAltGr":"̸","withShiftAltGr":"̓"},"Semicolon":{"value":"d","withShift":"D","withAltGr":":","withShiftAltGr":"δ"},"Quote":{"value":"y","withShift":"Y","withAltGr":"@","withShiftAltGr":"υ"},"Backquote":{"value":"̂","withShift":"̌","withAltGr":"↻","withShiftAltGr":"˞"},"Comma":{"value":",","withShift":"–","withAltGr":"\"","withShiftAltGr":"ϱ"},"Period":{"value":".","withShift":"•","withAltGr":"'","withShiftAltGr":"ϑ"},"Slash":{"value":"j","withShift":"J","withAltGr":";","withShiftAltGr":"θ"}}}
Copy link
Member

Choose a reason for hiding this comment

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

minor: please include a newline in addition to fixing the sign-off.

@vince-fugnitto vince-fugnitto added enhancement issues that are enhancements to current functionality - nice to haves keybindings issues related to keybindings labels Nov 16, 2020
@vince-fugnitto
Copy link
Member

@kreibaum do you still plan on updating the pull-request?
Unfortunately, we cannot accept the changes without a proper sign-off in the commit message which consequently fails the eca (EclipseFoundation Contributor Agreement) check.

@kreibaum
Copy link
Author

kreibaum commented Dec 7, 2020

Yes I do. Sorry that I didn't get around to that yet. Will still take a while to find time.

If you prefer, you can close this PR and I'll create a new one later.

@vince-fugnitto
Copy link
Member

Yes I do. Sorry that I didn't get around to that yet. Will still take a while to find time.

There's no rush, we look forward to your contribution :)

@kreibaum
Copy link
Author

With the Code IDE in Gitpod not having this problem and it becoming the default soon, I'll instead switch over to that and close this pull request.

@kreibaum kreibaum closed this Mar 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement issues that are enhancements to current functionality - nice to haves keybindings issues related to keybindings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants