Various fixes for the 'qmk' cli
See the individual commit messages.
Wider key sizes will make most layouts unreadable on default terminal widths.
Wider key sizes will make most layouts unreadable on default terminal widths.
Yes, it will often make it wider than 80 characters. I don't know if 80 still can be called the default terminal width these days. If users should face a problem with that, it is quite obvious how to solve it.
But even then, that seems like a smaller problem than the completely unusable output as shown in fe59798 .
Is 80 characters a hard limit for QMK?
If so, how do you look at the problem with short but unusable output? Any idea for making a compromise?
Also, while discussing this issue, it would be nice if you can cherry-pick previous more trivial changes from this PR.
I rebased and changed the code so it keeps the existing key width. The PR is thus only cleanup and fixes.
Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.
Thank you for your contribution!
This pull request has been automatically marked as stale because it has not had activity in the last 45 days. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, or re-open when it's ready.
For maintainers: Please label with bug, awaiting review, breaking_change, in progress, or on hold to prevent the issue from being re-flagged.
Thank you for your contribution! This pull request has been automatically closed because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. // [stale-action-closed]