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

Added custom renderer for entire layer #1215

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ishan-ironclad
Copy link

Hi! I had to add a prop to custom-render all text-items at once, rather than one at a time for a search-highlighting use case at the organization I work for. We are looking to search for arbitrary-length phrases on a PDF page, and highlight them if they exist. The main problem being solved here is that longer items of text may be broken up across textLayer items, resulting in many search hits on the document being missed. Seems like this could be a popular use case of this API, so I think it'd be worth it to get this in. Let me know if there's any documentation or testing I'd need to add to get this contribution in.
Thanks!

@wojtekmaj
Copy link
Owner

Hmmm I'm not against it, think it could be a good addition, but yeah, we'll need docs and tests before this can be merged. I can't take care of this right now.

In the meantime, have a look here: #614 (comment)

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