Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

hyperlink and font_face problem #74

Closed
ZouhaierSebri opened this issue Apr 11, 2017 · 2 comments
Closed

hyperlink and font_face problem #74

ZouhaierSebri opened this issue Apr 11, 2017 · 2 comments
Assignees
Milestone

Comments

@ZouhaierSebri
Copy link

ZouhaierSebri commented Apr 11, 2017

Hi @gitbrent when trying to add font_face my pptx in damaged

slide.addText(
[
{ text:'Visit the ' },
{ text:'PptxGenJS Project', options:{ font_face:'Times',hyperlink:{ url:'https://github.com/gitbrent/pptxgenjs', tooltip:'Visit Homepage' } } },
{ text:' !' },
],
{ x:0.5, y:4.0, w:9, h:1, margin:0.5, fill:'F1F1F1', font_size:18 }
);

image

when i comment this code it works

startInfo += '<a:uFill>'+ genXmlColorSelection('0000FF') +'</a:uFill>';

why you add this code ?
To change color hyperlink is not possible actually by a text option it is a general properties

@gitbrent gitbrent self-assigned this Apr 11, 2017
@gitbrent gitbrent added this to the v1.5.0 milestone Apr 18, 2017
gitbrent pushed a commit that referenced this issue May 18, 2017
@gitbrent
Copy link
Owner

Hi @ZouhaierSebri ,

Thanks for reporting this.

I'd taken code from a newer version of PowerPoint and didn't know that it had deleterious effects on older versions!

I've removed this line now and presentations continue to work as they did before in newer versions, so let me know if it works for you as well now.

@ZouhaierSebri
Copy link
Author

Hi @gitbrent ,

It works like a charm.
Thank you for the fix.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

2 participants