summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorValery Sizov <valery@gitlab.com>2015-03-10 12:04:36 +0000
committerValery Sizov <valery@gitlab.com>2015-03-10 12:04:36 +0000
commit5ab27205ad25288625403b7611980e2d00928068 (patch)
tree7958e4489eb791fbe7ea8c712bc37af9ecd02aec
parent1c10220fa8bb519c97e6aea1af9620ac38c8a285 (diff)
parent6c7dfb5f37a4272144d6261fa48f1ab8d4d750fa (diff)
downloadgitlab-ci-5ab27205ad25288625403b7611980e2d00928068.tar.gz
Merge branch 'master' into 'master'
Change canceled builds color Having the same color for the failed and canceled builds is not nice so I'm proposing the following change request which implement the new CSS class `.alert-disabled` and use it for canceled builds. Here is the result: ![Screen Shot 2015-03-04 at 15.40.21](https://gitlab.com/uploads/zedtux/gitlab-ci/df01775b25/Screen_Shot_2015-03-04_at_15.40.21.png) See merge request !43
-rw-r--r--app/assets/stylesheets/gl_bootstrap.scss7
-rw-r--r--app/helpers/builds_helper.rb4
2 files changed, 10 insertions, 1 deletions
diff --git a/app/assets/stylesheets/gl_bootstrap.scss b/app/assets/stylesheets/gl_bootstrap.scss
index 7f45d64..b3f4449 100644
--- a/app/assets/stylesheets/gl_bootstrap.scss
+++ b/app/assets/stylesheets/gl_bootstrap.scss
@@ -218,3 +218,10 @@ ul.breadcrumb {
position: relative;
white-space: nowrap;
}
+
+// alerts
+.alert-disabled {
+ background-color: #e6e6e6;
+ border-color: #ebccd1;
+ color: #b0b0b0;
+}
diff --git a/app/helpers/builds_helper.rb b/app/helpers/builds_helper.rb
index 6266174..9f4eb11 100644
--- a/app/helpers/builds_helper.rb
+++ b/app/helpers/builds_helper.rb
@@ -28,8 +28,10 @@ module BuildsHelper
def build_status_alert_class(build)
if build.success?
'alert-success'
- elsif build.failed? || build.canceled?
+ elsif build.failed?
'alert-danger'
+ elsif build.canceled?
+ 'alert-disabled'
else
'alert-warning'
end