From 32ae3ffdf623a4acf1557813615761af1b6db1a1 Mon Sep 17 00:00:00 2001 From: Shawn Allen Date: Wed, 20 Jun 2018 14:02:03 -0700 Subject: [PATCH 1/3] remove btn-outline-blue docs references --- modules/primer-buttons/README.md | 5 +---- modules/primer-buttons/stories.js | 2 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/modules/primer-buttons/README.md b/modules/primer-buttons/README.md index 3ebbe61e36..697139e781 100644 --- a/modules/primer-buttons/README.md +++ b/modules/primer-buttons/README.md @@ -95,10 +95,9 @@ Use `.btn-large` to increase the padding and border radius of a button. This is [Type scale utilities](https://styleguide.github.com/primer/utilities/typography/#type-scale-utilities) can be used to alter the font-size if needed. Padding is applied in em's so that it scales proportionally with the font-size. ```html -

Large link button - +

``` @@ -106,12 +105,10 @@ Use `.btn-large` to increase the padding and border radius of a button. This is Use `.btn-large` with a type scale utility to transform the text to a bigger size. ```html -

Large link button

- ``` ## Disabled state diff --git a/modules/primer-buttons/stories.js b/modules/primer-buttons/stories.js index bb443ddda0..a111986975 100644 --- a/modules/primer-buttons/stories.js +++ b/modules/primer-buttons/stories.js @@ -62,7 +62,7 @@ storiesOf('Button', module) .add('btn-large', () => (
Large link button - +
)) .add('btn-link', () => ( From 6ccfe8e3e269094d4a4f524a39dacd64998492fe Mon Sep 17 00:00:00 2001 From: Shawn Allen Date: Wed, 20 Jun 2018 14:04:27 -0700 Subject: [PATCH 2/3] remove btn-secondary references --- modules/primer-buttons/stories.js | 9 --------- modules/primer-tooltips/stories.js | 4 ++-- modules/primer-utilities/docs/layout.md | 2 +- 3 files changed, 3 insertions(+), 12 deletions(-) diff --git a/modules/primer-buttons/stories.js b/modules/primer-buttons/stories.js index a111986975..81bd0a1997 100644 --- a/modules/primer-buttons/stories.js +++ b/modules/primer-buttons/stories.js @@ -14,15 +14,6 @@ storiesOf('Button', module) )) - .add('btn-secondary', () => ( -
- - - - - -
- )) .add('btn-danger', () => (
diff --git a/modules/primer-tooltips/stories.js b/modules/primer-tooltips/stories.js index 579876b385..9d52d3cd7f 100644 --- a/modules/primer-tooltips/stories.js +++ b/modules/primer-tooltips/stories.js @@ -77,6 +77,6 @@ storiesOf('Tooltips', module) )) .add('tooltip on button', () => (
- +
-)) + )) diff --git a/modules/primer-utilities/docs/layout.md b/modules/primer-utilities/docs/layout.md index cb2082ddf9..8401b9c024 100644 --- a/modules/primer-utilities/docs/layout.md +++ b/modules/primer-utilities/docs/layout.md @@ -204,7 +204,7 @@ Use `.position-absolute` to take elements out of the normal document flow. ```html
- +
Mashed potatoes Fries From a6c8d855ed4dea5124a241bf8312176ddb3fcb3e Mon Sep 17 00:00:00 2001 From: Shawn Allen Date: Wed, 20 Jun 2018 14:04:48 -0700 Subject: [PATCH 3/3] clean up tooltip storybook JS --- modules/primer-tooltips/stories.js | 110 ++++++++++++++--------------- 1 file changed, 55 insertions(+), 55 deletions(-) diff --git a/modules/primer-tooltips/stories.js b/modules/primer-tooltips/stories.js index 9d52d3cd7f..8a2f985034 100644 --- a/modules/primer-tooltips/stories.js +++ b/modules/primer-tooltips/stories.js @@ -2,61 +2,61 @@ import React from 'react' import { storiesOf } from '@storybook/react' storiesOf('Tooltips', module) -.add('tooltipped direction', () => ( -
- - Tooltip North - - - Tooltip North East - - - Tooltip East - - - Tooltip South East - - - Tooltip South - - - Tooltip South West - - - Tooltip West - - - Tooltip North West - -
-)) -.add('tooltip aligned', () => ( -
- - Tooltip North East align left 1 - - - Tooltip North East align left 2 - - - Tooltip South East align left 1 - - - Tooltip South East align left 2 - - - Tooltip North West align right 1 - - - Tooltip North West align right 2 - - - Tooltip South West align right 1 - - - Tooltip South West align right 2 - -
+ .add('tooltipped direction', () => ( +
+ + Tooltip North + + + Tooltip North East + + + Tooltip East + + + Tooltip South East + + + Tooltip South + + + Tooltip South West + + + Tooltip West + + + Tooltip North West + +
+ )) + .add('tooltip aligned', () => ( +
+ + Tooltip North East align left 1 + + + Tooltip North East align left 2 + + + Tooltip South East align left 1 + + + Tooltip South East align left 2 + + + Tooltip North West align right 1 + + + Tooltip North West align right 2 + + + Tooltip South West align right 1 + + + Tooltip South West align right 2 + +
)) .add('tootlipped-multiline', () => (