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

Select box (maybe not only it) margin bug #3684

Closed
kamikaze opened this Issue Feb 29, 2012 · 6 comments

Comments

Projects
None yet
2 participants
@kamikaze

kamikaze commented Feb 29, 2012

Correct margin in 1.0.1: http://jsfiddle.net/ha8CC/
Incorrect margin in master/1.1rc1: http://jsfiddle.net/ha8CC/1/

As I can see the problem is that ".ui-select .ui-btn" margin 0; overrides correct margin: 0.5em 5px; from simple ".ui-btn".
So when disabling magin 0 in firebug - everything looks fine.

I have this problem in Android 2.3.5 browser, Firefox 10.0.2 and 11beta. In IE8 it is a little bit better.

@kamikaze

This comment has been minimized.

Show comment
Hide comment
@kamikaze

kamikaze Mar 3, 2012

Looks like it's fixed in master. Closing

kamikaze commented Mar 3, 2012

Looks like it's fixed in master. Closing

@kamikaze kamikaze closed this Mar 3, 2012

@kamikaze kamikaze reopened this Mar 3, 2012

@kamikaze

This comment has been minimized.

Show comment
Hide comment
@kamikaze

kamikaze Mar 3, 2012

oh no, I'm very sorry... it's not

kamikaze commented Mar 3, 2012

oh no, I'm very sorry... it's not

@jaspermdegroot

This comment has been minimized.

Show comment
Hide comment
@jaspermdegroot

jaspermdegroot May 1, 2012

Member

Just reference #4098 because I will look into consistency of buttons/form elements in general while working on a solution for that issue.

Member

jaspermdegroot commented May 1, 2012

Just reference #4098 because I will look into consistency of buttons/form elements in general while working on a solution for that issue.

@kamikaze

This comment has been minimized.

Show comment
Hide comment
@kamikaze

kamikaze May 2, 2012

ok. should I close this issue?

kamikaze commented May 2, 2012

ok. should I close this issue?

@jaspermdegroot

This comment has been minimized.

Show comment
Hide comment
@jaspermdegroot

jaspermdegroot May 2, 2012

Member

No, don't close until it is actually fixed.

Member

jaspermdegroot commented May 2, 2012

No, don't close until it is actually fixed.

@kamikaze

This comment has been minimized.

Show comment
Hide comment
@kamikaze

kamikaze May 14, 2012

Just to be up to date: same problem with 1.1.0
http://jsfiddle.net/ha8CC/21/

kamikaze commented May 14, 2012

Just to be up to date: same problem with 1.1.0
http://jsfiddle.net/ha8CC/21/

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