Skip to content
This repository has been archived by the owner on Jan 14, 2018. It is now read-only.

border-radius > rounded #57

Open
deadlyhifi opened this issue Aug 2, 2012 · 4 comments · May be fixed by #58
Open

border-radius > rounded #57

deadlyhifi opened this issue Aug 2, 2012 · 4 comments · May be fixed by #58

Comments

@deadlyhifi
Copy link
Contributor

Again, I can't do a pull request because of my edits but in the SASS and SCSS and possibly some other flavours border-radius has been used where it should be rounded.

Namely any use of border-radius that has a single value in the brackets.

@jina
Copy link
Contributor

jina commented Aug 2, 2012

I believe you are looking in the Compass versions, in which +rounded is not necessary because +border-radius already exists in Compass. :)

@deadlyhifi
Copy link
Contributor Author

I mean within the actual use of the mixins.

e.g. https://github.com/malarkey/320andup/blob/master/scss/_buttons.scss#L33 creates a top right 4px rounded corner where it should be on all corners.

This occurs on a few buttons, and also on the wells within _texture.scss.

@jina
Copy link
Contributor

jina commented Aug 2, 2012

Hrm, okay I will look into that. When I noticed Andy was calling out a particular corner rather than all corners, I opted for Compass'es mixins instead of Andy's so it'd be less repetitive code. Perhaps I accidentally ported some of that over to the non-Compass files. Will fix once I get a chance to look at it. Thanks!

@deadlyhifi
Copy link
Contributor Author

Thats alright, thanks for doing all the porting in the first place.

@kevindavies kevindavies linked a pull request Aug 7, 2012 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants