search

mozilla / pdf.js

PDF Reader in JavaScript
https://mozilla.github.io/pdf.js/
Apache License 2.0
47.09k stars 9.81k forks source link

Use the new formatted issue templates #18308

Closed nicolo-ribaudo closed 1 week ago

nicolo-ribaudo commented 1 week ago

You can test this at https://github.com/nicolo-ribaudo/pdf.js/issues, and https://github.com/nicolo-ribaudo/pdf.js/issues/1 is an example of the generated issue. It's unfortunate that all the small questions (such as "Operating system and its version") get their own header, but hopefully it will help with getting all the necessary info filled in properly.

nicolo-ribaudo commented 1 week ago

It seems like there is no way to have a yes/no required option whose default value is "unselected". I made the questions about "does this happen in the latest version" and "is this relevant to Firefox" default to no.

timvandermeij commented 1 week ago

I have requested a review from myself and @calixteman as additional sign-off on this. I'll try to have this reviewed tomorrow.

marco-c commented 1 week ago

LGTM! Question: should we have "Is the bug present in the latest PDF.js version?" be "Yes" by default since that should be the most common answer?

nicolo-ribaudo commented 1 week ago

Default updated 👍

timvandermeij commented 1 week ago

Now that all four approvals are in and the discussed feedback is addressed, let's do this. This should increase the quality of the bug reports, and if need be we can always tweak it further. Thank you for doing this!