1804252 - Use `move` cursor for added text/drawing

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, enhancement)

Description

[Nicolas Chevobbe [:nchevobbe][PTO - back on October 21]]

Setting the cursor to move would make it more obvious that those elements can be moved

Comment 1

[Marco Castelluccio [:marco]]

Were you planning on helping us with this?

Comment 2

[Nicolas Chevobbe [:nchevobbe][PTO - back on October 21]]

Attachment: Bug 1804252 - Use cursor:move for draggable text/drawing in pdfjs. r=#pdfjs-reviewers.

Comment 3

[Nicolas Chevobbe [:nchevobbe][PTO - back on October 21]]

(In reply to Marco Castelluccio [:marco] from comment #1)

Were you planning on helping us with this?

I can try

Comment 4

[Calixte Denizet (:calixte)]

Would you mind to make a PR: https://github.com/mozilla/pdf.js/ ?
The files we've in m-c are generated and your changes will be overwritten in the next pdf.js release.

Comment 5

[BMO Github Automation]

Attachment: [mozilla/pdf.js] Use cursor:move for draggable texts and drawings (Bug 1804252) (#16192)

Comment 6

[Nicolas Chevobbe [:nchevobbe][PTO - back on October 21]]

(In reply to Calixte Denizet (:calixte) from comment #4)

Would you mind to make a PR: https://github.com/mozilla/pdf.js/ ?
The files we've in m-c are generated and your changes will be overwritten in the next pdf.js release.

sure: https://github.com/mozilla/pdf.js/pull/16192