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

Docs fixes for badges in buttons #23104

Merged
merged 3 commits into from
Aug 11, 2017
Merged

Docs fixes for badges in buttons #23104

merged 3 commits into from
Aug 11, 2017

Conversation

mdo
Copy link
Member

@mdo mdo commented Jul 16, 2017

This corrects some changes from #23060:

  • Buttons must have a base and modifier class. I've added .btn-primary to fix that.
  • Given the primary buttons, I've changed the badges to .badge-light for contrast.
  • .bd-example and {% highlight %} don't need to be separate, so I've combined them with our {% example %} blocks designed for these situations.
  • I've tweaked some docs content: added a serial comma, broke apart a rather large paragraph, and consolidated some text.

/cc @patrickhlauke

mdo added 2 commits July 16, 2017 12:15
- Given buttons a required modifier (.btn alone is incomplete)
- Given the primary button, use a .badge-light for better contrast
- Consolidate example and highlight code with a `{% example %}` block
Copy link
Contributor

@gijsbotje gijsbotje left a comment

Choose a reason for hiding this comment

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

found no typo's and looks good

@twbs twbs deleted a comment Aug 11, 2017
@mdo mdo merged commit 5b584c2 into v4-dev Aug 11, 2017
@mdo mdo deleted the btn-badge-fixes branch August 11, 2017 06:00
@mdo mdo mentioned this pull request Aug 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants