This seems like it might be expensive to call each time `scrollSpy()` is invoked\u2014unless there's a need to support HTML that might have been modified post page load.    Could it be created once in the constructor and then references as `this.links` instead?