Skip to content

Add hyperlink parsing under pictures#557

Merged
Progi1984 merged 3 commits into
PHPOffice:developfrom
ulziibuyan:develop
Oct 9, 2019
Merged

Add hyperlink parsing under pictures#557
Progi1984 merged 3 commits into
PHPOffice:developfrom
ulziibuyan:develop

Conversation

@ulziibuyan
Copy link
Copy Markdown
Contributor

@ulziibuyan ulziibuyan commented Oct 4, 2019

Hello @Progi1984, I added hyperlinks to pictures using PowerPoint. When I tried to read them using your awesome library, the hyperlinks were not showing up. Turns out, they were not being parsed. This PR adds support and actually extends #49. Please let me know what can I do to make this PR better. Thanks!

@auto-assign auto-assign Bot requested a review from Progi1984 October 4, 2019 21:40
@ulziibuyan ulziibuyan changed the title Add support for image hyperlinks Add hyperlink parsing under pictures Oct 4, 2019
@Progi1984
Copy link
Copy Markdown
Member

@ulziibuyan Thank you for your contribution. Could you update Changelog.md ? And the perfect thing, could you update the method displayShapeInfo for displaying hyperlink on images in Sample 12 ? Thanks

@Progi1984
Copy link
Copy Markdown
Member

@ulziibuyan After that, I could merge it.

@ulziibuyan
Copy link
Copy Markdown
Contributor Author

ulziibuyan commented Oct 8, 2019

@Progi1984 I'm not sure what caused the build to fail only for PHP: 7.1. Any ideas? https://travis-ci.org/PHPOffice/PHPPresentation/builds/595205590. The check has suddenly passed.

@Progi1984
Copy link
Copy Markdown
Member

@ulziibuyan I reset the failed Travis test.

@Progi1984 Progi1984 added this to the 0.10.0 milestone Oct 9, 2019
@Progi1984 Progi1984 merged commit fa28cf1 into PHPOffice:develop Oct 9, 2019
@Progi1984
Copy link
Copy Markdown
Member

Thank you @ulziibuyan for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants