search

DOI-USGS / OWDI-Lower-Colorado-Drought-Vis

Lower Colorado Drought Visualization
Other
13 stars 12 forks source link

mobile nav fix #570

Closed mwernimont closed 8 years ago

mwernimont commented 8 years ago

@jiwalker-usgs Is this a stupid fix?

The issue was the offset for the desktop was too big for the mobile phones and the nav was highlighting the numbers too late in the scroll or highlighting the wrong number for the section...so thought having an if statement to check screen size and then give the offset would be a good solution, but I'm no Walker.

jiwalker-usgs commented 8 years ago

I think there is a way to change the trigger from the center to the top which should achieve what you are trying to do with offset.

jiwalker-usgs commented 8 years ago

Try playing around with the triggerHook and remove the offsets.

mwernimont commented 8 years ago

@jiwalker-usgs Triggerhook works as it should, don't know why yesterday i thought it was off on mobile, but thanks for the tip!