Skip to content
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

feat(docs): updates to the glossary #14939

Merged
merged 12 commits into from
Jun 24, 2019
Merged

feat(docs): updates to the glossary #14939

merged 12 commits into from
Jun 24, 2019

Conversation

marcysutton
Copy link
Contributor

@marcysutton marcysutton commented Jun 20, 2019

Description

In reviewing #14846 I wanted to add a link to "backend" in the glossary, but I noticed both that term and "frontend" weren't written to be consistent with the Gatsby Style Guide. I added a few terms that I've noticed were missing and updated a few terms, plus added a navigation block to the top (suggested improvements below). Finally, I noticed the Client Data Fetching doc wasn't actually in the sidebar despite a whole PR for that so I added it.

Something I could use help on is writing the navigation block more dynamically so we can give it better semantics. Right now the Markdown looks like this:

[A](#a) | [B](#b) | [C](#c) | [D](#d) | [E](#e) | [F](#f) | [G](#g) | [H](#h) | [I](#i) | [J](#j) | [K](#k) | [L](#l) | [M](#m) | [N](#n) | [O](#o) | [P](#p) | [Q](#q) | [R](#r) | [S](#s) | [T](#t) | [U](#u) | [V](#v) | [W](#w) | [X](#x) | [Y](#y) | [Z](#z)

The two things I think need to change for that are:

  • wrap the list in <nav><ul></ul></nav>
  • wrap each item in an <li> use a border for the separators instead of pipe characters

Seems like a job for MDX. @gillkyle could you help me with that?

@marcysutton marcysutton requested a review from a team June 20, 2019 00:28
@marcysutton marcysutton requested a review from a team as a code owner June 20, 2019 00:28
@gillkyle
Copy link
Contributor

I can look into this later today, I think starting to use components for tables and navigation like this will be really helpful 👍

@gillkyle
Copy link
Contributor

Here's a comparison of the old vs new (the old one was removed in favor of the MDX component that's shown on the bottom):

mdx-rel-headers

@lannonbr lannonbr added the type: documentation An issue or pull request for improving or updating Gatsby's documentation label Jun 21, 2019
@marcysutton
Copy link
Contributor Author

I added two new terms: CSS and Progressive Enhancement. Also added a link to #frontend about How Browsers Work. It's such a great resource!

Copy link
Contributor

@gillkyle gillkyle left a comment

Choose a reason for hiding this comment

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

Love these definitions, super awesome! I particularly like the Themes definition.

@@ -84,6 +97,10 @@ The process of [building](#build) your website or app and uploading onto a [host

The [environment](#environment) when you're developing your code. It's accessed through the [CLI](#cli) using `gatsby develop`, and provides extra error reporting and things to help you debug before building for [production](#production-environment).

### DOM

The Document Object Model, commonly referred to as "the DOM", is a standard browser API that connects web pages to scripts or programming languages by representing the structure of an HTML document in memory. Developers commonly interact with the DOM through [HTML](#html) markup (written in [JSX](#jsx) in Gatsby), as well as both [React](https://reactjs.org/docs/react-dom.html) and [vanilla JavaScript](https://developer.mozilla.org/en-US/docs/Web/API/Document_Object_Model/Introduction#DOM_and_JavaScript) code. Another important aspect of utilizing the DOM to its full potential is writing [accessible](#accessibility) HTML markup to expose a page's structure to assistive technology.
Copy link
Contributor

Choose a reason for hiding this comment

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

Reading this definition alone makes me want to link to this page for anyone trying to find info to start learning about web dev 🥇 these are awesome!

Marcy Sutton and others added 3 commits June 21, 2019 17:19
@marcysutton
Copy link
Contributor Author

Going to merge this since it's blocking another PR!

@marcysutton marcysutton merged commit 6582cc4 into master Jun 24, 2019
@marcysutton marcysutton deleted the glossary-updates branch June 24, 2019 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation An issue or pull request for improving or updating Gatsby's documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants