[#22826] Ensure find_file_link and download button is part of file tree header

- Move the find_file_link and download partial into the project tree_header partial
- Added Changelog
This commit is contained in:
TM Lee 2017-03-10 09:58:48 +08:00
parent 1005389f70
commit 7940a5d49c
5 changed files with 37 additions and 10 deletions

View file

@ -160,7 +160,6 @@
.tree-controls {
float: right;
margin-top: 11px;
position: relative;
z-index: 2;

View file

@ -1,3 +1,7 @@
.tree-controls
= render 'projects/find_file_link'
= render 'projects/buttons/download', project: @project, ref: @ref
.tree-ref-holder
= render 'shared/ref_switcher', destination: 'tree', path: @path

View file

@ -7,12 +7,4 @@
= render 'projects/last_push'
%div{ class: container_class }
.tree-controls
= render 'projects/find_file_link'
= render 'projects/buttons/download', project: @project, ref: @ref
#tree-holder.tree-holder.clearfix
.nav-block
= render 'projects/tree/tree_header', tree: @tree
= render 'projects/tree/tree_content', tree: @tree
= render 'projects/files'

View file

@ -0,0 +1,4 @@
---
title: Fix UI inconsistency different files view (find file button missing)
merge_request: 9847
author: TM Lee

View file

@ -0,0 +1,28 @@
require 'spec_helper'
feature 'Find files button in the tree header', feature: true do
given(:user) { create(:user) }
given(:project) { create(:project) }
background do
login_as(user)
project.team << [user, :developer]
end
scenario 'project main screen' do
visit namespace_project_path(
project.namespace,
project
)
expect(page).to have_selector('.tree-controls .shortcuts-find-file')
end
scenario 'project tree screen' do
visit namespace_project_tree_path(
project.namespace,
project,
project.default_branch
)
expect(page).to have_selector('.tree-controls .shortcuts-find-file')
end
end