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

0.5.6 #534

Merged
merged 15 commits into from
Nov 20, 2018
Merged

0.5.6 #534

merged 15 commits into from
Nov 20, 2018

Conversation

picturepan2
Copy link
Owner

@picturepan2 picturepan2 merged commit 81ee834 into master Nov 20, 2018
top: 60%;
width: .8em;
}
}

Choose a reason for hiding this comment

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

Files should end with a trailing newline

left: 40%;
top: 35%;
width: .8em;
}

Choose a reason for hiding this comment

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

Rule declaration should be followed by an empty line

.icon-copy {
&::before {
border: $icon-border-width solid currentColor;
border-radius: $border-radius;

Choose a reason for hiding this comment

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

Properties should be ordered border, border-bottom, border-radius, border-right, height, left, top, width

@@ -7,5 +7,4 @@
@import "mixins/position";
@import "mixins/shadow";
@import "mixins/text";
@import "mixins/toast";
@import "mixins/transition";
@import "mixins/toast";

Choose a reason for hiding this comment

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

Files should end with a trailing newline

.hero-body {
padding: $layout-spacing;
}
}

Choose a reason for hiding this comment

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

Files should end with a trailing newline

font-weight: 700;
text-align: left;
opacity: 0.5;
&:before {

Choose a reason for hiding this comment

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

Begin pseudo elements with double colons: ::
Rule declaration should be preceded by an empty line
Selector should have depth of applicability no greater than 2, but was 3

font-size: .9em;
font-weight: 700;
text-align: left;
opacity: 0.5;

Choose a reason for hiding this comment

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

0.5 should be written without a leading zero as .5

width: auto;
float: left;
padding: 0;
color: #02060c;

Choose a reason for hiding this comment

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

Color literals like #02060c should only be used in variable declarations; they should be referred to via variable everywhere else.

@media (max-width: 768px) {
.algolia-autocomplete .algolia-docsearch-suggestion {
.algolia-docsearch-suggestion--subcategory-column {
display: inline-block;

Choose a reason for hiding this comment

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

Properties should be ordered color, display, float, font-size, font-weight, opacity, padding, text-align, width


@media (max-width: 768px) {
.algolia-autocomplete .algolia-docsearch-suggestion {
.algolia-docsearch-suggestion--subcategory-column {

Choose a reason for hiding this comment

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

Selector should have depth of applicability no greater than 2, but was 3

@picturepan2 picturepan2 self-assigned this Nov 20, 2018
@picturepan2 picturepan2 added this to the v0.5.6 milestone Nov 20, 2018
@picturepan2 picturepan2 deleted the 0.5.6 branch November 24, 2018 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants