Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] fixed es-lint issues #4223

Closed
wants to merge 6 commits into from
Closed

[WIP] fixed es-lint issues #4223

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
6d14c25
fixed es-lint issues
Oct 3, 2019
e72564b
removed unnecessary try/catch
Oct 5, 2019
ea0a5fe
used forEach instead of ‘for’
Oct 5, 2019
a1e5dd3
fixed forEach typo in initializeAllFollowButts()
Oct 5, 2019
9d6533b
uncommented shouldNotFetch
Oct 5, 2019
5a6f2af
switch to anonymous functions and let
Oct 8, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
used forEach instead of ‘for’
  • Loading branch information
uchihamalolan committed Oct 5, 2019
commit ea0a5fe741a9f8531e60c8f9a33a5bcc0d5ccc57
4 changes: 1 addition & 3 deletions app/assets/javascripts/initializers/initializeAllFollowButts.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,5 @@ function initializeFollowButt(butt) {

function initializeAllFollowButts() {
var followButts = document.getElementsByClassName('follow-action-button');
for (var i = 0; i < followButts.length; i += 1) {
initializeFollowButt(followButts[i]);
}
followButts.forEach(initializeFollowButt(followButts));
}