Convert examples/text-only/pdf2svg.js to await/async

[adenicole]

This is my first pull request.

[calixteman]

You're adding the function pageLoaded but it isn't called anywhere so it's useless.

[Snuffleupagus]

You're adding the function pageLoaded but it isn't called anywhere so it's useless.

Isn't the problem actually that the patch adds a new function, rather than simply replacing the old one (as intended)?

Furthermore, was this successfully tested locally?
There's also a number of linting failures, please remember to run gulp lint and fix all the reported problems.
Finally, please squash the commits; see https://github.com/mozilla/pdf.js/wiki/Squashing-Commits

[calixteman]

You're adding the function pageLoaded but it isn't called anywhere so it's useless.

Isn't the problem actually that the patch adds a new function, rather than simply replacing the old one (as intended)?

Oh! I've been confused because of the github UI.

Furthermore, was this successfully tested locally? There's also a number of linting failures, please remember to run gulp lint and fix all the reported problems. Finally, please squash the commits; see https://github.com/mozilla/pdf.js/wiki/Squashing-Commits

[Snuffleupagus]

As mentioned in #14141 (comment):

• Please confirm that you've successfully tested this locally.
• Please squash the commits, since we don't allow either fixup (or merge) commits to land; see https://github.com/mozilla/pdf.js/wiki/Squashing-Commits

[adenicole]

Hi @Snuffleupagus @calixteman,

I am really sorry for all the trouble. 🙏 This happens to be my first pull request and it being an open-source project got me all nervous and excited. 😅

I have now locally tested and fixed lint errors and squashed all commits into one. Thank you for your patience. 😊 I look forward to learning more! 💪 💃

[Snuffleupagus]

The second sentence of the commit message, or at least the "and corrected lint errors"-part, doesn't seem appropriate here since it suggests that there were lint errors in the old code which isn't actually the case.

[adenicole]

The second sentence of the commit message, or at least the "and corrected lint errors"-part, doesn't seem appropriate here since it suggests that there were lint errors in the old code which isn't actually the case.

Oh, my bad! Fixed the commit message. 🙂

[Snuffleupagus]

For future reference: As outlined in https://github.com/mozilla/pdf.js/wiki/Contributing, you want to use feature branches rather than working directly against the "master" branch (since that makes working on more than thing at a time really difficult and causes e.g. rebasing to become more difficult).

r=me, thank you.