diff options
author | Takuya Noguchi <takninnovationresearch@gmail.com> | 2017-10-30 00:24:11 +0900 |
---|---|---|
committer | Takuya Noguchi <takninnovationresearch@gmail.com> | 2017-10-30 15:39:05 +0900 |
commit | 1cd0fbd96b29c0713a926fe9602eae703f77525d (patch) | |
tree | 2171e94d42d3443ee055f0bb6c9a34f91a6b9e75 | |
parent | 6d7384f59525e2d54c30011237499c27bde145ef (diff) | |
download | gitlab-ce-1cd0fbd96b29c0713a926fe9602eae703f77525d.tar.gz |
Enable NestingDepth (level 6) on scss-lint
-rw-r--r-- | .scss-lint.yml | 3 | ||||
-rw-r--r-- | app/assets/stylesheets/framework/gitlab-theme.scss | 7 | ||||
-rw-r--r-- | app/assets/stylesheets/framework/header.scss | 6 | ||||
-rw-r--r-- | app/assets/stylesheets/pages/notes.scss | 12 | ||||
-rw-r--r-- | changelogs/unreleased/39582-nestingdepth-6.yml | 5 |
5 files changed, 12 insertions, 21 deletions
diff --git a/.scss-lint.yml b/.scss-lint.yml index 73f8d27f78c..d2c972fa9c4 100644 --- a/.scss-lint.yml +++ b/.scss-lint.yml @@ -121,7 +121,8 @@ linters: # Avoid nesting selectors too deeply. NestingDepth: - enabled: false + enabled: true + max_depth: 6 # Always use placeholder selectors in @extend. PlaceholderInExtend: diff --git a/app/assets/stylesheets/framework/gitlab-theme.scss b/app/assets/stylesheets/framework/gitlab-theme.scss index 52b87de7a3d..dc591c06c88 100644 --- a/app/assets/stylesheets/framework/gitlab-theme.scss +++ b/app/assets/stylesheets/framework/gitlab-theme.scss @@ -216,12 +216,9 @@ body { color: $theme-gray-900; } - &.active > a { + &.active > a, + &.active > a:hover { color: $white-light; - - &:hover { - color: $white-light; - } } } } diff --git a/app/assets/stylesheets/framework/header.scss b/app/assets/stylesheets/framework/header.scss index d79444fad79..62ba74ff582 100644 --- a/app/assets/stylesheets/framework/header.scss +++ b/app/assets/stylesheets/framework/header.scss @@ -239,10 +239,8 @@ fill: currentColor; } - &.header-user-dropdown-toggle { - .header-user-avatar { - border-color: $white-light; - } + &.header-user-dropdown-toggle .header-user-avatar { + border-color: $white-light; } } } diff --git a/app/assets/stylesheets/pages/notes.scss b/app/assets/stylesheets/pages/notes.scss index 3bd0e3ad535..312917bd13a 100644 --- a/app/assets/stylesheets/pages/notes.scss +++ b/app/assets/stylesheets/pages/notes.scss @@ -269,7 +269,7 @@ ul.notes { display: none; } - &.system-note-commit-list { + &.system-note-commit-list:not(.hide-shade) { max-height: 70px; overflow: hidden; display: block; @@ -291,16 +291,6 @@ ul.notes { bottom: 0; background: linear-gradient(rgba($white-light, 0.1) -100px, $white-light 100%); } - - &.hide-shade { - max-height: 100%; - overflow: auto; - - &::after { - display: none; - background: transparent; - } - } } } } diff --git a/changelogs/unreleased/39582-nestingdepth-6.yml b/changelogs/unreleased/39582-nestingdepth-6.yml new file mode 100644 index 00000000000..efe15f0a5f3 --- /dev/null +++ b/changelogs/unreleased/39582-nestingdepth-6.yml @@ -0,0 +1,5 @@ +--- +title: Enable NestingDepth (level 6) on scss-lint +merge_request: 15073 +author: Takuya Noguchi +type: other |