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.

Sign up for GitHub

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

Jump to bottom

[annotation] Text widget background color is not taken into account when printed #14928

Closed
calixteman opened this issue May 17, 2022 · 0 comments · Fixed by #14929
Closed

[annotation] Text widget background color is not taken into account when printed #14928

calixteman opened this issue May 17, 2022 · 0 comments · Fixed by #14929
Assignees
@calixteman
Labels
form-acroform

Comments

@calixteman
Copy link
Contributor

Attach (recommended) or Link to PDF file here:
cerfa_15615-01-2.pdf

Configuration:

  • Web browser and its version: Firefox nightly
  • Operating system and its version: Windows 11

Steps to reproduce the problem:

  1. File the 3rd field from the top-left on page 1 (i.e. Né(e) le):
    image

  2. print

The printed field is unreadable:
image

The annotation has a white background which should be used when the appearance is generated when printing.
@calixteman calixteman self-assigned this May 17, 2022
calixteman added a commit to calixteman/pdf.js that referenced this issue May 17, 2022
@calixteman
Display background when printing or saving a text widget (issue mozil…
18b8d1e 
…la#14928)
calixteman added a commit to calixteman/pdf.js that referenced this issue May 18, 2022
@calixteman
Display background when printing or saving a text widget (issue mozil…
09f17fd 
…la#14928)
@Snuffleupagus Snuffleupagus linked a pull request May 19, 2022 that will close this issue
Display background when printing or saving a text widget (issue #14928) #14929
Merged
calixteman added a commit to calixteman/pdf.js that referenced this issue May 19, 2022
@calixteman
Set the text fields font size based on their height
7f68aed 
- right now we're using the font size from the pdf itself but we use an other font
  in the annotation layer. So this size doesn't really make sense and leads to bad
  rendering (see pdf in mozilla#14928);
- use a sans-serif font for the fields containing text;
- text fields allow/disallow scrolling bars (see bit 24 in Ff entry), so use this
  value to hide/show scrollbars in annotation layer.
@calixteman calixteman mentioned this issue May 19, 2022
Merged
calixteman added a commit to calixteman/pdf.js that referenced this issue May 19, 2022
@calixteman
Display background when printing or saving a text widget (issue mozil…
30f2f9e 
…la#14928)
calixteman added a commit to calixteman/pdf.js that referenced this issue May 19, 2022
@calixteman
Display background when printing or saving a text widget (issue mozil…
60498c6 
…la#14928)
calixteman added a commit to calixteman/pdf.js that referenced this issue May 20, 2022
@calixteman
Set the text fields font size based on their height
d32dbb9 
- right now we're using the font size from the pdf itself but we use an other font
  in the annotation layer. So this size doesn't really make sense and leads to bad
  rendering (see pdf in mozilla#14928);
- use a sans-serif font for the fields containing text (fix issue mozilla#14736);
- remove useless padding in text-based fields (fix issue mozilla#14301);
- text fields allow/disallow scrolling bars (see bit 24 in Ff entry), so use this
  value to hide/show scrollbars in annotation layer.
Snuffleupagus added a commit that referenced this issue May 20, 2022
@Snuffleupagus
Merge pull request #14929 from calixteman/14928
5b67202 
Display background when printing or saving a text widget (issue #14928)
calixteman added a commit to calixteman/pdf.js that referenced this issue May 20, 2022
@calixteman
Set the text fields font size based on their height
c5a64db 
- right now we're using the font size from the pdf itself but we use an other font
  in the annotation layer. So this size doesn't really make sense and leads to bad
  rendering (see pdf in mozilla#14928);
- use a sans-serif font for the fields containing text (fix issue mozilla#14736);
- remove useless padding in text-based fields (fix issue mozilla#14301);
- text fields allow/disallow scrolling bars (see bit 24 in Ff entry), so use this
  value to hide/show scrollbars in annotation layer.
calixteman added a commit to calixteman/pdf.js that referenced this issue May 20, 2022
@calixteman
Set the text fields font size based on their height
52aa598 
- right now we're using the font size from the pdf itself but we use an other font
  in the annotation layer. So this size doesn't really make sense and leads to bad
  rendering (see pdf in mozilla#14928);
- use a sans-serif font for the fields containing text (fix issue mozilla#14736);
- remove useless padding in text-based fields (fix issue mozilla#14301);
- text fields allow/disallow scrolling bars (see bit 24 in Ff entry), so use this
  value to hide/show scrollbars in annotation layer.
calixteman added a commit to calixteman/pdf.js that referenced this issue May 28, 2022
@calixteman
Set the text fields font size based on their height
12dd4ac 
- right now we're using the font size from the pdf itself but we use an other font
  in the annotation layer. So this size doesn't really make sense and leads to bad
  rendering (see pdf in mozilla#14928);
- use a sans-serif font for the fields containing text (fix issue mozilla#14736);
- remove useless padding in text-based fields (fix issue mozilla#14301);
- text fields allow/disallow scrolling bars (see bit 24 in Ff entry), so use this
  value to hide/show scrollbars in annotation layer.
calixteman added a commit to calixteman/pdf.js that referenced this issue May 28, 2022
@calixteman
Set the text fields font size based on their height
9d82106 
- right now we're using the font size from the pdf itself but we use an other font
  in the annotation layer. So this size doesn't really make sense and leads to bad
  rendering (see pdf in mozilla#14928);
- use a sans-serif font for the fields containing text (fix issue mozilla#14736);
- remove useless padding in text-based fields (fix issue mozilla#14301);
- text fields allow/disallow scrolling bars (see bit 24 in Ff entry), so use this
  value to hide/show scrollbars in annotation layer.
bh213 pushed a commit to bh213/pdf.js that referenced this issue Jun 3, 2022
@calixteman @bh213
Display background when printing or saving a text widget (issue mozil…
1e0be05 
…la#14928)
bh213 pushed a commit to bh213/pdf.js that referenced this issue Jun 3, 2022
@calixteman @bh213
Set the text fields font size based on their height
a435a02 
- right now we're using the font size from the pdf itself but we use an other font
  in the annotation layer. So this size doesn't really make sense and leads to bad
  rendering (see pdf in mozilla#14928);
- use a sans-serif font for the fields containing text (fix issue mozilla#14736);
- remove useless padding in text-based fields (fix issue mozilla#14301);
- text fields allow/disallow scrolling bars (see bit 24 in Ff entry), so use this
  value to hide/show scrollbars in annotation layer.
rousek pushed a commit to signosoft/pdf.js that referenced this issue Aug 10, 2022
@calixteman @rousek
Display background when printing or saving a text widget (issue mozil…
c977314 
…la#14928)
rousek pushed a commit to signosoft/pdf.js that referenced this issue Aug 10, 2022
@calixteman @rousek
Set the text fields font size based on their height
807b2e5 
- right now we're using the font size from the pdf itself but we use an other font
  in the annotation layer. So this size doesn't really make sense and leads to bad
  rendering (see pdf in mozilla#14928);
- use a sans-serif font for the fields containing text (fix issue mozilla#14736);
- remove useless padding in text-based fields (fix issue mozilla#14301);
- text fields allow/disallow scrolling bars (see bit 24 in Ff entry), so use this
  value to hide/show scrollbars in annotation layer.
@ZeroXClem ZeroXClem mentioned this issue Aug 12, 2024
Closed
@earthywh earthywh mentioned this issue Sep 24, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@calixteman calixteman

Labels
form-acroform
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Display background when printing or saving a text widget (issue #14928) calixteman/pdf.js
1 participant
@calixteman