Skip to content

Commit

Permalink
docs(Button): Move around the disabled button docs for clarity.
Browse files Browse the repository at this point in the history
  • Loading branch information
ryanoglesby08 committed Aug 10, 2017
1 parent 2e40da3 commit 6a74ccc
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions src/components/Button/Button.md
Original file line number Diff line number Diff line change
@@ -1,14 +1,15 @@
## Minimal Usage

Provide a function as the `onClick` prop to perform an action when clicked. **Avoid using a button if navigation
is the primary action, as a link is more appropriate.**
is the button's primary action, as a link is more appropriate.**

### Recommendations

* Use buttons to move though a transaction.
* Aim to use only one button per page.
* Avoid excessively long button text.
* Make sure the button text describes an action.
* Avoid disabling buttons. Use error messaging to provide feedback rather than a disabled button.

By default, Buttons will be displayed in the `primary` variant. Use primary buttons for the main action on a page or
in a form.
Expand Down Expand Up @@ -56,7 +57,3 @@ const Hero = require('./__docs__/Hero').default;
<Button variant="outlined" invert>Advanced solutions</Button>
</Hero>
```

## Disabling buttons

We prefer buttons to always be interactive, so there is no need to disable it.

0 comments on commit 6a74ccc

Please sign in to comment.