As of Drupal 8 all JS needs to follow our JavaScript coding standards. For this we use ESLint to check most of our standards.

Here is the summary of running ESLint on this module JS:
✖ 4 problems (4 errors, 0 warnings)

Please find the patch attached.

CommentFileSizeAuthor
scroll_to_top-js.patch2.07 KBpashupathi nath gajawada

Issue fork scroll_to_top-2715539

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

GPN created an issue. See original summary.

AstonVictor made their first commit to this issue’s fork.

AstonVictor’s picture

AstonVictor CreditAttribution: AstonVictor as a volunteer and at DevBranch commented
Version: 8.x-0.x-dev » 8.x-1.x-dev

AstonVictor opened merge request !14

AstonVictor’s picture

AstonVictor CreditAttribution: AstonVictor as a volunteer and at DevBranch commented
Assigned: pashupathi nath gajawada » Unassigned
Status: Needs review » Fixed

thanks for the patch.

fixed and merged.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.