From bb4f03c5823fca4a958bc173fcf2d280627c7e0b Mon Sep 17 00:00:00 2001
From: Chris Pearce <crite70@hotmail.com>
Date: Sun, 5 Apr 2015 16:33:11 +1000
Subject: [PATCH] Updates after Kate's code review

---
 objects/_o-link-disguised.scss |  2 +-
 objects/_o-list-block.scss     |  2 +-
 objects/_o-table.scss          |  2 +-
 utilities/_u-spacing.scss      | 27 +++++++++++++++------------
 4 files changed, 18 insertions(+), 15 deletions(-)

diff --git a/objects/_o-link-disguised.scss b/objects/_o-link-disguised.scss
index dcf0b33..68c72f7 100644
--- a/objects/_o-link-disguised.scss
+++ b/objects/_o-link-disguised.scss
@@ -7,7 +7,7 @@
  * Make a link not look like a link.
  *
  * @markup
-   <a href="#" class="o-link-disguided">Some link text</a>
+   <a href="#" class="o-link-disguised">Some link text</a>
  */
 
 
diff --git a/objects/_o-list-block.scss b/objects/_o-list-block.scss
index ba625d6..9a83fa6 100644
--- a/objects/_o-list-block.scss
+++ b/objects/_o-list-block.scss
@@ -8,7 +8,7 @@
  * vertical list of links.
  *
  * @markup
-   <ul class="o-arrow [modifier]">
+   <ul class="o-list-block [modifier]">
      <li>
        <a href="#">Some link text</a>
      </li>
diff --git a/objects/_o-table.scss b/objects/_o-table.scss
index 477a13e..c9df718 100644
--- a/objects/_o-table.scss
+++ b/objects/_o-table.scss
@@ -142,7 +142,7 @@ $o-table-striped-background-cell-colour:               $o-table-border-colour !d
 
 
 /**
- * Modifier: layout fixed, force tables into having equal-width column.
+ * Modifier: layout fixed, force tables into having equal-width columns.
  */
 
 %o-table--fixed,
diff --git a/utilities/_u-spacing.scss b/utilities/_u-spacing.scss
index 5af8ae5..086936f 100644
--- a/utilities/_u-spacing.scss
+++ b/utilities/_u-spacing.scss
@@ -53,6 +53,21 @@
  * 6. Spacing remove:           this is explicit
  * 7. Spacing amount:           this is explicit
  *
+ * As they're so many spacing utilities the format for applying them at different
+ * breakpoints in your master stylesheet is a little different to the
+ * standard format. In addition to listing your breakpoints we have to list out
+ * the exact spacing utility classes we want those breakpoints applied to. So
+ * say we wanted to apply 3 `margin-bottom` spacing utilities at the 'Small',
+ * 'Base', and 'Large' sizing categories which are applied to the 'Palm' and 'Lap'
+ * breakpoints plus at a breakpoint of '886', we would do it like this:
+ *
+   $u-spacing-apply-at-breakpoints: (palm, lap, 886);
+   $u-spacing-apply-classes-for-breakpoints: (
+    ".u-s-mb-small",
+    ".u-s-mb-base",
+    ".u-s-mb-large"
+   );
+ *
  * All the utilities (available as silent placeholder selectors also):
  *
    // Base
@@ -251,18 +266,6 @@
    .u-s-me-none
    .u-s-pe-none
  *
- * As they're so many spacing utilities the format for applying them at different
- * breakpoints in your master stylesheet is a little different to the
- * standard format. In addition to listing your breakpoints we have to list out
- * the exact spacing utilities we want those breakpoints applied to e.g.
- *
-   $u-spacing-apply-at-breakpoints: (palm, lap, 886);
-   $u-spacing-apply-classes-for-breakpoints: (
-    ".u-s-mb-small",
-    ".u-s-mb-base",
-    ".u-s-mt-base"
-   );
- *
  * @credit
  * https://github.com/stubbornella/oocss/blob/master/oocss/src/components/whitespace/_whitespace.scss
  */