diff options
author | Annabel Dunstone Gray <annabel.dunstone@gmail.com> | 2016-11-14 18:06:34 +0300 |
---|---|---|
committer | Annabel Dunstone Gray <annabel.dunstone@gmail.com> | 2016-11-14 18:06:34 +0300 |
commit | 7f151a7898bf6b836ba7ae0b53d718a1263b5d6f (patch) | |
tree | d5ed8a389d47183664ef2e9be446e038f0fdf50c /app/views | |
parent | 36fa5d660df96c49484cd27695da1df6e361eb8f (diff) | |
parent | d9a0682d891e26a8ad0e5eb1a1b9b41d15fb5b27 (diff) |
Merge branch 'always-show-download-button' into 'master'
Project download buttons always show
## What does this MR do?
Removes `hidden-xs hidden-sm` classes from `download-button` so it doesn’t hide itself individually from the rest of the `project-repo-buttons` block
## Why was this MR needed?
Fixes #23960. Not sure why this element had these classes in the first place, but at the very least we shouldn’t be hiding the Download button at the `hidden-sm` threshold since that’s still fairly large viewport and users with viewports of that size likely still want to download source code.
## Screenshots (if relevant)
Screenshots taken at ~830px width
__Before__
![before](/uploads/21955cf7a12f6eb9cfef96bbb27b7110/before.png)
__After__
![after](/uploads/c082884f225184d56a920adbc0a0a732/after.png)
## What are the relevant issue numbers?
#23960
See merge request !7405
Diffstat (limited to 'app/views')
-rw-r--r-- | app/views/projects/buttons/_download.html.haml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/views/projects/buttons/_download.html.haml b/app/views/projects/buttons/_download.html.haml index 7e83a88913a..7b995bd8735 100644 --- a/app/views/projects/buttons/_download.html.haml +++ b/app/views/projects/buttons/_download.html.haml @@ -1,5 +1,5 @@ - if !project.empty_repo? && can?(current_user, :download_code, project) - %span{class: 'hidden-xs hidden-sm download-button'} + %span{class: 'download-button'} .dropdown.inline %button.btn{ 'data-toggle' => 'dropdown' } = icon('download') |