Use each instead of $.each

Iterating through the jQuery elements with `each` is more similar to
other iterators we use. Furthermore, we avoid declaring the `index`
variable we don't use.
This commit is contained in:
Javi Martín
2019-06-30 04:59:53 +02:00
parent 3b5d295cdc
commit 1634cd5263
3 changed files with 6 additions and 7 deletions

View File

@@ -3,8 +3,8 @@ App.Documentable =
initialize: ->
inputFiles = $(".js-document-attachment")
$.each inputFiles, (index, input) ->
App.Documentable.initializeDirectUploadInput(input)
inputFiles.each ->
App.Documentable.initializeDirectUploadInput(this)
$("#nested-documents").on "cocoon:after-remove", ->
App.Documentable.unlockUploads()

View File

@@ -2,8 +2,8 @@ App.Imageable =
initialize: ->
inputFiles = $(".js-image-attachment")
$.each inputFiles, (index, input) ->
App.Imageable.initializeDirectUploadInput(input)
inputFiles.each ->
App.Imageable.initializeDirectUploadInput(this)
$("#nested-image").on "cocoon:after-remove", ->
$("#new_image_link").removeClass("hide")

View File

@@ -3,9 +3,8 @@ App.Map =
initialize: ->
maps = $("*[data-map]")
if maps.length > 0
$.each maps, (index, map) ->
App.Map.initializeMap map
maps.each ->
App.Map.initializeMap(this)
$(".js-toggle-map").on
click: ->