kotovalexarian-likes-gitlab/gitlab-org--gitlab-foss
1
0
Fork 0
Code Releases Activity

Revert "Merge branch '61145-fix-button-dimensions' into 'master'"

Browse source 
This reverts merge request !28545
This commit is contained in:
Annabel Dunstone Gray 2019-07-16 16:03:29 +00:00
parent cca71da16a
commit 41b77136ee
15 changed files with 91 additions and 100 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/assets/stylesheets/framework/animations.scss
View file

@ -104,7 +104,7 @@
}
.btn {
@include transition(border-color);
@include transition(background-color, border-color, color, box-shadow);
}
.dropdown-menu-toggle,

132
app/assets/stylesheets/framework/buttons.scss
View file

@ -24,11 +24,12 @@
border-radius: $border-radius-default;
font-size: $gl-font-size;
font-weight: $gl-font-weight-normal;
padding: $gl-bordered-btn-vert-padding $gl-bordered-btn-horz-padding;
padding: $gl-vert-padding $gl-btn-padding;
&:focus,
&:active {
background-color: $btn-active-gray;
box-shadow: $gl-btn-active-background;
}
}
@ -49,89 +50,77 @@
color: $text;
}
&:not(:disabled):not(.disabled) {
&:hover {
box-shadow: inset 0 0 0 1px $hover-border, 0 2px 2px 0 $gl-btn-hover-shadow-light;
}
&:hover,
&:focus {
background-color: $hover-background;
border-color: $hover-border;
color: $hover-text;
&:focus {
box-shadow: inset 0 0 0 1px $hover-border, 0 0 4px 1px $blue-300;
}
&:hover,
&:focus {
background-color: $hover-background;
border-color: $hover-border;
> .icon {
color: $hover-text;
}
}
> .icon {
color: $hover-text;
}
&:focus {
box-shadow: 0 0 4px 1px $blue-300;
}
&:active {
background-color: $active-background;
border-color: $active-border;
box-shadow: inset 0 2px 4px 0 rgba($black, 0.2);
color: $active-text;
> .icon {
color: $active-text;
}
&:active,
&:active:focus {
background-color: $active-background;
border-color: $active-border;
box-shadow: inset 0 0 0 1px $hover-border, inset 0 2px 4px 0 rgba($black, 0.2);
color: $active-text;
> .icon {
color: $active-text;
}
&:focus {
box-shadow: inset 0 2px 4px 0 rgba($black, 0.2);
}
}
}
@mixin btn-color($light, $border-light, $normal, $border-normal, $dark, $border-dark, $color, $hover-shadow-color: $gl-btn-hover-shadow-dark) {
@mixin btn-color($light, $border-light, $normal, $border-normal, $dark, $border-dark, $color) {
background-color: $light;
border-color: $border-light;
color: $color;
&:not(:disabled):not(.disabled) {
&:hover {
box-shadow: inset 0 0 0 1px $border-normal, 0 2px 2px 0 $hover-shadow-color;
}
&:hover,
&:focus {
background-color: $normal;
border-color: $border-normal;
color: $color;
}
&:focus {
box-shadow: inset 0 0 0 1px $border-normal, 0 0 4px 1px $blue-300;
}
&:active,
&.active {
box-shadow: $gl-btn-active-background;
&:hover,
&:focus {
background-color: $normal;
border-color: $border-normal;
color: $color;
}
&:active,
&.active {
box-shadow: inset 0 2px 4px 0 $gl-btn-hover-shadow-dark;
background-color: $dark;
border-color: $border-dark;
color: $color;
}
background-color: $dark;
border-color: $border-dark;
color: $color;
}
}
@mixin btn-green {
@include btn-color($green-500, $green-600, $green-500, $green-700, $green-600, $green-800, $white-light);
@include btn-color($green-500, $green-600, $green-600, $green-700, $green-700, $green-800, $white-light);
}
@mixin btn-blue {
@include btn-color($blue-500, $blue-600, $blue-500, $blue-700, $blue-600, $blue-800, $white-light);
@include btn-color($blue-500, $blue-600, $blue-600, $blue-700, $blue-700, $blue-800, $white-light);
}
@mixin btn-orange {
@include btn-color($orange-500, $orange-600, $orange-500, $orange-700, $orange-600, $orange-800, $white-light);
@include btn-color($orange-500, $orange-600, $orange-600, $orange-700, $orange-700, $orange-800, $white-light);
}
@mixin btn-red {
@include btn-color($red-500, $red-600, $red-500, $red-700, $red-600, $red-800, $white-light);
@include btn-color($red-500, $red-600, $red-600, $red-700, $red-700, $red-800, $white-light);
}
@mixin btn-white {
@include btn-color($white-light, $gray-400, $gray-200, $gray-400, $gl-gray-200, $gray-500, $gl-text-color, $gl-btn-hover-shadow-light);
@include btn-color($white-light, $border-color, $white-normal, $border-white-normal, $white-dark, $border-gray-dark, $gl-text-color);
}
@mixin btn-with-margin {
@ -160,20 +149,21 @@
color: $gl-text-color;
white-space: nowrap;
line-height: $gl-btn-line-height;
&:focus:active {
outline: 0;
}
&.btn-xs {
font-size: $gl-btn-xs-font-size;
line-height: $gl-btn-xs-line-height;
&.btn-sm {
padding: 4px 10px;
font-size: $gl-btn-small-font-size;
line-height: $gl-btn-small-line-height;
}
&.btn-sm,
&.btn-xs {
padding: 3px $gl-bordered-btn-vert-padding;
padding: 2px $gl-btn-padding;
font-size: $gl-btn-xs-font-size;
line-height: $gl-btn-xs-line-height;
}
&.btn-success,
@ -249,7 +239,7 @@
&.dropdown-toggle {
.fa-caret-down {
margin: 0;
margin-left: 3px;
}
}
@ -282,7 +272,10 @@
}
svg {
@include btn-svg;
height: 15px;
width: 15px;
position: relative;
top: 2px;
}
svg,
@ -337,12 +330,6 @@
&.btn-grouped {
@include btn-with-margin;
}
.btn {
border-radius: $border-radius-default;
font-size: $gl-font-size;
line-height: $gl-btn-line-height;
}
}
.btn-clipboard {
@ -500,25 +487,18 @@
&:active,
&:focus {
color: $gl-text-color-secondary;
border: 1px solid $border-gray-normal-dashed;
background-color: $white-normal;
}
}
.btn-svg {
padding: $gl-bordered-btn-vert-padding;
svg {
@include btn-svg;
display: block;
}
.btn-svg svg {
@include btn-svg;
}
// All disabled buttons, regardless of color, type, etc
%disabled {
background-color: $gray-light !important;
border-color: $gray-200 !important;
box-shadow: none;
color: $gl-text-color-disabled !important;
opacity: 1 !important;
cursor: default !important;

17
app/assets/stylesheets/framework/dropdowns.scss
View file

@ -8,6 +8,12 @@
}
}
@mixin chevron-active {
.fa-chevron-down {
color: $gray-darkest;
}
}
@mixin set-visible {
transform: translateY(0);
display: block;
@ -43,6 +49,7 @@
.dropdown-toggle,
.dropdown-menu-toggle {
@include chevron-active;
border-color: $gray-darkest;
}
@ -58,12 +65,12 @@
.dropdown-toggle,
.confidential-merge-request-fork-group .dropdown-toggle {
padding: $gl-bordered-btn-vert-padding $gl-bordered-btn-horz-padding;
padding: 6px 8px 6px 10px;
background-color: $white-light;
color: $gl-text-color;
font-size: 14px;
line-height: $gl-btn-line-height;
text-align: left;
border: 1px solid $border-color;
border-radius: $border-radius-base;
white-space: nowrap;
@ -96,6 +103,10 @@
padding-right: 25px;
}
.fa {
color: $gray-darkest;
}
.fa-chevron-down {
font-size: $dropdown-chevron-size;
position: relative;
@ -104,10 +115,12 @@
}
&:hover {
@include chevron-active;
border-color: $gray-darkest;
}
&:focus:active {
@include chevron-active;
border-color: $dropdown-toggle-active-border-color;
outline: 0;
}

4
app/assets/stylesheets/framework/variables.scss
View file

@ -405,8 +405,6 @@ $tanuki-yellow: #fca326;
*/
$green-500-focus: rgba($green-500, 0.4);
$gl-btn-active-background: rgba(0, 0, 0, 0.16);
$gl-btn-hover-shadow-dark: rgba($black, 0.2);
$gl-btn-hover-shadow-light: rgba($black, 0.1);
$gl-btn-active-gradient: inset 0 2px 3px $gl-btn-active-background;
/*
@ -483,8 +481,6 @@ $gl-btn-padding: 10px;
$gl-btn-line-height: 16px;
$gl-btn-vert-padding: 8px;
$gl-btn-horz-padding: 12px;
$gl-bordered-btn-vert-padding: $gl-btn-vert-padding - 1px;
$gl-bordered-btn-horz-padding: $gl-btn-horz-padding - 1px;
$gl-btn-small-font-size: 13px;
$gl-btn-small-line-height: 18px;
$gl-btn-xs-font-size: 13px;

4
app/assets/stylesheets/pages/commits.scss
View file

@ -214,10 +214,10 @@
.label,
.btn {
padding: $gl-bordered-btn-vert-padding $gl-bordered-btn-horz-padding;
padding: $gl-vert-padding $gl-btn-padding;
border: 1px $border-color solid;
font-size: $gl-font-size;
line-height: $gl-btn-line-height;
line-height: $line-height-base;
border-radius: 0;
display: flex;
align-items: center;

4
app/assets/stylesheets/pages/issuable.scss
View file

@ -929,6 +929,10 @@
margin: 0;
}
}
.dropdown-toggle > .icon {
margin: 0 3px;
}
}
.right-sidebar-collapsed {

1
app/assets/stylesheets/pages/issues.scss
View file

@ -267,6 +267,7 @@ ul.related-merge-requests > li {
.fa-caret-down {
pointer-events: none;
color: inherit;
margin-left: 0;
}
}
}

1
app/assets/stylesheets/pages/note_form.scss
View file

@ -417,6 +417,7 @@ table {
i {
color: $white-light;
padding-right: 2px;
margin-top: 2px;
}

3
app/assets/stylesheets/pages/projects.scss
View file

@ -429,7 +429,7 @@
padding: 0;
background: transparent;
border: 0;
line-height: 2;
line-height: 34px;
margin: 0;
> li + li::before {
@ -792,6 +792,7 @@
.btn {
margin-top: $gl-padding;
padding: $gl-btn-vert-padding $gl-btn-padding;
line-height: $gl-btn-line-height;
.icon {

2
app/assets/stylesheets/pages/tree.scss
View file

@ -90,7 +90,7 @@
.add-to-tree {
vertical-align: top;
padding: $gl-bordered-btn-vert-padding;
padding: 8px;
svg {
top: 0;

2
app/helpers/dropdowns_helper.rb
View file

@ -46,7 +46,7 @@ module DropdownsHelper
def dropdown_toggle(toggle_text, data_attr, options = {})
default_label = data_attr[:default_label]
content_tag(:button, disabled: options[:disabled], class: "dropdown-menu-toggle btn #{options[:toggle_class] if options.key?(:toggle_class)}", id: (options[:id] if options.key?(:id)), type: "button", data: data_attr) do
content_tag(:button, disabled: options[:disabled], class: "dropdown-menu-toggle #{options[:toggle_class] if options.key?(:toggle_class)}", id: (options[:id] if options.key?(:id)), type: "button", data: data_attr) do
output = content_tag(:span, toggle_text, class: "dropdown-toggle-text #{'is-default' if toggle_text == default_label}")
output << icon('chevron-down')
output.html_safe

2
app/views/projects/issues/import_csv/_button.html.haml
View file

@ -1,6 +1,6 @@
- type = local_assigns.fetch(:type, :icon)
%button.csv-import-button.btn.btn-svg{ title: _('Import CSV'), class: ('has-tooltip' if type == :icon),
%button.csv-import-button.btn{ title: _('Import CSV'), class: ('has-tooltip' if type == :icon),
data: { toggle: 'modal', target: '.issues-import-modal' } }
- if type == :icon
= sprite_icon('upload')

8
app/views/shared/issuable/_feed_buttons.html.haml
View file

@ -1,4 +1,4 @@
= link_to safe_params.merge(rss_url_options), class: 'btn btn-svg has-tooltip js-rss-button', data: { container: 'body' }, title: _('Subscribe to RSS feed') do
= sprite_icon('rss')
= link_to safe_params.merge(calendar_url_options), class: 'btn btn-svg has-tooltip', data: { container: 'body' }, title: _('Subscribe to calendar') do
= sprite_icon('calendar')
= link_to safe_params.merge(rss_url_options), class: 'btn has-tooltip', data: { container: 'body' }, title: _('Subscribe to RSS feed') do
= icon('rss')
= link_to safe_params.merge(calendar_url_options), class: 'btn has-tooltip', data: { container: 'body' }, title: _('Subscribe to calendar') do
= custom_icon('icon_calendar')

5
changelogs/unreleased/61145-fix-button-dimensions.yml
View file

@ -1,5 +0,0 @@
---
title: Updating button dimensions according to design spec
merge_request: 28545
author:
type: fixed

4
spec/support/features/rss_shared_examples.rb
View file

@ -6,7 +6,7 @@ end
shared_examples "it has an RSS button with current_user's feed token" do
it "shows the RSS button with current_user's feed token" do
expect(page).to have_css("a:has(.fa-rss)[href*='feed_token=#{user.feed_token}'], .js-rss-button[href*='feed_token=#{user.feed_token}']")
expect(page).to have_css("a:has(.fa-rss)[href*='feed_token=#{user.feed_token}']")
end
end
@ -18,6 +18,6 @@ end
shared_examples "it has an RSS button without a feed token" do
it "shows the RSS button without a feed token" do
expect(page).to have_css("a:has(.fa-rss):not([href*='feed_token']), .js-rss-button:not([href*='feed_token'])")
expect(page).to have_css("a:has(.fa-rss):not([href*='feed_token'])")
end
end