You can save time and bytes when you map jQuery to $ so you don't write jQuery(this), you write $(this).

CommentFileSizeAuthor
#2 views_flag_refresh-mapping-jquery-1591180-2.patch13.82 KByannickoo
#1 views_flag_refresh-mapping-jquery-1591180-1.patch13.84 KByannickoo

Comments

yannickoo’s picture

yannickoo
Location Berlin
commented
Status: Active » Needs review
StatusFileSize
new views_flag_refresh-mapping-jquery-1591180-1.patch13.84 KB
yannickoo’s picture

yannickoo
Location Berlin
commented
StatusFileSize
new views_flag_refresh-mapping-jquery-1591180-2.patch13.82 KB

This patch also removes some whitespaces and adds a newline at the end of the file.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

This patch looks great. Even better if you use references to items instead of calling $(), to look things up in the DOM that are already found, separate issue though.