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

Add less/scss files to getting started #8878

Closed
wants to merge 3 commits into from

Conversation

JeffryBooher
Copy link
Contributor

  • Add less/scss files to getting started
  • NOTE: This merges into the Release branch

colors, and animation timing functions--and we're adding more and more all the time.
colors, and animation timing functions -- and we're adding more and more all the time.
You can even use the quick edit feature to edit rules defined in LESS/SCSS files even if they
haven't been processed into CSS files.<br/><br/>
Copy link

Choose a reason for hiding this comment

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

Seems like it would make sense to put this above "You can use the same shortcut to edit other things as well", since this is more related to the CSS example than to the other cases. Also, I don't think we need to explain that it works without the preprocessor running. So, maybe we could move it to the end of the first paragraph above - i.e., after "If you hit ESC outside of a quick inline editor, they'll all collapse", you could add "Quick Edit will also find rules defined in LESS and SCSS files, including nested rules."

@@ -65,8 +66,9 @@
</a>

<p>
You can use the same shortcut to edit other things as well--like functions in JavaScript,
colors, and animation timing functions--and we're adding more and more all the time.
You can use the same shortcut to edit other things as well &mdash; like functions in JavaScript,
Copy link
Member

Choose a reason for hiding this comment

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

Nit: an M dash shouldn't have spaces around it. If we want spaces around it (which is probably good for readability of the source code, which is important since reading this code is part of the tutorial), we should use &ndash; instead.

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.

3 participants