Removed unused args
Styling changes for consistency (mostly white space: indentation, if() vs if (), trailing spaces, etc)
I tried to keep this light, there are other changes to content.js I considered:
replacing the var declarations with block scoped (let and const) declarations (I wasn't sure if this was intentional to support chrome versions <40)
removing variable assignment from while conditions in removeHighlight (line 98), I appreciate conciseness but I think you're losing readability here
I think how you're iterating through nodes in selectNode (line 131) could probably be improved (at least to lower the line length) but I think the readability is ok here so maybe this is moot
Removed unused args Styling changes for consistency (mostly white space: indentation,
if()
vsif ()
, trailing spaces, etc)I tried to keep this light, there are other changes to content.js I considered:
replacing the var declarations with block scoped (let and const) declarations (I wasn't sure if this was intentional to support chrome versions <40)
removing variable assignment from while conditions in removeHighlight (line 98), I appreciate conciseness but I think you're losing readability here
I think how you're iterating through nodes in selectNode (line 131) could probably be improved (at least to lower the line length) but I think the readability is ok here so maybe this is moot
your thoughts?