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

Various keymap editor fixes #91

Merged
merged 3 commits into from Dec 21, 2018
Merged

Various keymap editor fixes #91

merged 3 commits into from Dec 21, 2018

Conversation

algernon
Copy link
Collaborator

These changes make read-only keymaps uneditable, fixing #79, and also mark them (RO) in the tab bar, fixing #86.

When checking whether the current layer is read-only or not, use `this.state`
instead of `this.props`.

Signed-off-by: Gergely Nagy <algernon@keyboard.io>
Along with the previous change, this makes read-only layers uneditable.

Fixes #79.

Signed-off-by: Gergely Nagy <algernon@keyboard.io>
Fixes #86.

Signed-off-by: Gergely Nagy <algernon@keyboard.io>
@algernon algernon added bug Something isn't working page: keymap editor labels Dec 21, 2018
@algernon algernon added this to the Chrysalis 0.1.0 milestone Dec 21, 2018
@obra obra merged commit 710a068 into master Dec 21, 2018
@algernon algernon deleted the keymap-editor/roLayers branch December 21, 2018 23:06
0957758592 referenced this pull request in Dygmalab/Bazecor Nov 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants