Skip to content

Comments

docs: glossary for keyboard author#1827

Merged
mcdurdin merged 18 commits intomasterfrom
glossary-keyboard-author
Apr 2, 2025
Merged

docs: glossary for keyboard author#1827
mcdurdin merged 18 commits intomasterfrom
glossary-keyboard-author

Conversation

@Meng-Heng
Copy link
Collaborator

@Meng-Heng Meng-Heng commented Jan 28, 2025

Fix: Build a glossary page for keyboard authors (developer/language/guide?) (link to Google Sheet)

I put together a glossary page with a few new glossaries but the others are from Keyman Glossary

@mcdurdin, @darcywong00, @LornaSIL, could I ask for your input on what else to add?


Questions and Request:

  1. Is developer/language/guide a suitable location for this? ✔️ Answered
  2. Appearance preferences? ✔️ Answered
  • image
  • image
  1. Please help add more to the content.

Thank you!

@Meng-Heng Meng-Heng marked this pull request as draft January 28, 2025 09:30
@Meng-Heng Meng-Heng marked this pull request as draft January 28, 2025 09:30
@Meng-Heng Meng-Heng marked this pull request as ready for review January 29, 2025 06:19
@LornaSIL
Copy link
Contributor

I would like us to define "package" and then maybe start using the word "bundle" for when we put 2 or more keyboards together into one distribution.

Then, I'd like the packages folder to be renamed bundles (there are no packages folders in experimental or legacy. What do you think @mcdurdin ?

@darcywong00 darcywong00 added this to the B18S1 milestone Feb 1, 2025
@mcdurdin
Copy link
Member

mcdurdin commented Feb 2, 2025

Then, I'd like the packages folder to be renamed bundles (there are no packages folders in experimental or legacy. What do you think @mcdurdin ?

I think this needs to be a separate discussion. The term 'bundle' or 'bundling' was used in the past for bundling Keyman together with keyboard packages, so I am hesitant to overload the term now. But a change like this needs to be in a separate issue on the keyboards repo if we want to explore it.

For this PR, it'd be good to define existing terminology as clearly as possible, without changes. We can iterate on changes once we clarify how we already use terminology.

@mcdurdin
Copy link
Member

mcdurdin commented Feb 2, 2025

I still need to review the terms in the list; I think there are quite a few we still need to define, and some of the definitions probably need a bit of input from me and others on the team.

@ermshiperete
Copy link
Contributor

As for the appearance, I like the first option better. I think it makes it easier to see the terms if they show up as headings, plus I like it to see the terms in the index as well.

@darcywong00 darcywong00 modified the milestones: B18S1, B18S2 Feb 15, 2025
@darcywong00 darcywong00 modified the milestones: B18S2, B18S3 Feb 28, 2025
@darcywong00 darcywong00 modified the milestones: B18S3, B18S4 Mar 14, 2025
@Meng-Heng Meng-Heng requested a review from mcdurdin March 19, 2025 04:52
@ermshiperete
Copy link
Contributor

You might want to look at my comments in the "Files Changed" tab - here in the "Conversations" tab you loose the context which makes some of them hard to understand.

@Meng-Heng
Copy link
Collaborator Author

Thanks @ermshiperete!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the file should be called glossary.md

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is already renamed to that.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, sorry, the comment was pending on my review and could not be deleted due to the file rename! Ironic.


## M {#m-glossary}

### Mnemonic Keyboard Layout
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should link to the &mnemoniclayout system store documentation here too

Copy link
Contributor

