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

Search highlighting not cleared after search is done #7468

Closed
qwer1304 opened this issue Jul 2, 2016 · 0 comments
Closed

Search highlighting not cleared after search is done #7468

qwer1304 opened this issue Jul 2, 2016 · 0 comments
Labels

Comments

@qwer1304
Copy link

qwer1304 commented Jul 2, 2016

Link to PDF file (or attach file here): https://www.cca.edu/sites/default/files/pdf/08/word-to-pdf.pdf

Configuration:

  • Web browser and its version: Chrome Version 54.0.2786.0 canary (64-bit)
  • Operating system and its version: Win 10, 1511
  • PDF.js version: 1.5.294
  • Is an extension: Yes

Steps to reproduce the problem:

  1. Open pdf
  2. Search for a string (e.g., pdf)
  3. Close the search box

What is the expected behavior? (add screenshot)
Text highlighting should be removed

What went wrong? (add screenshot)
Text highlighting stays

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

timvandermeij added a commit to timvandermeij/pdf.js that referenced this issue Sep 21, 2018
…ency on the find controller from the find bar

If the find bar is closed, the matches for the rendered pages remain
highlighted (issue mozilla#7468 tracks this). The only purpose of the `active`
member variable is to not highlight matches for not yet rendered pages,
so if the find bar is closed and a not yet rendered page is scrolled
into view, then the text layer is built without match highlights.

However, this is not the behavior of the built-in version in Firefox,
which uses the native find bar of Firefox. Therefore, this behavior is
inconsistent. Moreover, this is the only reason we have a dependency on
the find controller from the find bar.

Therefore, we remove this functionality altogether since, if we want
this, it should be implemented properly as tracked in issue mozilla#7468 and
not partially and inconsistently as in the current state.
@timvandermeij timvandermeij added viewer and removed ux labels Sep 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants