Merge branch 'feature/writeboard'

app/assets/stylesheets/sections/notes.scss

@@ -284,6 +284,6 @@ ul.notes {
     margin-top: 8px;
     margin-left: 15px;
     @extend .pull-left;
-    @extend .span4;
+    width: 35%;
   }
 }

app/helpers/tab_helper.rb

@@ -73,11 +73,19 @@ module TabHelper
   end
 
   def project_tab_class
-    [:files, :edit].each do |action|
+    return "active" if current_page?(controller: "projects", action: :edit, id: @project)
+
+    if ['services', 'hooks', 'deploy_keys', 'team_members'].include? controller.controller_name
+     "active"
+    end
+  end
+
+  def project_wiki_tab_class
+    [:files, :wall].each do |action|
       return "active" if current_page?(controller: "projects", action: action, id: @project)
     end
 
-    if ['snippets', 'services', 'hooks', 'deploy_keys', 'team_members'].include? controller.controller_name
+    if ['wikis', 'snippets'].include? controller.controller_name
      "active"
     end
   end

app/views/commits/_diffs.html.haml

@@ -25,7 +25,7 @@
             %span= diff.old_path
 
             - if @commit.prev_commit
-              = link_to project_tree_path(@project, tree_join(@commit.prev_commit_id, diff.new_path)), {:class => 'btn pull-right view-file'} do
+              = link_to project_tree_path(@project, tree_join(@commit.prev_commit_id, diff.new_path)), {:class => 'btn btn-tiny pull-right view-file'} do
                 View file @
                 %span.commit-short-id= @commit.short_id(6)
           - else

app/views/hooks/index.html.haml

@@ -18,7 +18,7 @@
     .input
       = f.text_field :url, class: "text_field xxlarge"
        
-      = f.submit "Add Web Hook", class: "btn btn-primary"
+      = f.submit "Add Web Hook", class: "btn btn-create"
 %hr
 
 -if @hooks.any?

app/views/layouts/project_resource.html.haml

@@ -35,17 +35,11 @@
               Merge Requests
               %span.count.merge_counter= @project.merge_requests.opened.count
 
-        - if @project.wall_enabled
+        = nav_link(html_options: {class: "#{project_wiki_tab_class}"}) do
-          = nav_link(path: 'projects#wall') do
+          = link_to 'Wiki', project_wiki_path(@project, :index)
-            = link_to 'Wall', wall_project_path(@project)
-
-        - if @project.wiki_enabled
-          = nav_link(controller: :wikis) do
-            = link_to 'Wiki', project_wiki_path(@project, :index)
 
         - if can? current_user, :admin_project, @project
           = nav_link(html_options: {class: "#{project_tab_class}"}) do
             = link_to edit_project_path(@project), class: "stat-tab tab " do
               Settings
-
       .content= yield

app/views/projects/files.html.haml

@@ -1,4 +1,4 @@
-= render "project_head"
+= render 'wikis/nav'
 - unless @notes.empty?
   %table
     %thead

app/views/projects/wall.html.haml

@@ -1,2 +1,3 @@
+= render 'wikis/nav'
 %div.wall_page
   = render "notes/reversed_notes_with_form"

app/views/snippets/edit.html.haml

@@ -1,2 +1 @@
-= render "projects/project_head"
 = render "snippets/form"

app/views/snippets/index.html.haml

@@ -1,5 +1,4 @@
-= render "projects/project_head"
+= render 'wikis/nav'
-
 %h3.page_title
   Snippets
   %small share code pastes with others out of git repository

app/views/snippets/new.html.haml

@@ -1,2 +1 @@
-= render "projects/project_head"
 = render "snippets/form"

app/views/snippets/show.html.haml

@@ -1,5 +1,3 @@
-= render "projects/project_head"
-
 %h3.page_title
   = @snippet.title
   %small= @snippet.file_name

app/views/wikis/_nav.html.haml

@@ -0,0 +1,14 @@
+%ul.nav.nav-tabs
+  - if @project.wiki_enabled
+    = nav_link(controller: 'wikis') do
+      = link_to 'Wiki', project_wiki_path(@project, :index)
+
+  - if @project.wall_enabled
+    = nav_link(path: 'projects#wall') do
+      = link_to 'Wall', wall_project_path(@project)
+
+  = nav_link(path: 'projects#files') do
+    = link_to 'Attachments', files_project_path(@project), class: "files-tab tab"
+
+  = nav_link(controller: :snippets) do
+    = link_to 'Snippets', project_snippets_path(@project), class: "snippets-tab tab"

app/views/wikis/edit.html.haml

@@ -1,5 +1,4 @@
 %h3.page_title Editing page
-%hr
 = render 'form'
 
 .pull-right

app/views/wikis/pages.html.haml

@@ -1,3 +1,4 @@
+= render 'wikis/nav'
 %h3.page_title All Pages
 %br
 %table

app/views/wikis/show.html.haml

@@ -1,21 +1,22 @@
-%h3.page_title
+= render 'wikis/nav'
-  = @wiki.title
-  %span.pull-right
-    = link_to pages_project_wikis_path(@project), class: "btn btn-small grouped" do
-      Pages
-    - if can? current_user, :write_wiki, @project
-      = link_to history_project_wiki_path(@project, @wiki), class: "btn btn-small grouped" do
-        History
-      = link_to edit_project_wiki_path(@project, @wiki), class: "btn btn-small grouped" do
-        %i.icon-edit
-        Edit
-%br
 - if @wiki != @most_recent_wiki
-  .warning_message
+  .alert
     This is an old version of this page.
     You can view the #{link_to "most recent version", project_wiki_path(@project, @wiki)} or browse the #{link_to "history", history_project_wiki_path(@project, @wiki)}.
 
 .file_holder
+  .file_title
+    = @wiki.title
+    %span.options
+      = link_to pages_project_wikis_path(@project), class: "btn btn-tiny grouped" do
+        Pages
+      - if can? current_user, :write_wiki, @project
+        = link_to history_project_wiki_path(@project, @wiki), class: "btn btn-tiny grouped" do
+          History
+        = link_to edit_project_wiki_path(@project, @wiki), class: "btn btn-tiny grouped" do
+          %i.icon-edit
+          Edit
+
   .file_content.wiki
     = preserve do
       = markdown @wiki.content

features/project/active_tab.feature

@@ -37,7 +37,7 @@ Feature: Project active tab
 
   Scenario: On Project Wall
     Given I visit my project's wall page
-    Then the active main tab should be Wall
+    Then the active main tab should be Wiki
     And no other main tabs should be active
 
   Scenario: On Project Wiki

spec/features/gitlab_flavored_markdown_spec.rb

@@ -218,7 +218,7 @@ describe "Gitlab Flavored Markdown" do
     end
 
     it "should NOT render title in wikis#show" do
-      within(".content h3") do # page title
+      within(".content .file_title") do # page title
         page.should have_content("Circumvent ##{issue.id}")
         page.should_not have_link("##{issue.id}")
       end