Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
waldenn opened this issue Sep 15, 2019 · 1 comment

Comments

@waldenn
Copy link

waldenn commented Sep 15, 2019

Thanks for a great library!

@ploppynette
Copy link

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 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants