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

New button styles #6

Closed
owenversteeg opened this Issue Jan 10, 2014 · 18 comments

Comments

Projects
None yet
@owenversteeg
Owner

owenversteeg commented Jan 10, 2014

Some people have criticized the current button styles, so this thread is where people can vote on button styles.

Current buttons:
Buttons

New button design A:
Buttons

New button design B:
Buttons

New button design C:
Buttons

Please vote (with comments) for which button design you like.

@ghost ghost assigned owenversteeg Jan 10, 2014

@dkuntz2

This comment has been minimized.

Show comment
Hide comment
@dkuntz2

dkuntz2 Jan 14, 2014

Why have the rounded corners? Why not just get rid of the shadow? The shadow's the only thing that looks funky to me.

dkuntz2 commented Jan 14, 2014

Why have the rounded corners? Why not just get rid of the shadow? The shadow's the only thing that looks funky to me.

@owenversteeg

This comment has been minimized.

Show comment
Hide comment
@owenversteeg

owenversteeg Jan 14, 2014

Owner

Hmm. I've added an option (C) which is the buttons with no rounded corners and no shadow.

That said, I don't think that the square buttons show affordance. To each their own, I suppose.

Owner

owenversteeg commented Jan 14, 2014

Hmm. I've added an option (C) which is the buttons with no rounded corners and no shadow.

That said, I don't think that the square buttons show affordance. To each their own, I suppose.

@scottaohara

This comment has been minimized.

Show comment
Hide comment
@scottaohara

scottaohara Jan 17, 2014

Collaborator

changing my vote to C.

Collaborator

scottaohara commented Jan 17, 2014

changing my vote to C.

@nsanden

This comment has been minimized.

Show comment
Hide comment
@nsanden

nsanden Jan 19, 2014

Definitely B

nsanden commented Jan 19, 2014

Definitely B

@owenversteeg

This comment has been minimized.

Show comment
Hide comment
@owenversteeg

owenversteeg Jan 19, 2014

Owner

I vote A

Owner

owenversteeg commented Jan 19, 2014

I vote A

@rofrol

This comment has been minimized.

Show comment
Hide comment
@rofrol

rofrol commented Jan 28, 2014

B

@enkrates

This comment has been minimized.

Show comment
Hide comment
@enkrates

enkrates Jan 30, 2014

I prefer B.

enkrates commented Jan 30, 2014

I prefer B.

@hliyan

This comment has been minimized.

Show comment
Hide comment
@hliyan

hliyan commented Jan 31, 2014

B

@alfg

This comment has been minimized.

Show comment
Hide comment
@alfg

alfg Jan 31, 2014

C. Seems the most minimal.

alfg commented Jan 31, 2014

C. Seems the most minimal.

@mcallan83

This comment has been minimized.

Show comment
Hide comment
@mcallan83

mcallan83 commented Feb 1, 2014

B

@jboverfelt

This comment has been minimized.

Show comment
Hide comment
@jboverfelt

jboverfelt commented Feb 1, 2014

B

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost commented Feb 3, 2014

C

@ramirovjr

This comment has been minimized.

Show comment
Hide comment
@ramirovjr

ramirovjr commented Feb 5, 2014

C

@owenversteeg

This comment has been minimized.

Show comment
Hide comment
@owenversteeg

owenversteeg Feb 9, 2014

Owner

Thanks everyone for voting! In preparation for the launch of Min 2, I'm closing this issue. Here's the vote breakdown:

Votes

A (1)

B (6)

C (5)

If you prefer another button choice, feel free to say so. Remember, there's always another opportunity to change the button look & feel for Min 3! If you have another button style idea, just comment with an image.

Owner

owenversteeg commented Feb 9, 2014

Thanks everyone for voting! In preparation for the launch of Min 2, I'm closing this issue. Here's the vote breakdown:

Votes

A (1)

B (6)

C (5)

If you prefer another button choice, feel free to say so. Remember, there's always another opportunity to change the button look & feel for Min 3! If you have another button style idea, just comment with an image.

@owenversteeg

This comment has been minimized.

Show comment
Hide comment
@owenversteeg

owenversteeg Feb 9, 2014

Owner

hey @scottaohara could you check over my edit to /sass/_button.scss? I think my LESS background shows when I push to the sass folder. (also, I killed some whitespace - not sure if you had that there for a reason.)

Owner

owenversteeg commented Feb 9, 2014

hey @scottaohara could you check over my edit to /sass/_button.scss? I think my LESS background shows when I push to the sass folder. (also, I killed some whitespace - not sure if you had that there for a reason.)

@scottaohara

This comment has been minimized.

Show comment
Hide comment
@scottaohara

scottaohara Feb 9, 2014

Collaborator

I just submitted a pull request for some updates to the sass branch.

There are a few things I've done in the button.scss that you should probably look at and see if you agree (mostly adding a display: inline-block; to the button, .btn selector).

I'm guessing the white space you were referring to was the extra line breaks between rulesets? I do that just to make things a bit easier to read. Since extra spacing gets removed when compiling to the css / minified css, I'd like to keep it in there.

Also take note of the example I posted as http://scottohara.me/min, as it is directly related to why an inline-block should be applied to the button, .btn selector.

thanks

Collaborator

scottaohara commented Feb 9, 2014

I just submitted a pull request for some updates to the sass branch.

There are a few things I've done in the button.scss that you should probably look at and see if you agree (mostly adding a display: inline-block; to the button, .btn selector).

I'm guessing the white space you were referring to was the extra line breaks between rulesets? I do that just to make things a bit easier to read. Since extra spacing gets removed when compiling to the css / minified css, I'd like to keep it in there.

Also take note of the example I posted as http://scottohara.me/min, as it is directly related to why an inline-block should be applied to the button, .btn selector.

thanks

@enkrates

This comment has been minimized.

Show comment
Hide comment
@enkrates

enkrates Feb 9, 2014

Please correct me if I'm wrong, but it seems to me that option (B) won the vote. In any case, thanks for all the work on this great little CSS framework!

enkrates commented Feb 9, 2014

Please correct me if I'm wrong, but it seems to me that option (B) won the vote. In any case, thanks for all the work on this great little CSS framework!

@owenversteeg

This comment has been minimized.

Show comment
Hide comment
@owenversteeg

owenversteeg Feb 9, 2014

Owner

@enkrates You're correct, thanks! I've updated the CSS and the chart accordingly.

Owner

owenversteeg commented Feb 9, 2014

@enkrates You're correct, thanks! I've updated the CSS and the chart accordingly.

@owenversteeg owenversteeg added the poll label Aug 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment