summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--CHANGELOG1
-rw-r--r--app/assets/javascripts/project.js.coffee15
-rw-r--r--app/assets/stylesheets/pages/projects.scss3
-rw-r--r--app/views/projects/_home_panel.html.haml8
-rw-r--r--app/views/projects/buttons/_dropdown.html.haml29
-rw-r--r--app/views/projects/buttons/_fork.html.haml4
-rw-r--r--app/views/projects/buttons/_star.html.haml4
7 files changed, 38 insertions, 26 deletions
diff --git a/CHANGELOG b/CHANGELOG
index 8a24bf412d2..5d78e2556c3 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -26,6 +26,7 @@ v 7.14.0 (unreleased)
- Add fetch command to the MR page.
- Add ability to manage user email addresses via the API.
- Show buttons to add license, changelog and contribution guide if they're missing.
+ - Tweak project page buttons.
- Disabled autocapitalize and autocorrect on login field (Daryl Chan)
- Mention group and project name in creation, update and deletion notices (Achilleas Pipinellis)
- Remove redis-store TTL monkey patch
diff --git a/app/assets/javascripts/project.js.coffee b/app/assets/javascripts/project.js.coffee
index eb8c1fa1426..87c1b67a772 100644
--- a/app/assets/javascripts/project.js.coffee
+++ b/app/assets/javascripts/project.js.coffee
@@ -1,12 +1,12 @@
class @Project
constructor: ->
# Git clone panel switcher
- scope = $ '.git-clone-holder'
- if scope.length > 0
- $('a, button', scope).click ->
- $('a, button', scope).removeClass 'active'
+ cloneHolder = $('.git-clone-holder')
+ if cloneHolder.length
+ $('a, button', cloneHolder).click ->
+ $('a, button', cloneHolder).removeClass 'active'
$(@).addClass 'active'
- $('#project_clone', scope).val $(@).data 'clone'
+ $('#project_clone', cloneHolder).val $(@).data 'clone'
$(".clone").text("").append $(@).data 'clone'
# Ref switcher
@@ -24,3 +24,8 @@ class @Project
$.cookie('hide_no_password_message', 'false', { path: path })
$(@).parents('.no-password-message').remove()
e.preventDefault()
+
+ $('.js-toggle-clone-holder').on 'click', (e) ->
+ cloneHolder.toggle()
+
+ cloneHolder.hide() unless $('.empty-project').length
diff --git a/app/assets/stylesheets/pages/projects.scss b/app/assets/stylesheets/pages/projects.scss
index 21d958db80c..029b167c38e 100644
--- a/app/assets/stylesheets/pages/projects.scss
+++ b/app/assets/stylesheets/pages/projects.scss
@@ -16,7 +16,6 @@
.project-home-panel {
text-align: center;
- margin-bottom: 20px;
.project-identicon-holder {
margin-bottom: 15px;
@@ -39,7 +38,7 @@
.git-clone-holder {
max-width: 600px;
- margin: 0 auto;
+ margin: 20px auto;
}
.visibility-level-label {
diff --git a/app/views/projects/_home_panel.html.haml b/app/views/projects/_home_panel.html.haml
index 7b6b4b35c8d..159ce1ecf3f 100644
--- a/app/views/projects/_home_panel.html.haml
+++ b/app/views/projects/_home_panel.html.haml
@@ -20,8 +20,14 @@
= forked_from_project.namespace.try(:name)
- if can? current_user, :download_code, @project
+ = link_to "#", class: 'btn js-toggle-clone-holder' do
+ = icon('cloud-download fw')
+ Clone
+
+ - if can? current_user, :download_code, @project
= link_to archive_namespace_project_repository_path(@project.namespace, @project, ref: @ref, format: 'zip'), class: 'btn', rel: 'nofollow' do
- %i.fa.fa-download
+ = icon('download fw')
+ Download
= render 'projects/buttons/dropdown'
diff --git a/app/views/projects/buttons/_dropdown.html.haml b/app/views/projects/buttons/_dropdown.html.haml
index 99c2ed62545..cade930c8cc 100644
--- a/app/views/projects/buttons/_dropdown.html.haml
+++ b/app/views/projects/buttons/_dropdown.html.haml
@@ -1,31 +1,32 @@
- if current_user
%span.dropdown
%a.dropdown-toggle.btn.btn-new{href: '#', "data-toggle" => "dropdown"}
- %i.fa.fa-plus
+ = icon('plus')
%ul.dropdown-menu
- - if @project.issues_enabled && can?(current_user, :create_issue, @project)
+ - if can?(current_user, :create_issue, @project)
%li
- = link_to url_for_new_issue, title: "New Issue" do
+ = link_to url_for_new_issue do
+ = icon('exclamation-circle fw')
New issue
- - if @project.merge_requests_enabled && can?(current_user, :create_merge_request, @project)
+ - if can?(current_user, :create_merge_request, @project)
%li
- = link_to new_namespace_project_merge_request_path(@project.namespace, @project), title: "New Merge Request" do
+ = link_to new_namespace_project_merge_request_path(@project.namespace, @project) do
+ = icon('tasks fw')
New merge request
- - if @project.snippets_enabled && can?(current_user, :create_snippet, @project)
+ - if can?(current_user, :create_snippet, @project)
%li
- = link_to new_namespace_project_snippet_path(@project.namespace, @project), title: "New Snippet" do
+ = link_to new_namespace_project_snippet_path(@project.namespace, @project) do
+ = icon('file-text-o fw')
New snippet
- - if can?(current_user, :admin_project_member, @project)
- %li
- = link_to namespace_project_project_members_path(@project.namespace, @project), title: "New project member" do
- New project member
- - if can? current_user, :push_code, @project
+ - if can?(current_user, :push_code, @project)
%li.divider
%li
= link_to new_namespace_project_branch_path(@project.namespace, @project) do
- New git branch
+ = icon('code-fork fw')
+ New branch
%li
= link_to new_namespace_project_tag_path(@project.namespace, @project) do
- New git tag
+ = icon('tags fw')
+ New tag
diff --git a/app/views/projects/buttons/_fork.html.haml b/app/views/projects/buttons/_fork.html.haml
index f0483c79edc..854c154824d 100644
--- a/app/views/projects/buttons/_fork.html.haml
+++ b/app/views/projects/buttons/_fork.html.haml
@@ -1,13 +1,13 @@
- if current_user && can?(current_user, :fork_project, @project)
- if current_user.already_forked?(@project) && current_user.manageable_namespaces.size < 2
= link_to namespace_project_path(current_user, current_user.fork_of(@project)), title: 'Go to your fork', class: 'btn' do
- = icon('code-fork')
+ = icon('code-fork fw')
Fork
%span.count
= @project.forks_count
- else
= link_to new_namespace_project_fork_path(@project.namespace, @project), title: "Fork project", class: 'btn' do
- = icon('code-fork')
+ = icon('code-fork fw')
Fork
%span.count
= @project.forks_count
diff --git a/app/views/projects/buttons/_star.html.haml b/app/views/projects/buttons/_star.html.haml
index b5f14b43bfd..5d7df5ae099 100644
--- a/app/views/projects/buttons/_star.html.haml
+++ b/app/views/projects/buttons/_star.html.haml
@@ -1,6 +1,6 @@
- if current_user
= link_to toggle_star_namespace_project_path(@project.namespace, @project), class: 'btn star-btn toggle-star', method: :post, remote: true do
- = icon('star')
+ = icon('star fw')
- if current_user.starred?(@project)
Unstar
- else
@@ -16,7 +16,7 @@
- else
= link_to new_user_session_path, class: 'btn has_tooltip star-btn', title: 'You must sign in to star a project' do
- = icon('star')
+ = icon('star fw')
Star
%span.count
= @project.star_count