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

Win10 - update styling for dropdown menu in PDF forms #12216

Open
cfogelSV opened this issue Aug 17, 2020 · 1 comment
Open

Win10 - update styling for dropdown menu in PDF forms #12216

cfogelSV opened this issue Aug 17, 2020 · 1 comment

Comments

@cfogelSV
Copy link

@cfogelSV cfogelSV commented Aug 17, 2020

Affected versions

  • 81.0a1 (2020-08-16)

Affected platforms

  • Windows 10

Steps to reproduce
pdfjs.renderInteractiveForms - enabled

  1. Launch Firefox;
  2. Access a PDF form with dropdown menus;

Expected result

  • consistent icons across OSs;

Actual result

  • dropdown buttons look ugly;

Additional notes

  • S4 as suggested severity;
  • attached screenshot with the issue;

Screenshot 2020-08-17 at 10 12 16

@timvandermeij
Copy link
Contributor

@timvandermeij timvandermeij commented Aug 17, 2020

I originally tried to style all interactive form elements according to how Adobe Reader/Acrobat rendered them since users are likely to be more familiar with that, but I'm now thinking that it might be better to just let the browser do all rendering (according to OS styles) since that is more consistent with other browser form elements and avoids us having to do quite a bit of styling that can even differ when overridden in the PDF file itself. If we print/save, the appearance stream of the PDF will be used anyway, so for editing purposes users might indeed expect rendering like any other normal form.

Moreover, not all browsers even support styling all form elements, and the reference tests do not display selection marks in fields when checkbox/radio button styling is supported by the browser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants