Commit 3a74ab29 authored by Jacob Schatz's avatar Jacob Schatz

Merge branch 'blank-state' into 'master'

Updated blank state for environments and deployments

## What does this MR do?

Adds a designed blank state to environments and deployments.

@markpundsack do we want to hide the new environments when empty? Looks weird otherwise with the green button at the top.

## What are the relevant issue numbers?

Closes #18661 

## Screenshots (if relevant)

![Screen_Shot_2016-06-21_at_09.55.03](/uploads/cf768aba3fd85a596d24b7be4fe6a6c0/Screen_Shot_2016-06-21_at_09.55.03.png)

![Screen_Shot_2016-06-21_at_09.55.10](/uploads/b7b7dba90b613cfa867b084661b36b06/Screen_Shot_2016-06-21_at_09.55.10.png)

See merge request !4818
parents ac4529f7 c207b9e0
...@@ -37,3 +37,4 @@ ...@@ -37,3 +37,4 @@
@import "framework/timeline.scss"; @import "framework/timeline.scss";
@import "framework/typography.scss"; @import "framework/typography.scss";
@import "framework/zen.scss"; @import "framework/zen.scss";
@import "framework/blank";
.blank-state {
padding-top: 20px;
padding-bottom: 20px;
text-align: center;
}
.blank-state-no-icon {
padding-top: 40px;
padding-bottom: 40px;
}
.blank-state-title {
margin-top: 0;
margin-bottom: 5px;
font-size: 19px;
font-weight: normal;
}
.blank-state-text {
margin-top: 0;
margin-bottom: $gl-padding;
font-size: 15px;
}
...@@ -3,16 +3,24 @@ ...@@ -3,16 +3,24 @@
= render "projects/pipelines/head" = render "projects/pipelines/head"
%div{ class: (container_class) } %div{ class: (container_class) }
- if can?(current_user, :create_environment, @project) - if can?(current_user, :create_environment, @project) && !@environments.blank?
.top-area .top-area
.nav-controls .nav-controls
= link_to new_namespace_project_environment_path(@project.namespace, @project), class: 'btn btn-create' do = link_to new_namespace_project_environment_path(@project.namespace, @project), class: 'btn btn-create' do
New environment New environment
- if @environments.blank? - if @environments.blank?
%ul.content-list.environments .blank-state.blank-state-no-icon
%li.nothing-here-block %h2.blank-state-title
No environments to show You don't have any environments right now.
%p.blank-state-text
Environments are places where code gets deployed, such as staging or production.
%br
= succeed "." do
= link_to "Read more about environments", help_page_path("ci", "environments")
- if can?(current_user, :create_environment, @project)
= link_to new_namespace_project_environment_path(@project.namespace, @project), class: 'btn btn-create' do
New environment
- else - else
.table-holder .table-holder
%table.table.environments %table.table.environments
......
...@@ -4,6 +4,9 @@ ...@@ -4,6 +4,9 @@
.col-lg-3 .col-lg-3
%h4.prepend-top-0 %h4.prepend-top-0
New Environment New Environment
%p Environments allow you to track deployments of your application %p
Environments allow you to track deployments of your application
= succeed "." do
= link_to "Read more about environments", help_page_path("ci", "environments")
= render 'form' = render 'form'
...@@ -13,10 +13,14 @@ ...@@ -13,10 +13,14 @@
= link_to 'Destroy', namespace_project_environment_path(@project.namespace, @project, @environment), data: { confirm: 'Are you sure you want to delete this environment?' }, class: 'btn btn-danger', method: :delete = link_to 'Destroy', namespace_project_environment_path(@project.namespace, @project, @environment), data: { confirm: 'Are you sure you want to delete this environment?' }, class: 'btn btn-danger', method: :delete
- if @deployments.blank? - if @deployments.blank?
%ul.content-list.environments .blank-state.blank-state-no-icon
%li.nothing-here-block %h2.blank-state-title
No deployments for You don't have any deployments right now.
%strong= @environment.name %p.blank-state-text
Define environments in the deploy stage(s) in
%code .gitlab-ci.yml
to track deployments here.
= link_to "Read more", help_page_path("ci", "environments"), class: "btn btn-success"
- else - else
.table-holder .table-holder
%table.table.environments %table.table.environments
......
...@@ -20,7 +20,7 @@ feature 'Environments', feature: true do ...@@ -20,7 +20,7 @@ feature 'Environments', feature: true do
context 'without environments' do context 'without environments' do
scenario 'does show no environments' do scenario 'does show no environments' do
expect(page).to have_content('No environments to show') expect(page).to have_content('You don\'t have any environments right now.')
end end
end end
...@@ -61,7 +61,7 @@ feature 'Environments', feature: true do ...@@ -61,7 +61,7 @@ feature 'Environments', feature: true do
context 'without deployments' do context 'without deployments' do
scenario 'does show no deployments' do scenario 'does show no deployments' do
expect(page).to have_content('No deployments for') expect(page).to have_content('You don\'t have any deployments right now.')
end end
end end
...@@ -108,7 +108,7 @@ feature 'Environments', feature: true do ...@@ -108,7 +108,7 @@ feature 'Environments', feature: true do
end end
scenario 'does create a new pipeline' do scenario 'does create a new pipeline' do
expect(page).to have_content('production') expect(page).to have_content('Production')
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment