scss fixes #348

Merged
merged 2 commits into from Sep 7, 2013

Projects

None yet

3 participants

@gajeev
Contributor
gajeev commented Sep 5, 2013

fixed .flash nest (using &)

@parndt
Member
parndt commented Sep 5, 2013

Per #347

@parndt parndt commented on an outdated diff Sep 5, 2013
app/assets/stylesheets/refinery/blog/frontend.css.scss
@@ -56,21 +56,21 @@
margin-bottom: 15px;
position: relative;
}
-.flash {
- .flash_notice {
- &, * {
- color: #00A017;
- }
- }
- .flash_notice, .flash_message {
- border: 1px solid #00A017;
+.flash_notice {
@parndt
parndt Sep 5, 2013 Member

this can go inside .flash, too

@parndt parndt commented on an outdated diff Sep 5, 2013
app/assets/stylesheets/refinery/blog/frontend.css.scss
}
- .flash_notice #flash_close, .flash.flash_error #flash_close {
+}
+.flash_notice, .flash_message {
@parndt
parndt Sep 5, 2013 Member

this can go inside .flash, too

@parndt parndt commented on an outdated diff Sep 5, 2013
app/assets/stylesheets/refinery/blog/frontend.css.scss
@@ -95,10 +95,10 @@
}
}
}
- .flash_message {
- &, & * {
- color: #262719;
- font-size: 14px;
- }
+}
+.flash_message {
@parndt
parndt Sep 5, 2013 Member

This can go inside .flash, too

@gajeev
Contributor
gajeev commented Sep 5, 2013

ok I was going by the original css file, i'll add those as well

@gajeev
Contributor
gajeev commented Sep 5, 2013

done

@ugisozols ugisozols merged commit 5ae082d into refinery:master Sep 7, 2013

1 check failed

default The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment