Move the link farm to the top, to the same line as the header.#187
Open
jordanbrown0 wants to merge 1 commit intoopenscad:mainfrom
Open
Move the link farm to the top, to the same line as the header.#187jordanbrown0 wants to merge 1 commit intoopenscad:mainfrom
jordanbrown0 wants to merge 1 commit intoopenscad:mainfrom
Conversation
Member
|
I'm not that fussed, but the usefulness of that line is pretty low (particularly the last item), it doesn't deserve to be prominent. |
Contributor
Author
|
Based on the available history, it looks like that last entry belongs to one of the authors of the cheat sheet, so I'm inclined to keep it as a hat tip to them. The rest... shrug. Yeah, I have all of those URLs more or less memorized, but some people coming to the cheat sheet might not. If you don't have them memorized, then they are important. (Except for MCAD.) And, appended to that header line, it's not taking up any more space. But like I said, I'm not emotional about it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
It occurred to me that the link farm line at the bottom of the cheat sheet would fit to the right of the header, reducing the vertical size of the page, and would arguably be even better there.
I don't know of an easy way to view a github HTML page, but here's what it looks like at full width:
Note that github will squash the image here to fit.
I can't say that I'm super happy with the vertical spacing if you narrow the window further. It's usable, but not aesthetically pleasing. I expect that there's a way to move the entire line down if space requires, but I'd rather not mess with that until I get some buy-in on the overall concept. Let me know what you think. Here's what it looks like when taken narrower.