LC-Lightbox-LITE icon indicating copy to clipboard operation
LC-Lightbox-LITE copied to clipboard

After de-activating captions in the menu, a mouse scroll no longer activates next / previous action

Open waldenn opened this issue 6 years ago • 1 comments

Thanks for a great library!

waldenn avatar Sep 15 '19 09:09 waldenn

Hello,

Désolée je parle français. J'avais le même souci.

J'ai apporté une modification dans le js au niveau de : elems navigation with mousewheel

le code vérifie si l’élément cible ou ses parents ont des barres de défilement. Si c’est le cas, il ne déclenche pas l’événement switch_elem. En conséquence, le scroll de la souris ne fonctionne pas lorsque le texte est caché (txt_hidden est true).

Modification sur la condition qui vérifie si l’élément cible ou ses parents ont des barres de défilement : j'ai ajouté une condition supplémentaire pour vérifier si txt_hidden est true. Si c’est le cas, on force l’exécution de switch_elem.

`$(document).on('wheel', '#lcl_overlay, #lcl_window, #lcl_thumbs_nav:not(.lcl_tn_has_arr)', function(e) { if(obj != lcl_curr_obj || !lcl_curr_opts.mousewheel) {return true;} var $target = $(e.target);

// if not in window, do it!
if(!$target.is('#lcl_window') && !$target.parents('#lcl_window').length) {
	e.preventDefault();
	var delta = e.originalEvent.deltaY;
			
	if(delta > 0) 	{switch_elem('next');}
	else 			{switch_elem('prev');}	
}
		
else {
// cycle to know if parents have scrollers
var perform = true;
for(a=0; a<20; a++) {
if($target.is('#lcl_window')) {break;}
				
if($target[0].scrollHeight > $target.outerHeight() && !lcl_settings.txt_hidden) {
	perform = false;	
	break;
}
else {
	$target = $target.parent();	
	}
}
			
if(perform) {
	e.preventDefault();
	var delta = e.originalEvent.deltaY;
					
	if(delta > 0) 	{switch_elem('next');}
	else 			{switch_elem('prev');}	
	}
}

}); `

Dans mon cas, la modification a fonctionné. Prudence quand même à d'autres éventuels effets secondaires.

LCweb-ita, au cas où, tu pourras confirmer si c'est correct ou pas ?

ploppynette avatar Mar 04 '24 19:03 ploppynette