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

Remove charlooks field references in TEDITKEY and TKDORADO #2020

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rmkaplan
Copy link
Contributor

@rmkaplan rmkaplan commented Feb 7, 2025

I'm not sure whether/how these work, but I removed these obvious dependencies on Tedit internals

@masinter
Copy link
Member

masinter commented Mar 3, 2025

does "I'm not sure whether/how these work" mean it's untested?

@rmkaplan
Copy link
Contributor Author

rmkaplan commented Mar 3, 2025 via email

@rmkaplan rmkaplan marked this pull request as draft March 4, 2025 07:06
@rmkaplan
Copy link
Contributor Author

rmkaplan commented Mar 4, 2025

I converted this to draft. Although the documentation for TEDITKEY focuses on the behavior attached to special Dorado/Dandelion keys, it also specifies ctrl and meta equivalents that should work on standard keyboards. Some function-renaming and field-renaming that happened some time ago did not get applied to this file, so I need to do a more systematic update. The good news is that most of the equivalent keys do work as specified in the venue sysout, so we do have an operational baseline.

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