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

Inline sul_styles #2283

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Inline sul_styles #2283

wants to merge 1 commit into from

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Jan 24, 2023

This helps us so that we can migrate from libsass to dart sass

@import 'sul-styles/su_web_colors';
@import 'sul-styles/su_primary_colors';
@import 'sul-styles/variables';
@import 'sul-styles/sul_footer';

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
Prefer double-quoted strings

@@ -0,0 +1,4 @@
@import 'sul-styles/su_web_colors';
@import 'sul-styles/su_primary_colors';
@import 'sul-styles/variables';

Choose a reason for hiding this comment

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

Prefer double-quoted strings

@@ -0,0 +1,4 @@
@import 'sul-styles/su_web_colors';
@import 'sul-styles/su_primary_colors';

Choose a reason for hiding this comment

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

Prefer double-quoted strings

@@ -0,0 +1,4 @@
@import 'sul-styles/su_web_colors';

Choose a reason for hiding this comment

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

Prefer double-quoted strings

@@ -0,0 +1,34 @@
/*

Choose a reason for hiding this comment

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

Use // comments everywhere

@@ -0,0 +1,34 @@
/*

Choose a reason for hiding this comment

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

Use // comments everywhere


li {
display: inline;
margin: 0 13px 3px 0;

Choose a reason for hiding this comment

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

Shorthands of length 4 are not allowed. Value was 0 13px 3px 0

clear: both;
margin-top: -1 * $footer-height;
min-height: $footer-height;
padding: 0 !important;

Choose a reason for hiding this comment

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

!important should not be used

@@ -0,0 +1,77 @@
#su-content {
padding-bottom: $footer-height !important;

Choose a reason for hiding this comment

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

!important should not be used

$sul-footer-border-top-color: $color-beige-20;
$footer-height: 200px;
$sul-footer-shadow-color: rgba(0, 0, 0, 0.2);
$sul-footer-shadow: 0 4px 8px -8px $sul-footer-shadow-color inset;

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

This helps us so that we can migrate from libsass to dart sass
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Maintenance or dev-specific - will need dev help to prioritize
Development

Successfully merging this pull request may close these issues.

2 participants