Refresh group overview to match project overview

- Avatar, group name, and group description now left-aligned
- Notification setting and "New project" CTA right-aligned
  with group avatar and name
- Leave group / request access now a link next to the 'Group'
  label below the group name
- Notification setting label removed in favor of icons
  - Tooltip added to indicate notification setting
- Search option moved inside table header next to "Sort by"

4 files changed, 17 insertions, 17 deletions

diff --git a/spec/features/groups/group_settings_spec.rb b/spec/features/groups/group_settings_spec.rb
index 2cdbdcffbc3..378e4d5febc 100644
--- a/spec/features/groups/group_settings_spec.rb
+++ b/spec/features/groups/group_settings_spec.rb
@@ -18,14 +18,14 @@ describe 'Edit group settings' do
       update_path(new_group_path)
       visit new_group_full_path
       expect(current_path).to eq(new_group_full_path)
-      expect(find('h1.group-title')).to have_content(group.name)
+      expect(find('h1.home-panel-title')).to have_content(group.name)
     end
 
     it 'the old group path redirects to the new path' do
       update_path(new_group_path)
       visit old_group_full_path
       expect(current_path).to eq(new_group_full_path)
-      expect(find('h1.group-title')).to have_content(group.name)
+      expect(find('h1.home-panel-title')).to have_content(group.name)
     end
 
     context 'with a subgroup' do
@@ -37,14 +37,14 @@ describe 'Edit group settings' do
         update_path(new_group_path)
         visit new_subgroup_full_path
         expect(current_path).to eq(new_subgroup_full_path)
-        expect(find('h1.group-title')).to have_content(subgroup.name)
+        expect(find('h1.home-panel-title')).to have_content(subgroup.name)
       end
 
       it 'the old subgroup path redirects to the new path' do
         update_path(new_group_path)
         visit old_subgroup_full_path
         expect(current_path).to eq(new_subgroup_full_path)
-        expect(find('h1.group-title')).to have_content(subgroup.name)
+        expect(find('h1.home-panel-title')).to have_content(subgroup.name)
       end
     end
 
diff --git a/spec/features/groups_spec.rb b/spec/features/groups_spec.rb
index d01fc04311a..5ba40b7b5c3 100644
--- a/spec/features/groups_spec.rb
+++ b/spec/features/groups_spec.rb
@@ -201,7 +201,7 @@ describe 'Group' do
 
       visit path
 
-      expect(page).to have_css('.group-home-desc > p > strong')
+      expect(page).to have_css('.home-panel-description-markdown > p > strong')
     end
 
     it 'passes through html-pipeline' do
@@ -209,7 +209,7 @@ describe 'Group' do
 
       visit path
 
-      expect(page).to have_css('.group-home-desc > p > gl-emoji')
+      expect(page).to have_css('.home-panel-description-markdown > p > gl-emoji')
     end
 
     it 'sanitizes unwanted tags' do
@@ -217,7 +217,7 @@ describe 'Group' do
 
       visit path
 
-      expect(page).not_to have_css('.group-home-desc h1')
+      expect(page).not_to have_css('.home-panel-description-markdown h1')
     end
 
     it 'permits `rel` attribute on links' do
@@ -225,7 +225,7 @@ describe 'Group' do
 
       visit path
 
-      expect(page).to have_css('.group-home-desc a[rel]')
+      expect(page).to have_css('.home-panel-description-markdown a[rel]')
     end
   end
 
diff --git a/spec/features/projects_spec.rb b/spec/features/projects_spec.rb
index eb70a3c41c1..a9bbe0a4b31 100644
--- a/spec/features/projects_spec.rb
+++ b/spec/features/projects_spec.rb
@@ -55,30 +55,30 @@ describe 'Project' do
     it 'parses Markdown' do
       project.update_attribute(:description, 'This is **my** project')
       visit path
-      expect(page).to have_css('.project-description > .project-description-markdown > p > strong')
+      expect(page).to have_css('.home-panel-description > .home-panel-description-markdown > p > strong')
     end
 
     it 'passes through html-pipeline' do
       project.update_attribute(:description, 'This project is the :poop:')
       visit path
-      expect(page).to have_css('.project-description > .project-description-markdown > p > gl-emoji')
+      expect(page).to have_css('.home-panel-description > .home-panel-description-markdown > p > gl-emoji')
     end
 
     it 'sanitizes unwanted tags' do
       project.update_attribute(:description, "```\ncode\n```")
       visit path
-      expect(page).not_to have_css('.project-description code')
+      expect(page).not_to have_css('.home-panel-description code')
     end
 
     it 'permits `rel` attribute on links' do
       project.update_attribute(:description, 'https://google.com/')
       visit path
-      expect(page).to have_css('.project-description a[rel]')
+      expect(page).to have_css('.home-panel-description a[rel]')
     end
 
     context 'read more', :js do
       let(:read_more_selector)         { '.read-more-container' }
-      let(:read_more_trigger_selector) { '.project-home-desc .js-read-more-trigger' }
+      let(:read_more_trigger_selector) { '.home-panel-home-desc .js-read-more-trigger' }
 
       it 'does not display "read more" link on desktop breakpoint' do
         project.update_attribute(:description, 'This is **my** project')
@@ -94,7 +94,7 @@ describe 'Project' do
 
         find(read_more_trigger_selector).click
 
-        expect(page).to have_css('.project-description .is-expanded')
+        expect(page).to have_css('.home-panel-description .is-expanded')
       end
     end
   end
@@ -111,14 +111,14 @@ describe 'Project' do
     it 'shows project topics' do
       project.update_attribute(:tag_list, 'topic1')
       visit path
-      expect(page).to have_css('.project-topic-list')
+      expect(page).to have_css('.home-panel-topic-list')
       expect(page).to have_content('topic1')
     end
 
     it 'shows up to 3 project tags' do
       project.update_attribute(:tag_list, 'topic1, topic2, topic3, topic4')
       visit path
-      expect(page).to have_css('.project-topic-list')
+      expect(page).to have_css('.home-panel-topic-list')
       expect(page).to have_content('topic1, topic2, topic3 + 1 more')
     end
   end
diff --git a/spec/views/projects/_home_panel.html.haml_spec.rb b/spec/views/projects/_home_panel.html.haml_spec.rb
index 006c93686d5..908ecb898e4 100644
--- a/spec/views/projects/_home_panel.html.haml_spec.rb
+++ b/spec/views/projects/_home_panel.html.haml_spec.rb
@@ -23,7 +23,7 @@ describe 'projects/_home_panel' do
       it 'makes it possible to set notification level' do
         render
 
-        expect(view).to render_template('projects/buttons/_notifications')
+        expect(view).to render_template('shared/notifications/_new_button')
         expect(rendered).to have_selector('.notification-dropdown')
       end
     end