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

added stinson filter #251

Merged
merged 3 commits into from
Nov 11, 2016
Merged

added stinson filter #251

merged 3 commits into from
Nov 11, 2016

Conversation

LadyCarni
Copy link
Contributor

@LadyCarni LadyCarni commented Oct 22, 2016

re: #21

Animated GIFs showing the on/off of the filter:

working-filters-2
working-filters

// }
@mixin stinson($filters...) {
@include filter-base;
filter: contrast(0.75) saturate(0.85) $filters;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would add a bit of brightness too, as its looking a bit darker than the filer, maybe brightness(1.15)

@include filter-base;
filter: contrast(0.75) saturate(0.85) $filters;

@content;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would also add a pseudo element with a light orange light to bring up the warmth in the shadows. This is what I tried and it looked pretty close:

&::before {
  background: rgba(240, 149, 128, .2);
  mix-blend-mode: soft-light;
}

Before changes:

screen shot 2016-11-02 at 10 13 59 am

After:

screen shot 2016-11-02 at 10 13 49 am

This was referenced Nov 3, 2016
@LadyCarni LadyCarni force-pushed the added-stinson-filter branch from dea7458 to 75bdff5 Compare November 4, 2016 02:18
@LadyCarni
Copy link
Contributor Author

I have made the requested changes and here's the screenshot:

screen shot 2016-11-03 at 10 27 40 pm

@una una merged commit 50e6e13 into una:master Nov 11, 2016
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