Open nexushoratio opened 9 months ago
Rather than make yet another service, it is probably appropriate to roll this into ScrollerService
itself. And while doing so, cover what LinkedInToolbarService
is doing as well.
This does leave secondary Scroller
s in the wind, however.
We can worry about that detail later, but to capture some initial thoughts:
base
on the fly rather than destroy. Though that would require enhancing to allow a null base, it would definitely simplify the various getters.
When the page resizes, the text "For Business", can wrap, increasing the height of the navbar, throwing everything off.
Bah.
Dispatcher
for thisScroller
to take changesService
ScrollerService
LinkedIn
should expose #globals.Scroller
via setter, empty sets to nullLinkedInToolbarService
?)LinkedIn.navbarScrollerFixup()