Closed Bug 1640217 Opened 4 years ago Closed 1 year ago

Find will not find text if entered with diacritics ("nikud") in Hebrew for PDF viewer

Categories

(Firefox :: PDF Viewer, defect, P1)

defect

Tracking

()

RESOLVED FIXED
112 Branch

People

(Reporter: eyalgruss, Assigned: calixte)

References

Details

(Keywords: intl, Whiteboard: [pdfjs-ux][pdfjs-text-search])

Attachments

(1 file)

trying to find הספר with match diacritics turned off does not find הַסֵּפֶר in the following pdf:

https://www.machonso.org/uploads/images/%D7%95%D7%95%D7%9C%D7%A3.pdf

Depends on: 202251, 1611568
Keywords: intl

The severity field is not set for this bug.
:bdahl, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(bdahl)
Severity: -- → S3
Flags: needinfo?(bdahl)
Whiteboard: [pdfjs-c-ux][pdfjs-d-text-search]

Any update on this? I've the same issue with Greek text. I now have to use Chrome just for this... :(

Assignee: nobody → cdenizet

Chiming in with my experience with PDF, search and diacritics which isn't exactly the same as what OP describes but surely related. In my case it's Spanish language so it's for á é í ó & ú. Basically search in this case is always diacritic-sensitive regardless of whether "Match Diacritic" is set on or off.

Example PDF: https://es.wikipedia.org/api/rest_v1/page/pdf/Ejemplo

Searching for "retórica" does find matches (contrary to OP's Hebrew experience if I understand correctly) but searching for "retorica" never matches even when it should (with Match Diacritic = off).

I proposed a patch upstream:
https://github.com/mozilla/pdf.js/pull/13261
:eyaler, I tested with your pdf and it seems ("seem" because I don't read hebrew) to work fine.

Status: NEW → ASSIGNED

eyal gruss, can you verify that the bug is solved in a current Firefox release?

Flags: needinfo?(eyalgruss)

The patch I worked on is still a wip and so has never landed.
Consequently, there are no chance this bug is fixed.

Flags: needinfo?(eyalgruss)
Whiteboard: [pdfjs-c-ux][pdfjs-d-text-search] → [pdfjs-ux][pdfjs-text-search]
Priority: -- → P1
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: