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

Doesnt work on minified version using angular-yeoman #19

Open
luispeerez opened this issue Jul 11, 2016 · 12 comments
Open

Doesnt work on minified version using angular-yeoman #19

luispeerez opened this issue Jul 11, 2016 · 12 comments

Comments

@luispeerez
Copy link

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
Copy link
Owner

m00s commented Jul 11, 2016

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
Copy link

cguilhermef commented Jul 28, 2016

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
Copy link

Pull request with the modify proposed.

@luispeerez
Copy link
Author

@cguilhermef Thank you, it finally worked with your solution

@gabovanlugo
Copy link

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

@ginanjarfm
Copy link

Hi @m00s, how about this issue?

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

@cguilhermef
Copy link

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

@ginanjarfm
Copy link

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

@luispeerez
Copy link
Author

The pull request isn't accepted yet?

@nhim175
Copy link

nhim175 commented Jul 5, 2018

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

@nhim175
Copy link

nhim175 commented Jul 5, 2018

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
Copy link

Hmm <details> is a new tag for me.

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

6 participants