Page MenuHomePhabricator

Paste check evaluates individual annotation-ranges by length, not the total size of the paste
Open, Needs TriagePublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Edit with VE with paste check enabled
  • Paste in some content that will trigger paste check that's overall long enough to trigger paste check, but that includes a paragraph/heading that's less than 50 characters long

What happens?:

The part of the content that's less than 50 characters long is excluded from the check, although the parts around it are triggered:

CleanShot 2026-01-15 at 10.31.50@2x.png (598×2 px, 244 KB)

What should have happened instead?:

Probably the overall length of the entire pasted content should be what the length requirement is evaluated against. This would, however, have the side effect that a user pasting something and then editing it post-paste in ways that break up the content would no longer resolve the check.

Event Timeline

Potential resolutions:

  1. the total length of the pasted content (from a given paste) is what matters; this will make it so that the user splitting that content up themself so that there are no longer any runs of 50+ characters will no longer stop the paste check from being shown
  2. a paste with *any* remaining range of content with sufficient length should trigger every remaining bit of that content to be shown for the check
  3. adjacent pasted content between paragraphs (/headings/etc) should be merged for purposes of content-length calculation
  4. any-of-the-above but all content in-between pasted ranges is counted as part of the paste for removal/highlighting purposes; this would clean up any edits that the user had made to the content
  5. this is fine actually

Note that any resolution which results in the paste check offering to remove content that's heavily intermingled with non-paste content is going make the "remove it" option have undesirable behavior. This is most likely with options 1 or 2.