@ermshiperete ermshiperete left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Co-authored-by: Eberhard Beilharz <ermshiperete@users.noreply.github.com>
Comment on lines 49 to 72
```plain
keyboard
│ HISTORY.md: A place to record changes you make to the project over time.
│ LICENSE.md: Explains the rights that you give to others.
│ README.md: Provides an introduction to the keyboard project when others stumble across it.
│ keyboard.kpj: The main project file, contains references to all the components: keyboards, models, and packages.
│ keyboard.kpj.user: A user preference file. You can safely delete it at any time.
├───build
| keyboard.keyboard_info: A metadata file detailing the keyboard's origin, version, requirements, and capabilities.
│ keyboard.kmp: The installable package file.
│ keyboard.kmx: The compiled keyboard. This intermediate file should not be distributed.
│ keyboard.kvk: The compiled on screen keyboard. This intermediate file should not be distributed.
│ keyboard.js: The keyboard compiled to Javascript for use with KeymanWeb.
└───source
keyboard.ico: The icon of the keyboard.
readme.htm: Introductory web page for end users of your keyboard.
welcome.htm: A web page which describes how to use your keyboard.
keyboard.keyman-touch-layout: A touch layout file description.
keyboard.kmn: The keyboard source file.
keyboard.kps: The package source, which is used to create a compressed .kmp.
keyboard.kvks: The on screen keyboard template for desktop platforms.
```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure about the advisability of including lots of content in the glossary that is also elsewhere. It's likely to get out of sync in the future as we edit. (@sgschantz you like glossaries, what do you think?)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tagging @sgschantz again

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It can be painful to maintain two locations with overlapping content because they are eventually going to diverge.

If we look at it from the perspective of the users, then we need to ask what will make their lives easier. I think it is justified to have two glossaries if the target audience is different enough. We also don't want either audience to have to know about both glossaries and check the second if the first does not contain the term they need. It's hard to remember where one is located.

So that means that we either repeat information to make it easy for every that's using either glossary, or the glossaries link to each other. Linking between glossaries is still pretty messy, because the user is still jumping back and forth.

I guess repeating information is probably best while possibly simplifying the entry if it is for a less technical audience.

The only other option I can think of is to have a single glossary that contains everything and is not specialized for different types of users. Maybe it could still cater to multiple audiences with an indicator on some entries as to identify them as more technical.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your suggestions @sgschantz.

How about we just have this to show file and folder structure, remove the short description, and with the use of a link (https://help.keyman.com/developer/current-version/reference/file-layout) to allow user to read more?

This way; I'm taking your concern into consideration, @mcdurdin; the future changes won't be too out of sync or hard to update.

Therefore, this glossary will provide just enough information for a less technical audience while also be a gateway to a more technical information.

Hopefully I have concluded it in the middle, or have I not made any progress on the concern?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sgschantz, this wasn't really about two glossaries, but rather the glossary copying content out of other documentation. On multiple glossaries, in terms of two different audiences, I think that will necessitate duplication. The glossaries we have for Keyman team developers will overlap with info for integrators and also with info for keyboard authors. I'm okay with that. Ideally the content for each term is tailored to the target audience, and even could be cross-linked between glossaries.

@Meng-Heng wrote:

How about we just have this to show file and folder structure, remove the short description, and with the use of a link (https://help.keyman.com/developer/current-version/reference/file-layout) to allow user to read more?

I think rather than showing the file layout here at all, just link to the file layout topic. Just have a single sentence stating that we have a standard file layout for Keyman keyboard projects.

(The file types list may be okay, perhaps take .ttf and .ico out of the list as they are not really specific to Keyman)

Therefore, this glossary will provide just enough information for a less technical audience while also be a gateway to a more technical information.

@Meng-Heng, that's perfect. A glossary should be a gateway to deeper information.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, I will make the changes later this afternoon.

Copy link
Member

@mcdurdin mcdurdin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, one question for Shawn

@darcywong00 darcywong00 modified the milestones: B18S4, B18S5 Mar 29, 2025
@Meng-Heng Meng-Heng requested a review from mcdurdin April 2, 2025 08:21
@mcdurdin mcdurdin merged commit 7a85b8b into master Apr 2, 2025
3 checks passed
@mcdurdin mcdurdin deleted the glossary-keyboard-author branch April 2, 2025 22:55
@github-project-automation github-project-automation bot moved this to Done in Keyman Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants