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

Add note on SCSS' overloaded rgba() function #29

Merged
merged 1 commit into from
Apr 15, 2015

Conversation

bbbrrriiiaaannn
Copy link
Contributor

Sass + SCSS overrides and overloads the rgba() function so it can also take parameters in the format rgba($color, $alpha), e.g. rgba(#000, .5).

It'd be pragmatic to leverage that overload and use hex colors that much more consistently in SCSS. If GitHub folks aren't writing much raw CSS this line can be trimmed further, * Use hex color codes #000(SCSS'rgba()function is overloaded to accept hex colors as a param, e.g.,rgba(#000, .5)). (since Sass' overriding of rgba() isn't obvious perhaps good to spell it out explicitly here).

Sass + SCSS overrides and overloads the [rgba() function](http://sass-lang.com/documentation/Sass/Script/Functions.html#rgba-instance_method) so it can also take parameters in the format `rgba($color, $alpha)`, e.g. `rgba(#000, .5)`.

It'd be pragmatic to leverage that overload and use hex colors that much more consistently in SCSS. If GitHub folks aren't writing much raw CSS this line can be trimmed further, ```* Use hex color codes `#000` (SCSS' `rgba()` function is overloaded to accept hex colors as a param, e.g., `rgba(#000, .5)`).``` (since Sass' overriding of `rgba()` isn't obvious perhaps good to spell it out explicitly here).
@mdo mdo added the docs label Mar 24, 2015
@mdo
Copy link
Contributor

mdo commented Mar 27, 2015

Howdy—thanks for the pull request, @briancs!

Before we can merge your changes, we need you to review our Contributor License Agreement (CLA). We think of CLAs as an open source software license between you and the project, letting us and other users know that we have permission to use the code you contribute.

At the bottom of the pull request, you'll see the note about a failed test. Just click the Details link, review the CLA, and sign off at the bottom.

Questions? Just holler—I'm here to help :).

<3

@mdo mdo added the css label Mar 30, 2015
@mdo mdo modified the milestone: v2.0.4 Mar 30, 2015
mdo added a commit that referenced this pull request Apr 15, 2015
Add note on SCSS' overloaded rgba() function
@mdo mdo merged commit 2d70ae5 into primer:master Apr 15, 2015
@mdo mdo mentioned this pull request Apr 15, 2015
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

Successfully merging this pull request may close these issues.

2 participants