From 809aefb828e0d4b5c06304fcde2dcfced66ab4e9 Mon Sep 17 00:00:00 2001 From: randx Date: Mon, 15 Oct 2012 19:51:11 +0300 Subject: [PATCH] Minor improve to UI and code formatting of gitlab web editor --- app/assets/stylesheets/sections/tree.scss | 8 ++++++++ app/controllers/tree_controller.rb | 12 ++++++------ app/roles/repository.rb | 4 ++++ app/views/tree/_tree_file.html.haml | 9 +++++---- app/views/tree/edit.html.haml | 5 ++++- features/project/source/browse_files.feature | 2 +- features/steps/project/project_browse_files.rb | 4 ++-- lib/gitlab/file_editor.rb | 12 +++++++----- 8 files changed, 37 insertions(+), 19 deletions(-) diff --git a/app/assets/stylesheets/sections/tree.scss b/app/assets/stylesheets/sections/tree.scss index cd31b6a3..c08a93fc 100644 --- a/app/assets/stylesheets/sections/tree.scss +++ b/app/assets/stylesheets/sections/tree.scss @@ -59,3 +59,11 @@ } } } + +.tree-btn-group { + .btn { + margin-right:-3px; + padding:2px 10px; + } +} + diff --git a/app/controllers/tree_controller.rb b/app/controllers/tree_controller.rb index 8109d1c1..475e2b61 100644 --- a/app/controllers/tree_controller.rb +++ b/app/controllers/tree_controller.rb @@ -21,23 +21,23 @@ class TreeController < ProjectResourceController end def edit - @last_commit = @project.commits(@ref, @path, 1).first.sha + @last_commit = @project.last_commit_for(@ref, @path).sha end def update file_editor = Gitlab::FileEditor.new(current_user, @project, @ref) update_status = file_editor.update( - @path, - params[:content], - params[:commit_message], + @path, + params[:content], + params[:commit_message], params[:last_commit] ) - + if update_status redirect_to project_tree_path(@project, @id), :notice => "File has been successfully changed" else flash[:notice] = "You can't save file because it has been changed" - render :edit + render :edit end end end diff --git a/app/roles/repository.rb b/app/roles/repository.rb index 35093a2f..88fd90d0 100644 --- a/app/roles/repository.rb +++ b/app/roles/repository.rb @@ -32,6 +32,10 @@ module Repository Commit.commits(repo, ref, path, limit, offset) end + def last_commit_for(ref, path = nil) + commits(ref, path, 1).first + end + def commits_between(from, to) Commit.commits_between(repo, from, to) end diff --git a/app/views/tree/_tree_file.html.haml b/app/views/tree/_tree_file.html.haml index bae43087..52021267 100644 --- a/app/views/tree/_tree_file.html.haml +++ b/app/views/tree/_tree_file.html.haml @@ -5,10 +5,11 @@ = tree_file.name.force_encoding('utf-8') %small #{tree_file.mode} %span.options - = link_to "raw", project_blob_path(@project, @id), class: "btn very_small", target: "_blank" - = link_to "history", project_commits_path(@project, @id), class: "btn very_small" - = link_to "blame", project_blame_path(@project, @id), class: "btn very_small" - = link_to "Edit", edit_project_tree_path(@project, @id), class: "btn very_small" + .btn-group.tree-btn-group + = link_to "raw", project_blob_path(@project, @id), class: "btn very_small", target: "_blank" + = link_to "history", project_commits_path(@project, @id), class: "btn very_small" + = link_to "blame", project_blame_path(@project, @id), class: "btn very_small" + = link_to "edit", edit_project_tree_path(@project, @id), class: "btn very_small" - if tree_file.text? - if gitlab_markdown?(tree_file.name) .file_content.wiki diff --git a/app/views/tree/edit.html.haml b/app/views/tree/edit.html.haml index a97abc68..b81373b5 100644 --- a/app/views/tree/edit.html.haml +++ b/app/views/tree/edit.html.haml @@ -3,7 +3,10 @@ .file_holder .file_title %i.icon-file - = @tree.path.force_encoding('utf-8') + %span.file_name + = @tree.path.force_encoding('utf-8') + %span.options + = link_to "cancel editing", project_tree_path(@project, @id), class: "btn very_small" .file_content.code #editor= @tree.data diff --git a/features/project/source/browse_files.feature b/features/project/source/browse_files.feature index 9b650d64..0b8495ff 100644 --- a/features/project/source/browse_files.feature +++ b/features/project/source/browse_files.feature @@ -23,5 +23,5 @@ Feature: Project Browse files @javascript Scenario: I can edit file Given I click on "Gemfile" file in repo - And I click button "Edit" + And I click button "edit" Then I can edit code diff --git a/features/steps/project/project_browse_files.rb b/features/steps/project/project_browse_files.rb index b0a09957..7fb9dccd 100644 --- a/features/steps/project/project_browse_files.rb +++ b/features/steps/project/project_browse_files.rb @@ -32,8 +32,8 @@ class ProjectBrowseFiles < Spinach::FeatureSteps page.source.should == ValidCommit::BLOB_FILE end - Given 'I click button "Edit"' do - click_link 'Edit' + Given 'I click button "edit"' do + click_link 'edit' end Then 'I can edit code' do diff --git a/lib/gitlab/file_editor.rb b/lib/gitlab/file_editor.rb index 1d52f16a..64155967 100644 --- a/lib/gitlab/file_editor.rb +++ b/lib/gitlab/file_editor.rb @@ -1,6 +1,9 @@ module Gitlab + # GitLab file editor + # + # It gives you ability to make changes to files + # & commit this changes from GitLab UI. class FileEditor - attr_accessor :user, :project, :ref def initialize(user, project, ref) @@ -35,22 +38,21 @@ module Gitlab r.git.sh "git add ." r.git.sh "git commit -am '#{commit_message}'" output = r.git.sh "git push origin #{ref}" + if output =~ /reject/ return false end end end end - true - end - + protected + def can_edit?(path, last_commit) current_last_commit = @project.commits(ref, path, 1).first.sha last_commit == current_last_commit end - end end