MidnightRiders/MemberPortal

View on GitHub
app/assets/stylesheets/vendor/foundation/components/_top-bar.scss

Summary

Maintainability
Test Coverage

Expected item on line 363 to appear before line 360. Rule sets should be ordered as follows: @extends, @includes without @content, properties, @includes with @content, nested rule sets
Open

          @include button-style($bg:$primary-color);

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

            &.parent-link {

Each selector in a comma sequence should be on its own single line
Open

        h1, h2, h3, h4, p, span {

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

            .has-dropdown.moved > .dropdown,

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

        > a {

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

          &.button.success { @include button-style($bg:$success-color); }

Rule declaration should be followed by an empty line
Open

          &.warning { @include button-style($bg:$warning-color); }

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

          &.info { @include button-style($bg:$info-color); }

Merge rule &.button.success with rule on line 345
Open

          &.button.success { @include button-style($bg:$success-color); }

Rule declaration should be preceded by an empty line
Open

            &.parent-link {

Prefer single quoted strings
Open

$topbar-sticky-class: ".sticky" !default;

Line contains trailing whitespace
Open

      input, 

Line contains trailing whitespace
Open

      input, 

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

            span::after {

Expected item on line 349 to appear before line 346. Rule sets should be ordered as follows: @extends, @includes without @content, properties, @includes with @content, nested rule sets
Open

            @include button-style($bg:$primary-color);

Merge rule &.button.secondary with rule on line 345
Open

          &.button.secondary { @include button-style($bg:$secondary-color); }

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

        &:hover:not(.has-form) > a {

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

        &.active > a {

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

          a {

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

          &.title h5,

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

          .button,

Property '#<:script::tree::interpolation:0x000055d4f60f9878>' should be placed on separate line from selector</:script::tree::interpolation:0x000055d4f60f9878>
Open

            li .dropdown { #{$default-float}: 100%; }

Each selector in a comma sequence should be on its own single line
Open

        h1, h2, h3, h4, p, span {

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

          &.button {

Rule declaration should be followed by an empty line
Open

          &.secondary { @include button-style($bg:$secondary-color); }

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

          &:hover {

Begin pseudo elements with double colons: ::
Open

          &:after {

Colon after property should be followed by one space
Open

              &:hover { background:none; }

Rule declaration should be followed by an empty line
Open

        .title-area { float: $default-float; }

Commas in mixin arguments should be followed by one space
Open

        @include single-transition(none,0,0);

Rule declaration should be preceded by an empty line
Open

            .js-generated { display: none; }

Property 'display' should be placed on separate line from selector
Open

            .js-generated { display: none; }

Rule declaration should be preceded by an empty line
Open

              &:hover {

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

        ul li {

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

          &:active > a {

Expected item on line 312 to appear before line 309. Rule sets should be ordered as follows: @extends, @includes without @content, properties, @includes with @content, nested rule sets
Open

      @include single-transition($default-float, $topbar-transition-speed);

Rule declaration should be followed by an empty line
Open

          &.button.success { @include button-style($bg:$success-color); }

Rule declaration should be followed by an empty line
Open

          &.button.alert { @include button-style($bg:$alert-color); }

Merge rule &.button.info with rule on line 345
Open

          &.button.info { @include button-style($bg:$info-color); }

Property 'display' should be placed on separate line from selector
Open

    .js-generated { display: block; }

Begin pseudo elements with double colons: ::
Open

                &:after {

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

            .dropdown {

$topbar-link-padding+rem-calc(25) should be written with a single space on each side of the operator: $topbar-link-padding + rem-calc(25)
Open

            padding: 0 $topbar-link-padding+rem-calc(25) 0 $topbar-link-padding;

Merge rule &.button.alert with rule on line 345
Open

          &.button.alert { @include button-style($bg:$alert-color); }

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

          &.secondary { @include button-style($bg:$secondary-color); }

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

          &.warning { @include button-style($bg:$warning-color); }

Rule declaration should be preceded by an empty line
Open

              &:hover { background:none; }

Commas in mixin arguments should be followed by one space
Open

        @include single-transition(none,0,0);

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

            .js-generated { display: none; }

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

            > a {

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

            li .dropdown { #{$default-float}: 100%; }

Prefer single quoted strings
Open

      font-family: "/" + unquote($topbar-media-query) + "/";

Each selector in a comma sequence should be on its own single line
Open

        h1, h2, h3, h4, p, span {

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

        > button {

Begin pseudo elements with double colons: ::
Open

              &:after {

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

            li .dropdown { #{$opposite-direction}: 100%; }

Use // comments everywhere
Open

    /* Wrapped around .top-bar to contain to grid width */

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

          &.button.alert { @include button-style($bg:$alert-color); }

Rule declaration should be followed by an empty line
Open

          &.success { @include button-style($bg:$success-color); }

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

          &.alert { @include button-style($bg:$alert-color); }

Property 'float' should be placed on separate line from selector
Open

        .title-area { float: $default-float; }

Rule declaration should be followed by an empty line
Open

        }

Properties should be ordered background, width
Open

      width: 100%;

Merge rule ul li with rule on line 314
Open

      ul li {

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

          &.button.warning { @include button-style($bg:$warning-color); }

Rule declaration should be followed by an empty line
Open

          &.button.warning { @include button-style($bg:$warning-color); }

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

          &.success { @include button-style($bg:$success-color); }

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

              &:hover > a:not(.button) {

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

            > .dropdown {

Prefer single quoted strings
Open

@include exports("top-bar") {

Each selector in a comma sequence should be on its own single line
Open

        h1, h2, h3, h4, p, span {

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

          a {

Rule declaration should be followed by an empty line
Open

          &.button.secondary { @include button-style($bg:$secondary-color); }

Nesting should be no greater than 5, but was 6
Open

              &:hover { background:none; }

Property 'background' should be placed on separate line from selector
Open

              &:hover { background:none; }

border-top: 0 is preferred over border-top: none
Open

          border-top: none;

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

          li .dropdown {

Each selector in a comma sequence should be on its own single line
Open

      .button, button {

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

            .has-dropdown:not(.moved) > ul {

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

        .name h1 a,

Property 'width' should be placed on separate line from selector
Open

        .name h6 a { width: auto; }

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

          .dropdown li.has-dropdown {

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

            label {

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

          li .dropdown {

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

          &.button.info { @include button-style($bg:$info-color); }

Rule declaration should be preceded by an empty line
Open

            a {

Rule declaration should be preceded by an empty line
Open

            > .dropdown {

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

              > a:not(.button) {

Prefer single quoted strings
Open

      font-family: "/" + unquote($topbar-media-query) + "/";

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

          &.button.secondary { @include button-style($bg:$secondary-color); }

Rule declaration should be followed by an empty line
Open

          &.alert { @include button-style($bg:$alert-color); }

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

            a {

Each selector in a comma sequence should be on its own single line
Open

        h1, h2, h3, h4, p, span {

Merge rule &.button.warning with rule on line 345
Open

          &.button.warning { @include button-style($bg:$warning-color); }

Begin pseudo elements with double colons: ::
Open

          > a:after {

Properties should be ordered font-size, margin-bottom, margin-top
Open

            margin-bottom: 0;

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

              &:hover { background:none; }

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

                &:after {

Prefer single quoted strings
Open

                  content: "\00bb";

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

            a {

Property 'display' should be placed on separate line from selector
Open

        .toggle-topbar { display: none; }

Rule declaration should be preceded by an empty line
Open

              &:hover {

Rule declaration should be preceded by an empty line
Open

              &:after {

Each selector in a comma sequence should be on its own single line
Open

          &.hover, &.not-click:hover {

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

          > a:focus + .dropdown {

Line contains trailing whitespace
Open

          position: relative; 

border: 0 is preferred over border: none
Open

                  border: none;

border-bottom: 0 is preferred over border-bottom: none
Open

          border-bottom: none;

Property '#<:script::tree::interpolation:0x000055d4f60d8588>' should be placed on separate line from selector</:script::tree::interpolation:0x000055d4f60d8588>
Open

            li .dropdown { #{$opposite-direction}: 100%; }

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

          &:hover > a {

There are no issues that match your filters.

Category
Status