-
Notifications
You must be signed in to change notification settings - Fork 42
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
Doesnt work on minified version using angular-yeoman #19
Comments
Hi @luispeerez, thanks for you're feedback. Are you doing something similar to what shown in the demo here? Is your code open-source? Or can you post here a snippet or take a few minutes to build a plunk? |
We had the same problem here. After one hard day of 'console.logs' and tests, we found an issue in the use of property 'open', in the directive script. I've forked this project and will make a pull request to change this. |
Pull request with the modify proposed. |
@cguilhermef Thank you, it finally worked with your solution |
@cguilhermef Thank you so much. I was facing the same problem. |
Hi @m00s, how about this issue? I tried @cguilhermef 's solution but it doesn't work for me. (change |
@ginanjar-fahrul-m , do you tried using my pull-requested commit? I had change some lines of code to offer |
@cguilhermef Thank you so much. Now, finally worked. |
The pull request isn't accepted yet? |
The |
@cguilhermef https://www.w3schools.com/tags/att_details_open.asp |
Hmm |
The directive template is inserted on the DOM but when I click on an image the photoswipe gallery doesn't show.
In the minified version of my yeoman proyect I can see the Photoswipe and ng-photoswipe code.
The text was updated successfully, but these errors were encountered: