m00s / angular-photoswipe

An angular directive for PhotoSwipe image gallery
MIT License
73 stars 42 forks source link

Doesnt work on minified version using angular-yeoman #19

Open luispeerez opened 8 years ago

luispeerez commented 8 years ago

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.

m00s commented 8 years ago

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? It would helps me to help you..

cguilhermef commented 8 years ago

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 replace 'open' by 'show', and it works. Why? We don't know.

I've forked this project and will make a pull request to change this.

cguilhermef commented 8 years ago

Pull request with the modify proposed.

luispeerez commented 8 years ago

@cguilhermef Thank you, it finally worked with your solution

gabovanlugo commented 8 years ago

@cguilhermef Thank you so much. I was facing the same problem.

ginanjarfm commented 7 years ago

Hi @m00s, how about this issue?

I tried @cguilhermef 's solution but it doesn't work for me. (change open with show)

cguilhermef commented 7 years ago

@ginanjar-fahrul-m , do you tried using my pull-requested commit? I had change some lines of code to offer show attribute.

ginanjarfm commented 7 years ago

@cguilhermef Thank you so much. Now, finally worked.

luispeerez commented 7 years ago

The pull request isn't accepted yet?

nhim175 commented 6 years ago

The open attribute is a conflict with standard HTML attribute -> the value got removed after the code being complied.

nhim175 commented 6 years ago

I replace 'open' by 'show', and it works. Why? We don't know.

@cguilhermef https://www.w3schools.com/tags/att_details_open.asp As I explained above, I used to have some problem with the compact attribute as well.

cguilhermef commented 6 years ago

Hmm <details> is a new tag for me.