Fix link togglers jumping to top

Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/29414

app/assets/javascripts/behaviors/toggler_behavior.js

@@ -18,7 +18,7 @@
     // Button does not change visibility. If button has icon - it changes chevron style.
     //
     // %div.js-toggle-container
-    //   %a.js-toggle-button
+    //   %button.js-toggle-button
     //   %div.js-toggle-content
     //
     $('body').on('click', '.js-toggle-button', function() {

app/assets/stylesheets/framework/variables.scss

@@ -101,6 +101,7 @@ $gl-text-green: #4a2;
 $gl-text-red: #d12f19;
 $gl-text-orange: #d90;
 $gl-link-color: #3777b0;
+$gl-link-hover-color: darken($gl-link-color, 15%);
 $gl-grayish-blue: #7f8fa4;
 $gl-gray: $gl-text-color;
 $gl-gray-dark: #313236;

app/assets/stylesheets/pages/diff.scss

@@ -431,6 +431,24 @@
   border-bottom: none;
 }
+.diff-stats-summary-toggler {
+  padding: 0;
+  background-color: transparent;
+  border: 0;
+
+  color: $gl-link-color;
+
+  transition: color 0.1s linear;
+
+  &:hover,
+  &:focus {
+    outline: none;
+    text-decoration: underline;
+
+    color: $gl-link-hover-color;
+  }
+}
+
 // Mobile
 @media (max-width: 480px) {
   .diff-title {

app/assets/stylesheets/pages/merge_requests.scss

@@ -60,7 +60,17 @@
       }
       .modify-merge-commit-link {
+        padding: 0;
+
+        background-color: transparent;
+        border: 0;
+
         color: $gl-text-color;
+
+        &:hover,
+        &:focus {
+          text-decoration: underline;
+        }
       }
       .merge-param-checkbox {

app/assets/stylesheets/pages/notes.scss

@@ -426,9 +426,20 @@ ul.notes {
 }
 .discussion-toggle-button {
+  padding: 0;
+
+  background-color: transparent;
+  border: 0;
+
   line-height: 20px;
   font-size: 13px;
+  &:hover,
+  &:focus {
+    text-decoration: underline;
+    outline: none;
+  }
+
   .fa {
     margin-right: 3px;
     font-size: 10px;

app/views/discussions/_discussion.html.haml

@@ -8,7 +8,7 @@
       .discussion.js-toggle-container{ class: discussion.id, data: { discussion_id: discussion.id } }
         .discussion-header
           .discussion-actions
-            = link_to "#", class: "note-action-button discussion-toggle-button js-toggle-button" do
+            %button.note-action-button.discussion-toggle-button.js-toggle-button
               - if expanded
                 = icon("chevron-up")
               - else

app/views/projects/commits/_commit.html.haml

@@ -24,7 +24,7 @@
           .visible-xs-inline
             = render_commit_status(commit, ref: ref)
         - if commit.description?
-          %a.text-expander.hidden-xs.js-toggle-button ...
+          %button.text-expander.hidden-xs.js-toggle-button ...
         - if commit.description?
           %pre.commit-row-description.js-toggle-content

app/views/projects/diffs/_stats.html.haml

 .js-toggle-container
   .commit-stat-summary
     Showing
-    = link_to '#', class: 'js-toggle-button' do
+    %button.diff-stats-summary-toggler.js-toggle-button
       %strong= pluralize(diff_files.size, "changed file")
     with
     %strong.cgreen #{diff_files.sum(&:added_lines)} additions

app/views/projects/merge_requests/widget/open/_accept.html.haml

@@ -37,7 +37,7 @@
             = check_box_tag :should_remove_source_branch
             Remove source branch
       .accept-control
-        = link_to "#", class: "modify-merge-commit-link js-toggle-button" do
+        %button.modify-merge-commit-link.js-toggle-button
           = icon('edit')
           Modify commit message
     .js-toggle-content.hide.prepend-top-default

app/views/projects/new.html.haml

@@ -75,7 +75,7 @@
                       Gitea
                 %div
                   - if git_import_enabled?
-                    = link_to "#", class: 'btn js-toggle-button import_git' do
+                    %button.btn.js-toggle-button.import_git
                       = icon('git', text: 'Repo by URL')
                 .import_gitlab_project
                   - if gitlab_project_import_enabled?

changelogs/unreleased/29414-fix-toggle-discussion-link-jump.yml

+---
+title: Update toggle buttons to be <button>
+merge_request:
+author: