Skip to content
This repository has been archived by the owner on Aug 3, 2021. It is now read-only.

Added stylesheet #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added stylesheet #17

wants to merge 1 commit into from

Conversation

cloewen8
Copy link

Added a stylesheet to improve readability.

  • Added a gutter to keep text out of the screen edge.
  • Switched to a sans-serif font (don't use a sans font on the web, it was designed for typesetting, not digital display).
  • Added a max-width to paragraphs. 30 characters is common, but this could be fine tuned later.
  • Corrected header scaling to create a hierarchy.
  • Added space between headers.

I avoided making changes to the html to avoid conflicts with other pull request (I am hoping to make these changes later). The styles are non-essential, so they were added to an external file.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@cloewen8
Copy link
Author

cloewen8 commented Jun 24, 2021

This prob wont get merged in for a few reasons. I want to redo this version site and move it onto fabricmc.net I also want to supply a json file listing all the javadoc versions

#5

I just noticed this comment after submitting. Might want to note that on the version site.
Some of these improvements could be made to the main fabricmc.net site. I would be happy to redo them, if it is on GitHub.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant