angular-photoswipe icon indicating copy to clipboard operation
angular-photoswipe copied to clipboard

Doesnt work on minified version using angular-yeoman

Open luispeerez opened this issue 8 years ago • 12 comments

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.

luispeerez avatar Jul 11 '16 19:07 luispeerez

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..

m00s avatar Jul 11 '16 21:07 m00s

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 avatar Jul 28 '16 11:07 cguilhermef

Pull request with the modify proposed.

cguilhermef avatar Jul 28 '16 16:07 cguilhermef

@cguilhermef Thank you, it finally worked with your solution

luispeerez avatar Aug 02 '16 22:08 luispeerez

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

gabovanlugo avatar Sep 06 '16 22:09 gabovanlugo

Hi @m00s, how about this issue?

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

ginanjarfm avatar Jan 09 '17 09:01 ginanjarfm

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

cguilhermef avatar Jan 09 '17 10:01 cguilhermef

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

ginanjarfm avatar Jan 09 '17 11:01 ginanjarfm

The pull request isn't accepted yet?

luispeerez avatar Jan 10 '17 15:01 luispeerez

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

nhim175 avatar Jul 05 '18 06:07 nhim175

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.

nhim175 avatar Jul 05 '18 06:07 nhim175

Hmm <details> is a new tag for me.

cguilhermef avatar Jul 16 '18 16:07 cguilhermef