-
Notifications
You must be signed in to change notification settings - Fork 4
Add HTML export target #40
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
base: master
Are you sure you want to change the base?
Add HTML export target #40
Conversation
Introduced a new "Embed" option in the dropdown menu that allows users to copy an iframe code for embedding the keyboard layout. The code includes the keyboard image and metadata, and handles both modern and fallback clipboard copying methods. This enhances the sharing capabilities of the keyboard layouts.
|
@adamws @eugenesvk Instead of a download HTML, I made it an embed button. |
|
Preview deployment ready! Preview URL: https://kle-kel7u8sap-adams-projects-465f639f.vercel.app Built from commit: d16051c |
|
Thanks for looking into it, but it's not what I meant. For be the huge benefit of html export is that it would human/machine parseable/searchable. For example, I can save a local offline html cheatsheet and lookup a symbol via Ctrl+F. Or embed html in a blog with the same properties, and then you could also attach extra style/tooltip/popup stuff since every button is a dedicated html entity. For example, with a KLE html you can have a button that simply replaces all the QWERTY labels with DVORAK with a little script, it won't work with an embedded image |
|
Thanks for the clarification. I’ll revise my approach based on your explanation and update the implementation. |
|
To be on the same page, https://codepen.io/adamws/pen/JoXxJxO
This whole feature would basically become separate html based renderer (like the original keyboard-layout-editor had). This is major feature. I'm not personally invested in it but I'm ok with adding it if implemented. |
|
@eugenesvk
|
|
|
@eugenesvk Additionally, if you select “ANSI 104” in the presets I was able to implement the feature itself, but I don't think I'll be able to allocate any more time to testing and fixes. Since it will be difficult to update this PR going forward, I apologize for the situation turning out this way. |
|
Preview deployment ready! Preview URL: https://kle-9ub75r5e0-adams-projects-465f639f.vercel.app Built from commit: d16051c |



#27