Merge branch 'ui-improvements' into 'master'

Ui improvements

* show page context title instead of `Dashboard` keyword
* fix event feed overflow
* remove annoying tooltips where its not needed
* fix tests broken in previous mr

See merge request !1242

app/assets/stylesheets/base/gl_variables.scss

@@ -136,7 +136,7 @@ $kbd-bg:                      #333;
 //== Buttons
 //
 //##
-$btn-default-color:              #7f8fa4;
+$btn-default-color:              $gl-text-color;
 $btn-default-bg:                 #fff;
 $btn-default-border:             #e7e9ed;
 

app/assets/stylesheets/generic/common.scss

@@ -385,7 +385,7 @@ table {
     a {
       padding: 15px;
       font-size: 20px;
-      line-height: 27px;
+      line-height: 28px;
       color: #7f8fa4;
       border-bottom: 2px solid transparent;
 

app/assets/stylesheets/pages/events.scss

@@ -34,8 +34,6 @@
 
   .event-title {
     @include str-truncated(calc(100% - 174px));
-
-    max-width: 70%;
     font-weight: 500;
 
     .author_name {

app/helpers/diff_helper.rb

@@ -137,7 +137,7 @@ module DiffHelper
     # Always use HTML to handle case where JSON diff rendered this button
     params_copy.delete(:format)
 
-    link_to url_for(params_copy), id: "commit-diff-viewtype", class: (params[:view] != 'parallel' ? 'btn btn-sm active' : 'btn btn-sm') do
+    link_to url_for(params_copy), id: "inline-diff-btn", class: (params[:view] != 'parallel' ? 'btn btn-sm active' : 'btn btn-sm') do
       'Inline'
     end
   end
@@ -148,7 +148,7 @@ module DiffHelper
     # Always use HTML to handle case where JSON diff rendered this button
     params_copy.delete(:format)
 
-    link_to url_for(params_copy), id: "commit-diff-viewtype", class: (params[:view] == 'parallel' ? 'btn active btn-sm' : 'btn btn-sm') do
+    link_to url_for(params_copy), id: "parallel-diff-btn", class: (params[:view] == 'parallel' ? 'btn active btn-sm' : 'btn btn-sm') do
       'Side-by-side'
     end
   end

app/helpers/events_helper.rb

@@ -30,7 +30,6 @@ module EventsHelper
       class: "event-filter-link btn btn-default #{active}",
       id:    "#{key}_event_filter",
       title: "Filter by #{tooltip.downcase}",
-      data:  { toggle: 'tooltip', placement: 'top' }
     }
 
     link_to request.path, link_opts do

app/views/dashboard/_groups_head.html.haml

@@ -3,5 +3,5 @@
     = link_to dashboard_groups_path, title: 'Your groups', data: {placement: 'right'} do
       Your Groups
   = nav_link(page: [explore_groups_path]) do
-    = link_to explore_groups_path, title: 'Explore groups', data: {toggle: 'tooltip', placement: 'bottom'} do
+    = link_to explore_groups_path, title: 'Explore groups', data: {placement: 'bottom'} do
       Explore Groups

app/views/dashboard/_projects_head.html.haml

@@ -6,5 +6,5 @@
     = link_to starred_dashboard_projects_path, title: 'Starred Projects', data: {placement: 'right'} do
       Starred Projects
   = nav_link(page: [explore_root_path, trending_explore_projects_path, starred_explore_projects_path, explore_projects_path], html_options: { class: 'hidden-xs' }) do
-    = link_to explore_root_path, title: 'Explore', data: {toggle: 'tooltip', placement: 'bottom'} do
+    = link_to explore_root_path, title: 'Explore', data: {placement: 'right'} do
       Explore Projects

app/views/dashboard/activity.html.haml

@@ -2,6 +2,7 @@
   - if current_user
     = auto_discovery_link_tag(:atom, dashboard_url(format: :atom, private_token: current_user.private_token), title: "All activity")
 
+- header_title  "Activity", activity_dashboard_path
 = render 'dashboard/activity_head'
 
 %section.activities

app/views/dashboard/groups/index.html.haml

@@ -1,4 +1,5 @@
 - page_title "Groups"
+- header_title  "Groups", dashboard_groups_path
 = render 'dashboard/groups_head'
 
 .gray-content-block

app/views/dashboard/issues.html.haml

@@ -1,14 +1,11 @@
 - page_title "Issues"
+- header_title  "Issues", issues_dashboard_path(assignee_id: current_user.id)
 = content_for :meta_tags do
   - if current_user
     = auto_discovery_link_tag(:atom, issues_dashboard_url(format: :atom, private_token: current_user.private_token), title: "#{current_user.name} issues")
 
-%h3.page-title
-  Issues
-
 %p.light
   List all issues from all projects you have access to.
-%hr
 
 .append-bottom-20
   .pull-right

app/views/dashboard/merge_requests.html.haml

@@ -1,11 +1,9 @@
 - page_title "Merge Requests"
-%h3.page-title
-  Merge Requests
-
+- header_title  "Merge Requests", merge_requests_dashboard_path(assignee_id: current_user.id)
 
 %p.light
   List all merge requests from all projects you have access to.
-%hr
+
 .append-bottom-20
   = render 'shared/issuable/filter', type: :merge_requests
 = render 'shared/merge_requests'

app/views/dashboard/milestones/index.html.haml

@@ -1,12 +1,9 @@
 - page_title "Milestones"
-%h3.page-title
-  Milestones
-  %span.pull-right #{@dashboard_milestones.count} milestones
+- header_title  "Milestones", dashboard_milestones_path
 
 %p.light
   List all milestones from all projects you have access to.
 
-%hr
 
 = render 'shared/milestones_filter'
 .milestones

app/views/dashboard/projects/starred.html.haml

@@ -1,4 +1,5 @@
 - page_title "Starred Projects"
+- header_title "Projects", (current_user ? root_path : explore_root_path)
 = render 'dashboard/projects_head'
 
 - if @projects.any?

app/views/dashboard/show.html.haml

@@ -2,6 +2,7 @@
   - if current_user
     = auto_discovery_link_tag(:atom, dashboard_url(format: :atom, private_token: current_user.private_token), title: "All activity")
 
+- header_title "Projects", (current_user ? root_path : explore_root_path)
 = render 'dashboard/projects_head'
 
 - if @last_push

app/views/explore/groups/index.html.haml

@@ -1,4 +1,5 @@
 - page_title "Groups"
+- header_title "Groups", (current_user ? dashboard_groups_path : explore_groups_path)
 - if current_user
   = render 'dashboard/groups_head'
 .gray-content-block.clearfix

app/views/layouts/dashboard.html.haml

@@ -1,5 +1,6 @@
 - page_title    "Dashboard"
-- header_title  "Dashboard", root_path
+- unless @header_title
+  - header_title  "Dashboard", root_path
 - sidebar       "dashboard"
 
 = render template: "layouts/application"

app/views/layouts/explore.html.haml

@@ -1,6 +1,7 @@
 - page_title    "Explore"
 - if current_user
-  - header_title  "Dashboard", root_path
+  - unless @header_title
+    - header_title "Projects", (current_user ? root_path : explore_root_path)
 - else
   - header_title  "Explore GitLab", explore_root_path
 - sidebar       "dashboard"

app/views/layouts/snippets.html.haml

@@ -1,6 +1,6 @@
 - page_title    'Snippets'
 - if current_user
-  - header_title  "Dashboard", root_path
+  - header_title  "Snippets", user_snippets_path(current_user)
 - else
   - header_title  'Snippets', snippets_path
 - sidebar       "dashboard"

app/views/snippets/current_user_index.html.haml

@@ -1,14 +1,15 @@
 - page_title "Your Snippets"
 = render 'head'
 
-.slead
-  Share code pastes with others out of git repository
-
+.gray-content-block
   .pull-right
-    = link_to new_snippet_path, class: "btn btn-new btn-sm", title: "New Snippet" do
+    = link_to new_snippet_path, class: "btn btn-new", title: "New Snippet" do
       Add new snippet
 
-%ul.nav.nav-tabs
+  .oneline
+    Share code pastes with others out of git repository
+
+%ul.nav.nav-tabs.prepend-top-20
   = nav_tab :scope, nil do
     = link_to user_snippets_path(@user) do
       All

app/views/snippets/index.html.haml

@@ -2,8 +2,14 @@
 - if current_user
   = render 'head'
 
-.slead
-  Public snippets created by you and other users are listed here
+.gray-content-block
+  - if current_user
+    .pull-right
+      = link_to new_snippet_path, class: "btn btn-new", title: "New Snippet" do
+        Add new snippet
+
+  .oneline
+    Public snippets created by you and other users are listed here
 
 = render 'snippets'
 

features/steps/project/merge_requests.rb

@@ -128,7 +128,8 @@ class Spinach::Features::ProjectMergeRequests < Spinach::FeatureSteps
   end
 
   step 'I should see the proper Inline and Side-by-side links' do
-    expect(page).to have_css('#commit-diff-viewtype', count: 2)
+    expect(page).to have_css('#parallel-diff-btn', count: 1)
+    expect(page).to have_css('#inline-diff-btn', count: 1)
   end
 
   step 'I switch to the merge request\'s comments tab' do
@@ -225,13 +226,13 @@ class Spinach::Features::ProjectMergeRequests < Spinach::FeatureSteps
 
   step 'I click link "Hide inline discussion" of the second file' do
     page.within '.files [id^=diff]:nth-child(2)' do
-      find('.js-toggle-diff-comments').click
+      find('.js-toggle-diff-comments').trigger('click')
     end
   end
 
   step 'I click link "Show inline discussion" of the second file' do
     page.within '.files [id^=diff]:nth-child(2)' do
-      find('.js-toggle-diff-comments').click
+      find('.js-toggle-diff-comments').trigger('click')
     end
   end
 

features/steps/shared/diff_note.rb

@@ -198,7 +198,7 @@ module SharedDiffNote
   end
 
   step 'I click side-by-side diff button' do
-    click_link "Side-by-side"
+    find('#parallel-diff-btn').trigger('click')
   end
 
   step 'I see side-by-side diff button' do