mozilla / pdf.js

PDF Reader in JavaScript

https://mozilla.github.io/pdf.js/

Apache License 2.0

47.14k stars 9.82k forks source link

Improve how the wait-cursor is toggled when copying all text #18244

Closed Snuffleupagus closed 2 weeks ago

Snuffleupagus commented 2 weeks ago

Use a CSS rule to display the wait-cursor during copying. Since copying may take a little while in long documents, there's a theoretical risk that something else could change the cursor in the meantime and just resetting to the saved-cursor could thus be incorrect.
Remove the interruptCopyCondition listener with an AbortController, since that's slightly shorter code.

Snuffleupagus commented 2 weeks ago

/botio integrationtest

moz-tools-bot commented 2 weeks ago

From: Bot.io (Linux m4)

Received

Command cmd_integrationtest from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.241.84.105:8877/3753bc8508fcac7/output.txt

moz-tools-bot commented 2 weeks ago

From: Bot.io (Windows)

Received

Command cmd_integrationtest from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.193.163.58:8877/a45c49c6ebdb655/output.txt

calixteman commented 2 weeks ago

/botio-windows integrationtest

moz-tools-bot commented 2 weeks ago

From: Bot.io (Windows)

Received

Command cmd_integrationtest from @calixteman received. Current queue size: 0

Live output at: http://54.193.163.58:8877/f19b5910ce879e5/output.txt

moz-tools-bot commented 2 weeks ago

From: Bot.io (Linux m4)

Success

Full output at http://54.241.84.105:8877/3753bc8508fcac7/output.txt

Total script time: 7.61 mins

Integration Tests: Passed

moz-tools-bot commented 2 weeks ago

From: Bot.io (Windows)

Failed

Full output at http://54.193.163.58:8877/f19b5910ce879e5/output.txt

Total script time: 20.56 mins

Integration Tests: FAILED

timvandermeij commented 2 weeks ago

Thanks!