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

feat: add javascriptFile property (The same as styleSheetFile property but for js files) #91

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

thederickff
Copy link

Hello,
I am using ngx-print with Ionic+Angular, and I figured out that the <ion-icon></ion-icon> element was not being rendered, because of the missing ionicons's javascript file:
<script src="https://unpkg.com/[email protected]/dist/ionicons.js"></script>
Therefore, I decided to add the possibility for users being able to add scripts to the printing window page just like it is possible to do with the stylesheets. I really need these icons on my prints, but I don't know for sure yet how can I get this new version of 'ngx-print' into my project, I'll be waiting for your response.

@broem broem self-assigned this Jul 8, 2020
@thederickff
Copy link
Author

thederickff commented Jul 28, 2020

Hello, I would really appreciate if you could add this functionality. You don't even need to merge it, you could add it into the project by yourself.

@thederickff thederickff changed the title feat: add the possibility to inject /assets/'s or CDN's js files feat: add javascriptFile property (Does the same as styleSheetFile property but for js files) Jul 28, 2020
@thederickff thederickff changed the title feat: add javascriptFile property (Does the same as styleSheetFile property but for js files) feat: add javascriptFile property (The same as styleSheetFile property but for js files) Jul 28, 2020
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

Successfully merging this pull request may close these issues.

2 participants