-
Notifications
You must be signed in to change notification settings - Fork 72
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 FnLock key #263
Add FnLock key #263
Conversation
825f115
to
d37ac12
Compare
Hello, is there a reason why this is left as draft and not merged? I've tested this change on |
d37ac12
to
6cd4335
Compare
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.
This works great on darp8.
I've opened pop-os/keyboard-configurator#159 so we can track getting it added to the configurator.
9528c30
to
fa2a4d8
Compare
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.
Re-tested on darp8 and it's still working well.
Sorry if my trying to update the branch messed something up 😅
fa2a4d8
to
cc745cf
Compare
Implement a FnLock toggle that behaves as follows: - Disabled: F1-F12 are normal - Enabled: F1-F12 are the alternate function Signed-off-by: Tim Crawford <[email protected]>
cc745cf
to
9876d9a
Compare
Default key has been changed from |
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.
Working nicely on the lemp12. I confused myself after resetting the keyboard layout to default in the configurator during testing, which made it stop working, but that's because the configurator doesn't yet support FnLock.
I have concerns with the lack of an indicator. This is a great thing for users who want it, but for users who are unaware of this feature, accidentally toggling function lock could be very hard for them to undo. |
9876d9a
to
a6b8c0d
Compare
I've dropped the commit adding it to the default layouts. Users will have to explicitly add the key to their layout if desired. |
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.
Sounds good to me
Implement a FnLock toggle and a custom key for it.
F1
-F12
behave normallyF1
-F12
behave as the alternate functionSet it to
Fn+Esc
by default.NOTE: Holding
Fn+Esc
at power on (S5 to S0) is used for resetting the EC configs. After S0,Fn+Esc
will only act as FnLock.NOTE: System76 laptops do not have an indicator for FnLock state.
Resolves: system76/firmware-open#78