Merge branch 'winh-eslint-no-jquery/no-animate' into 'master'

Disable ESLint rule no-jquery/no-animate inline

See merge request gitlab-org/gitlab!21879

.eslintrc.yml

@@ -45,7 +45,6 @@ rules:
   vue/no-use-v-if-with-v-for: off
   vue/no-v-html: off
   vue/use-v-on-exact: off
-  no-jquery/no-animate: off
   # all offenses of no-jquery/no-animate-toggle are false positives ( $toast.show() )
   no-jquery/no-animate-toggle: off
   no-jquery/no-event-shorthand: off

app/assets/javascripts/awards_handler.js

@@ -506,6 +506,8 @@ export class AwardsHandler {
     const options = {
       scrollTop: $('.awards').offset().top - 110,
     };
+
+    // eslint-disable-next-line no-jquery/no-animate
     return $('body, html').animate(options, 200);
   }
 

app/assets/javascripts/lib/utils/common_utils.js

@@ -247,6 +247,7 @@ export const scrollToElement = element => {
   }
   const { top } = $el.offset();
 
+  // eslint-disable-next-line no-jquery/no-animate
   return $('body, html').animate(
     {
       scrollTop: top - contentTop(),

app/assets/javascripts/pages/admin/admin.js

@@ -36,6 +36,8 @@ export default function adminInit() {
   $('.log-bottom').on('click', e => {
     e.preventDefault();
     const $visibleLog = $('.file-content:visible');
+
+    // eslint-disable-next-line no-jquery/no-animate
     $visibleLog.animate(
       {
         scrollTop: $visibleLog.find('ol').height(),