-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
question: inconsistencies in keyboard shortcut formatting #2408
Comments
Hi @benboeck. Many thanks for this research and proposal. I am not home at the moment but I'll make sure to reply in detail in the next couple days. |
Hello @waldyrious, thank you for your reply, looking forward to it. |
Hey @benboeck everything about your proposal looks great to me, except the capital letter - it makes me think I need to press |
Hi @jedahan, I agree with you, this was also my initial reaction. I refrained from proposing a change for the following reasons:
That said, I could also totally live with |
For the sake of moving things forward, I like your original proposal for now, unless reviewers are excited about lowercase, we can revisit it another time. |
As discussed, I am undecided on That said, there are other questions, especially in commands (not descriptions) such as: common/vim:
and common/ed:
How would you want to deal with these? Proposal: |
We had discussed this before. We had honed down on a format. But I believe in some pages, this has strayed from the decided convention. |
Indeed -- that was why I wanted to set out some time to address this properly. There are related PRs/discussions at #1354 and #1356. After reviewing @benboeck's proposal more carefully, I agree with everything in it. It's actually perfectly in line with what had been previously discussed. By the way, since we now have a separate style guide, it probably makes sense to include this convention there.
I think the real question is related, but somewhat different: are we focusing on clarity or compactness? This is valid and useful for both beginners and experienced CLI users. IMO, the answer has got to be a balance, as both are primary goals of the project; but in this case, including Finally, regarding the use of |
Ah thanks for digging up the issue @waldyrious ! Yes, seems reasonable. So anything less than 3 keys, we explicitly mention Shift. If there are more, we just write the upper or lower case to clarify. Thanks for taking it up. |
Hello @waldyrious and @jedahan, I sway towards Thinking about consistency and the differences between the two use cases (one key combination vs. a number of key combinations and button presses), what about agreeing on just one style (see 2 and 3):
What do you think? Thanks, Ben |
1.i, 1.ii , 3 lgtm. How about we make a minimal amount of changes + a linter for the changes, then propose style improvements incrementally. If we have okay tooling, it should be okay instead of making a ton of changes all at once. |
Hello. great proposal,
This should make future linting easier. |
Yeah that would be nice to move us closer to unified until we figure out exactly where <> is best balanced |
@benboeck I'm afraid your new comment is not clear to me. It seems as if you're making a number of changes to the initial proposal, whose motivation I can't discern clearly, as well as introducing some new concepts that are ambiguous due to overlapping terminology. In contrast, your initial exposition was crystal clear, so I'm sure we can communicate about this effectively. Would you please take a shot at editing your comment to hopefully make it reflect your thoughts more clearly? Sorry if my confusion is not clear to you 😅 if so, let me know and I'll try to make a more thorough exposition of what's confusing me. |
Hi @waldyrious you are correct I proposed changes to the original proposal but in discussion with @jedahan they were reduced back to (almost like original proposal):
I would go through the existing pages and try to update deviations from the above rules if this is OK with you. Other possible changes could then be discussed and would be easier to introduce once the deviations are gone. |
Sounds great, @benboeck. The changes in your latest comment would be very welcome, and later changes can be discussed in due time. Looking forward to the PR. |
Hi @benboeck - any updates on this ? |
Hey! I think I missed this issue. I'd vote for the option that looks like |
I would like to restart the discussion in this thread.
I assume this is only for the description and not for the command itself. I don't know any pages where this is used in the description. I know in tmux.md and vim.md angle brackets are used to indicate keypresses. I think we need a uniform syntax for keypresses. For starters we need to decide on the syntax for special keys (angle brackets <> are probably good), then we need something to differentiate if keys are pressed together or separately. I personally would for example describe "Expand glob" feature of bash with tldr:
|
Hello,
writing a new TLDR client I noticed inconsistencies in the way keyboard shortcuts are formatted.
I am willing to go through the tedious task of preparing a pull request unifying them but want to ask for your agreement on a common style first:
Using a Ctrl shortcut as example, different formats include the following and more:
CTRL + K
vs.Ctrl + K
("CTRL" in caps or not?)CTRL + K
vs.CTRL-K
(minus sign, no space)CTRL + K
vs.CTRL + k
(lower case "k")CTRL + K
vs. CTRL + K (format as code or not?)<CTRL-K>
The same goes for many other keyboard buttons such as
Alt
,Tab
,Enter
and so on.Some examples of different styles in descriptions:
Ctrl + D
when finished):Some examples of different styles in commands:
Ctrl + K
<CTRL-x> o
Ctrl + w {{h|l|k|j}}
Proposal (aligned with currently most used styles):
Ctrl
instead of no code formatting (use code formatting)Ctrl + K
instead of other formats (use plus sign, use spaces)Ctrl + X, O
instead of<CTRL-x> o
Ctrl + X
instead ofCtrl + x
(use upper case)Ctrl + A / Ctrl + B
instead ofCtrl + A/Ctrl + B
(spaces around "/" look nicer - this is my personal opinion and currently NOT the most used form)Thank you,
kind regards
Ben
The text was updated successfully, but these errors were encountered: