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

Is there a way to add text as link(url)? #41

Closed
nirber opened this issue Sep 20, 2015 · 8 comments
Closed

Is there a way to add text as link(url)? #41

nirber opened this issue Sep 20, 2015 · 8 comments

Comments

@nirber
Copy link

nirber commented Sep 20, 2015

No description provided.

@rkusa
Copy link
Owner

rkusa commented Sep 20, 2015

There is currently no way for creating links. In PDF, links are a subtype of annotations. That is, I would have to add annotations first. I like to have this feature in pdfjs, but I probably don't have the time to implement it in the near future 😞

@nirber
Copy link
Author

nirber commented Sep 20, 2015

Thanks

@nirber
Copy link
Author

nirber commented Sep 20, 2015

I just found out that if you just add text 'www.example.com' it is being parsed as a link in acrobat reader or google pdf viewer - so this is great.

But, there is another problem - if I add 'www.example.com/slug/test' i get 'www.example.com/ slug/ test' in the pdf doc. Is there a way to fix this (the 'spaces' after the '/' in the url)?

rkusa added a commit that referenced this issue Sep 20, 2015
@rkusa
Copy link
Owner

rkusa commented Sep 20, 2015

Thanks for reporting! Should be fixed. Please let me know, if it works for you

@nirber
Copy link
Author

nirber commented Sep 20, 2015

Wow, super fast.
Did npm update, but it didn't pulled the latest commit. How to get it?

@rkusa
Copy link
Owner

rkusa commented Sep 20, 2015

I've not yet update NPM. You could install from Github for now: npm install rkusa/pdfjs

@nirber
Copy link
Author

nirber commented Sep 20, 2015

Thank you so much! That did the trick (and the fix is working well)

@rkusa
Copy link
Owner

rkusa commented May 3, 2017

Just for the record. This is now possible b1240a9

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

2 participants