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
** Fixes #2907 @incude in sass files #2908
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,16 @@ | ||
SCSS_FILES=$(wildcard sass/*.scss) | ||
SOURCES=$(filter-out sass/_%,$(SCSS_FILES)) | ||
TARGETS=$(SOURCES:sass/%.scss=css/%.css) | ||
MAIN_SCSS_FILES=$(wildcard sass/*.scss) | ||
MAIN_SOURCES=$(filter-out sass/_%,$(MAIN_SCSS_FILES)) | ||
MAIN_TARGETS=$(MAIN_SOURCES:sass/%.scss=css/%.css) | ||
|
||
FLEX_SCSS_FILES=$(wildcard sass/flex/*.scss) | ||
FLEX_SOURCES=$(filter-out sass/flex/_%,$(FLEX_SCSS_FILES)) | ||
FLEX_TARGETS=$(FLEX_SOURCES:sass/flex/%.scss=css/flex/%.css) | ||
|
||
.PHONY: all | ||
all: $(TARGETS) | ||
all: $(MAIN_TARGETS) $(FLEX_TARGETS) | ||
|
||
css/%.css: sass/%.scss | ||
sassc $< $@ | ||
|
||
css/flex/%.css: sass/flex/%.scss | ||
sassc $< $@ |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -559,7 +559,7 @@ div[class*="wi-dock-static-navi"] { | |
.lu-badgeContainer { | ||
|
||
.lu-user-picture { | ||
@incude border(2px,#fff,solid); | ||
@include border(2px,#fff,solid); | ||
height:48px; | ||
width:48px; | ||
position:relative; | ||
|
@@ -581,7 +581,7 @@ div[class*="wi-dock-static-navi"] { | |
|
||
} | ||
.lu-infoContainer { | ||
@incude border(2px,#fff,solid); | ||
@include border(2px,#fff,solid); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Line should be indented 6 spaces, but was indented 7 spaces There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix pls, color literals can be ignored now |
||
background:#fff; | ||
height:48px; | ||
display:inline-block; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -73,7 +73,7 @@ body[class^='workspace'] { | |
margin-top:5px; | ||
|
||
.lu-picture-container { | ||
@incude border(2px,#fff,solid); | ||
@include border(2px,#fff,solid); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Commas in mixin arguments should be followed by one space There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix pls, color literals can be ignored now |
||
box-shadow:0 0 10px rgba(0, 0, 0, 0.1); | ||
height:48px; | ||
width:48px; | ||
|
@@ -598,4 +598,4 @@ body[class^='workspace'] { | |
} | ||
|
||
} | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commas in mixin arguments should be followed by one space
Color literals like
#fff
should only be used in variable declarations; they should be referred to via variable everywhere else.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix pls, color literals can be ignored now