Change the way we apply bouncefix

mlworks commented 10 years ago

First ever PR, hope I did this right!

Status: Opened for visibility Reviewers: @tedtate @scalvert @kpeatt

Changes

Previously, we were only adding bouncefix to pinny__content which made other absolute positioned scrollpanes within pinny__content unscrollable. (See: WHSmith tablet's sliding cards navigation)
Now, we will add bouncefix to pinny--is-scrollable and we can add that modifier class to any scrollpanes within Pinny that needs scrolling.
pinny__content by default will have the pinny--is-scrollable modifier class attached.
Research resources
https://github.com/jaridmargolin/bouncefix.js

kpeatt commented 10 years ago

@mlworks Can you add tests for this? Also what can I do to test to make sure this works myself?

scalvert commented 10 years ago

@kpeatt not sure there's any tests that could be written for this as it's dependent on content.