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.

Sign up for GitHub

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

Jump to bottom

Update forms.less #13592

Closed
wants to merge 1 commit into from
Closed

Update forms.less #13592

wants to merge 1 commit into from

Conversation

JelmerD
Copy link

@JelmerD JelmerD commented May 13, 2014

Fix #13591
At least, I guess. I just did this quick PR so that you don't have to do it yourself.

Disclaimer

I did not have the time to test it! Please do before releasing.

@cvrebert cvrebert added this to the v3.2.1 milestone May 13, 2014
@cvrebert cvrebert added the css label May 13, 2014
@hnrch02
Copy link
Collaborator

hnrch02 commented May 13, 2014

The way you used the & operator is not going to change the output of the compiler.

@juthilo
Copy link
Collaborator

juthilo commented May 13, 2014

Like @hnrch02 said, this doesn't change anything. The & operator is more useful for stuff like :hover where you need the nested selector to sit directly next to the parent selector.

@juthilo juthilo closed this May 13, 2014
@JelmerD
Copy link
Author

JelmerD commented May 13, 2014

Ok, I thought it was necessary :) Thanks for learning me that, saves me a character in my LESS files 😄

@XhmikosR XhmikosR modified the milestones: v3.2.0, v3.2.1 Jun 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
css
Projects
None yet
Milestone
v3.2.0
Development

Successfully merging this pull request may close these issues.

Padding left 20px on .checkbox is gone?
5 participants
@JelmerD @hnrch02 @juthilo @XhmikosR @cvrebert