add disable navigation config property

themyth92 / ngx-lightbox

Lightbox2 use with angular >= 5

MIT License

115 stars 70 forks source link

add disable navigation config property #29

Closed andrasbarabas closed 4 years ago

themyth92 commented 5 years ago

Hi, thanks for your PR. Could you tell me why this feature is needed ?

andrasbarabas commented 5 years ago

Hey, I was just browsing the open issues when I found this one: 28

Not sure if it'd be an overused feature though, up to you if you let it in or not.

themyth92 commented 5 years ago

Hey @eyho , I think she wants to still allow navigation on the arrow but disable it when clicking on the image. Right now, the problem is because the click event is covered the whole image instead of the arrow.

andrasbarabas commented 5 years ago

@themyth92 Oh fine, I get it now. In this case, feel free to close this PR. :)