there is no need for this.offsets and this.targets to be jQuery objects
no need to check for href attribute on the scrollable element
removing this.$element as it is a superfluous copy of this.$scrollElement
refactor scrollspy refresh method

Closes #13702 by merging a rebased version of it.
Merge branch 'mikegreiling-scrollspy-update'