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

Different height of buttons #44

Closed
thesealion opened this issue Oct 1, 2011 · 10 comments
Closed

Different height of buttons #44

thesealion opened this issue Oct 1, 2011 · 10 comments

Comments

@thesealion
Copy link

If you place together a normal button and an anchor with class="button", you'll notice that they have different height. And it looks really ugly.

@dhg
Copy link
Owner

dhg commented Oct 3, 2011

You just need to add a class .button to the tag and it will look exactly the same. This was intentional so that the style was tied to that class and not to the tag.

@dhg dhg closed this as completed Oct 3, 2011
@thesealion
Copy link
Author

No, this doesn't seem to work. I looked at the css and there are no rules for button.button or just .button, only for a.button.

@dhg
Copy link
Owner

dhg commented Oct 3, 2011

Right. Unless I'm mistaken (and I did a quick Firefox/Chrome check) adding the class button to the button element styles it properly. (e.g. <button class="button">Text</button>). Let me know if this is more specifically an IE issue or if there is something I am missing!

@thesealion
Copy link
Author

Here's what it looks like in Firefox 7.0.1: http://charadeyouare.org/buttons.png
You can clearly see the difference. Adding the class has no effect.

@thesealion
Copy link
Author

Any comment on that?

@dhg dhg reopened this Oct 12, 2011
@dhg
Copy link
Owner

dhg commented Oct 12, 2011

Sorry for the belated response - yes those are definitely different! I will investigate and try to commit some changes soon. Thanks for pushing on the issue (been busy and tough to keep up). Thanks so much!

@necolas
Copy link

necolas commented Oct 20, 2011

This is because you've set a line-height value other than normal on L158

Taken from the normalize.css wiki:

button, input {}
line-height: normal : FF3/4 sets line-height using !important in the UA stylesheet. This declaration is to set all other browsers to match the FF3/4 style.

@dhg
Copy link
Owner

dhg commented Jan 26, 2012

Added line-height: normal, but still have the 1px difference in Firefox for button. Will have to investigate further.

@necolas
Copy link

necolas commented Jan 26, 2012

@dhg
Copy link
Owner

dhg commented Jan 27, 2012

@necolas You're the man. Fixed it. Glorious.

@dhg dhg closed this as completed Jan 27, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants