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

USWDS-Site: Phase two compile updates #1766

Merged
merged 10 commits into from Oct 11, 2022
Merged

USWDS-Site: Phase two compile updates #1766

merged 10 commits into from Oct 11, 2022

Conversation

amyleadem
Copy link
Contributor

@amyleadem amyleadem commented Sep 9, 2022

Summary

Restructured the content in Phase 2 of the Getting Started for Developers guide to add clarity.

Related issue

Closes #1708

Recommend releasing this update with the settings page update in PR #1755

Preview link

Preview link: Phase 2 Compile

Problem statement

The guide for compiling USWDS Sass should give clear instructions for both users who want to use uswds-compile to create their Sass, and those who want to manage their own Sass compilation. However, the current page creates a single set of steps for both processes. Additionally, the instructions for uswds-compile can be confusing if the user is not very familiar with Sass compilation.

Solution

By separating out the sections and adding clear headers, we can help the user understand which steps they need to take for their project. Additionally, adding more information about setting up the gulpfile.js for uswds-compile will help give users a clear step-by-step process to get up and running.

Testing and review

For copyeditors:

  • Proofread for errors and clarity
  • Confirm that the page structure makes sense and is easy to follow

For developers:

  • Confirm that all tasks and items listed are accurate and comprehensive
  • Confirm that the page structure makes sense and is easy to follow


## Step 4: Create path settings and export compile functions
Now, create your path settings using the following table:
### Step 4: Customize path settings
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added detail to explain what the path settings are for and examples that show how a user might want to customize

exports.watch = uswds.watch;
```

### Step 3: Set USWDS version
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added this step to highlight the need for setting the USWDS version, as well as describe which values are available.

Copy link
Contributor

@bonnieAcameron bonnieAcameron left a comment

Choose a reason for hiding this comment

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

This is AMAZING and I think the improvements are really concrete.

I reviewed this for clarity, plain language (within reason). The main changes I've suggested are making sure to link to relevant documentation when there's a lot to know or unpack behind a seemingly simple step (like "load uswds settings"), but I absolutely defer to your knowledge of their knowledge.


A project often has many Sass files, but typically, there’s a single file that serves as the root — the “homepage” of the Sass — that links out to the others. This root file is also known as the “Sass entry point.” The Sass entry point is the most important stylesheet file in your project because it tells the compiler what source files make up your Sass codebase.
When your entry point is ready, you'll need to load USWDS Sass. Your Sass entry point will need to do the following three tasks in this order:
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a link to USWDS Sass or an explanation of how to do this that we could link to?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have added some language ("For your entry point to complete these tasks") immediately after these tasks are identified to help transition the user. Let me know if you think it needs more.

I am also wondering if USWDS Sass sounds a bit like its own product, rather than just referring to the Sass files inside USWDS. Do you see this too?

@amyleadem
Copy link
Contributor Author

@bonnieAcameron made some updates:

  1. Added two steps: Step 3: import uswds-compile package) and Step 4: Set USWDS Version so that we create explicit instructions and explanations for each of our tasks listed in Step 2
  2. Linked each task in the uswds-compile section to its corresponding step
  3. Rewrote some pieces of the About Sass entry points guide so that each piece is tied to number 1., 2., 3. from the task list
  4. Generally just tried to add more clarity throughout.

Let me know if you have thoughts!

@amyleadem amyleadem marked this pull request as ready for review September 21, 2022 15:04
@amyleadem
Copy link
Contributor Author

@mejiaj We are working through some restructuring of the Phase 2 compile page. Will you take a look and let us know what you think?

Copy link
Contributor

@mejiaj mejiaj left a comment

Choose a reason for hiding this comment

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

LGTM! Only requested change is an extra the in a sentence.

Copy link
Contributor

@mejiaj mejiaj left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!


After you import the USWDS source code, you can build new styles with USWDS design tokens, functions, and mixins. For the purposes of this guide, we won’t get into custom code, but the important thing to understand is that any custom code should follow the settings and USWDS source code in your Sass entry point.
After you've loaded the USWDS source code, you can build new styles with USWDS design tokens, functions, and mixins. For the purposes of this guide, we won’t get into custom code, but the important thing to understand is that any custom code should follow the settings and USWDS source code in your Sass entry point.
Copy link
Contributor

Choose a reason for hiding this comment

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

What if they did want to get into "custom code," is there a resource for that that we could add?

Copy link
Contributor

@bonnieAcameron bonnieAcameron left a comment

Choose a reason for hiding this comment

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

Looks great! I checked for clarity and typos.

I added a few comments with ideas for improvements but they are minor and I consider them optional.

@thisisdano thisisdano merged commit fdcf9d9 into main Oct 11, 2022
@thisisdano thisisdano deleted the al-add-table-context branch October 11, 2022 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide more context around table
4 participants