ui-list paste plaintext-only

[ollimeier]

Fixes #2043

[justvanrossum]

Awesome! I didn't know about "plaintext-only".

[justvanrossum]

Pls wait with merge, though, I found one thing, just a sec.

[justvanrossum]

fontra/src/fontra/client/core/utils.js

Line 314 in 7157665

if (element.contentEditable === "true") {

This may need to be changed as well. Maybe it is as simple as changing it to element.contentEditable !== "false".

[ollimeier]

fontra/src/fontra/client/core/utils.js

Line 314 in 7157665

if (element.contentEditable === "true") {

This may need to be changed as well. Maybe it is as simple as changing it to element.contentEditable !== "false".

element.contentEditable can also be inherit (which would also be !== "false") why I think it's probably better to use element.contentEditable === "plaintext-only" ||.
I would like to write a unittest for that, but I don't know how, because isActiveElementTypeable uses document.activeElement and I don't know how to mock up such an element for a unittest.

[justvanrossum]

Ah, actually, we should use isContentEditable: https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/isContentEditable

[ollimeier]

Ah, actually, we should use isContentEditable: https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/isContentEditable

Thanks for the hint. That's much better. Please see the latest commit.