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

Switch to shields.io for badges in README #15380

Closed
wants to merge 1 commit into from
Closed

Conversation

hnrch02
Copy link
Collaborator

@hnrch02 hnrch02 commented Dec 16, 2014

Also consolidates the Bower and npm badge into one generic release badge.

/cc @XhmikosR @cvrebert

@XhmikosR
Copy link
Member

:shipit:

@XhmikosR XhmikosR added this to the v3.3.2 milestone Dec 16, 2014
@mdo
Copy link
Member

mdo commented Dec 16, 2014

Consider adding a line break or something after the third badge to avoid the weird vertical align? Or flip the order maybe? Or dear gawd is there an alternate for the Sauce tests lol?

screen shot 2014-12-16 at 12 12 31 am

@XhmikosR
Copy link
Member

I agree with the line break though it shouldn't be visible in https://github.com/twbs/bootstrap#readme only when viewing the README.md file.

@cvrebert
Copy link
Collaborator

How about we move the version badges to be under the Quickstart section? They're relevant there.

@cvrebert cvrebert added the docs label Dec 17, 2014
@XhmikosR
Copy link
Member

To be honest, I'm used to seeing the badges to the top.

But we can add a separate section, maybe even a table for the badges.

@hnrch02 hnrch02 closed this in c3e6111 Dec 30, 2014
@XhmikosR XhmikosR deleted the readme-badges branch December 30, 2014 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants