Scroll top on filter must be configurable

vukhanhtruong / jquery-elastic-grid

jQuery Elastic Grid is a lightweight, easy to use gallery jquery plugin script inspired by Google Image Search with support for .PNG, .JPG and .GIF image files. It uses a thumbnail grid with expanding preview for displaying your images. It is also fully HTML5 and CSS3 compliant.

57 stars 32 forks source link

Scroll top on filter must be configurable #15

Open glerendegui opened 6 years ago

glerendegui commented 6 years ago

Not in all cases after a filter is applied we want the effect of scrollTop. By the case, we have a fixed menu (boostrap navbar-fixed-top). When the scrollTop goes, the filters get blocked by the menu.

I think it's easy to fix by adding some config variable and testing it in

porfolio_filter.find('a').bind('click',function(e){

.....

if(config.scrollTopOnFilter) $body.animate( { scrollTop : $this.offset().top }, settings.speed );