Skip to content

[WIP] Add suport for none, trans, etc in the quick editor.#1

Closed
Copilot wants to merge 1 commit into
nonefrom
copilot/fix-07496c4a-cec3-48f1-9f05-23e4eb49593c
Closed

[WIP] Add suport for none, trans, etc in the quick editor.#1
Copilot wants to merge 1 commit into
nonefrom
copilot/fix-07496c4a-cec3-48f1-9f05-23e4eb49593c

Conversation

Copy link
Copy Markdown

Copilot AI commented Sep 17, 2025

Analysis: Add support for none, trans, etc in the quick editor

Current State Analysis

  • Explored repository structure and identified key components
  • Found that &trans and &none behaviors already exist in zmk-behaviors.json
  • Identified current key editing flow using ValuePicker modal
  • Located key components: Key.js, KeyboardLayout.js, ValuePicker/index.js
  • Understood current defaults: &none for undefined keys, &trans for new layers

Problem Understanding

The "quick editor" appears to refer to a way to quickly set common behaviors like &none, &trans without having to:

  1. Click on a key to open ValuePicker modal
  2. Search through behaviors to find the desired option

Implementation Plan

  • Add keyboard shortcuts for quick behavior assignment (e.g., 'n' for none, 't' for trans)
  • Add right-click context menu with common behaviors
  • Ensure the quick actions work for both direct key clicks and behavior selection
  • Add visual feedback and help text for the new quick editor features
  • Test the implementation with different key types and scenarios

Next Steps

  1. Implement keyboard event handling for quick shortcuts
  2. Add context menu functionality
  3. Update UI to show available quick actions
  4. Test thoroughly

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@BrokenStandards
Copy link
Copy Markdown
Owner

rejected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants