1844576 - pdf-form rendering issue - some textfields seem to be rendered twice

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, defect, P1)

Description

[ThILLM_MG]

Attachment: Bewertungsraster_ModerneFremdsprache.pdf

Steps to reproduce:

(1) open attached pdf-form
(2) scroll to last page
(3) activate one of the checkboxes

Actual results:

Some of the textfields get rendered twice. One rendering process seems to be normal, the second one displays additional characters. Since the results of the two rendering processes overlap, the text is no longer readable.

Expected results:

Text shoud be rendered only once, without additional characters added

Comment 1

[ThILLM_MG]

Attachment: Screenshot showing the rendering-issue

This screenshot shows the issue, that the textfields seem to be rendered twice - one time normal, second time with additional characters.

Comment 2

[ThILLM_MG]

The issue seems to be introduced with Firefox 115. In Firefox 114 the file gets (more or less) rendered as expected (some text witch should have been bold, was rendered regular in normal view, but rendered correct in print-preview and was also printed correctly).

Comment 3

[ThILLM_MG]

Since the users of this from are expected to use this form in browser only, display of numbers depends on functionality from Intl namespace.

Comment 4

[ThILLM_MG]

Firefox v116.0.0_Beta7 and v117.0.0_Alpha1 are also affected by this issue.

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Firefox::PDF Viewer' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 6

[Marco Castelluccio [:marco]]

I wonder if this is the same as bug 1842431.

Could you try using mozregression to figure out exactly what caused this issue?

Comment 7

[ThILLM_MG]

Might in fact be the same bug, as all affected fields are readonly.

Last good revision: 88b584b11d1f5f2f55b652594b3e4d3fade80f6d
First bad revision: b3cac12e312d2870e027fe953f5e325c486d4643

Comment 8

[Marco Castelluccio [:marco]]

Thanks for the mozregression run, that's really helpful.

Comment 9

[BugBot [:suhaib / :marco/ :calixte]]

:calixte, since you are the author of the regressor, bug 1832018, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Comment 10

[Calixte Denizet (:calixte)]

In the annotation_layer we rely on data.hasOwnCanvas to decide to show/hide some readonly fields.
But sometimes we don't create the canvas (when we should), hence this bug:
https://github.com/mozilla/pdf.js/blob/7220b93911686f698e38495140bdaf6d53a2c4ae/src/core/annotation.js#L1880

So instead having this false value we must the one from data.hasOwnCanvas.

Comment 11

[BMO Github Automation]

Attachment: [mozilla/pdf.js] Text annotations must use their own canvas when their appearance is generated (bug 1844576) (#16720)

Comment 12

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:calixte, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
• If no, please set status-firefox116 to wontfix.

For more information, please visit BugBot documentation.

Comment 13

[Calixte Denizet (:calixte)]

Done in bug 1845506.