Merge branch 'ee-50728-re-arrange-help-related-user-menu-items-into-new-help-menu' into 'master'

Resolve "Re-arrange help-related user menu items into new Help menu"

See merge request gitlab-org/gitlab-ee!7887

app/assets/stylesheets/framework/header.scss

@@ -530,8 +530,6 @@
 
 .header-user {
   &.show .dropdown-menu {
-    width: auto;
-    min-width: unset;
     max-height: 323px;
     margin-top: 4px;
     color: $gl-text-color;
@@ -548,7 +546,7 @@
         margin-right: 0;
         display: block;
         vertical-align: text-top;
-        max-width: 148px;
+        max-width: 240px;
         font-size: 12px;
 
         gl-emoji {

app/views/layouts/header/_current_user_dropdown.html.haml

@@ -19,14 +19,7 @@
   - if current_user_menu?(:settings)
     %li
       = link_to s_("CurrentUser|Settings"), profile_path
-  - if current_user_menu?(:help)
-    %li
-      = link_to _("Help"), help_path
-  - if current_user_menu?(:help) || current_user_menu?(:settings) || current_user_menu?(:profile)
-    %li.divider
-  = render 'shared/user_dropdown_contributing_link'
-  - if instance_review_permitted?
-    = render 'shared/user_dropdown_instance_review'
   - if current_user_menu?(:sign_out)
+    %li.divider
     %li
       = link_to _("Sign out"), destroy_user_session_path, class: "sign-out-link"

app/views/layouts/header/_default.html.haml

@@ -53,6 +53,12 @@
                 = sprite_icon('todo-done', size: 16)
                 %span.badge.badge-pill.todos-count{ class: ('hidden' if todos_pending_count.zero?) }
                   = todos_count_format(todos_pending_count)
+          %li.nav-item.header-help.dropdown
+            = link_to help_path, class: 'header-help-dropdown-toggle', data: { toggle: "dropdown" } do
+              = sprite_icon('question', size: 16)
+              = sprite_icon('angle-down', css_class: 'caret-down')
+            .dropdown-menu.dropdown-menu-right
+              = render 'layouts/header/help_dropdown'
           - if header_link?(:user_dropdown)
             %li.nav-item.header-user.dropdown
               = link_to current_user, class: user_dropdown_class, data: { toggle: "dropdown" } do

app/views/layouts/header/_help_dropdown.html.haml

+%ul
+  - if current_user_menu?(:help)
+    %li
+      = link_to _("Help"), help_path
+  - if current_user_menu?(:help) || current_user_menu?(:settings) || current_user_menu?(:profile)
+    = render 'shared/user_dropdown_contributing_link'
+  - if instance_review_permitted?
+    = render 'shared/user_dropdown_instance_review'

app/views/shared/_user_dropdown_contributing_link.html.haml

 %li
   = link_to "https://about.gitlab.com/contributing", target: '_blank', class: 'text-nowrap' do
     = _("Contribute to GitLab")
-    = sprite_icon('external-link', size: 16)
-%li.divider

ee/app/views/shared/_user_dropdown_instance_review.html.haml

+%li.divider
 %li
   = link_to admin_instance_review_path, target: '_blank', class: 'text-nowrap' do
     = _("Get a free instance review")
-    = sprite_icon('external-link', size: 16)
-%li.divider