| Summary: | Pressing Backspace or Delete within a contenteditable="true" summary removes all content | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | hi | ||||
| Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | NEW --- | ||||||
| Severity: | Normal | CC: | karlcow, megan_gardner, rniwa, thespyder, vivinkrishna99, webkit-bug-importer, wenson_hsieh | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | Safari 15 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| URL: | https://glitch.com/edit/#!/prosemirror-safari-backspace-bug | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 252223 | ||||||
| Attachments: |
|
||||||
|
Description
hi
2023-06-06 06:38:51 PDT
The problem tag name makes it quite difficult to search for a duplicate. Confirmed by various members of our team on: Version 16.3 (18614.4.6.1.6) Version 16.5 (18615.2.9.11.4) Version 16.6 (18615.3.5.11.2) STP Release 171 (Safari 16.4, WebKit 18616.1.15) I'll test the new version 17 when I get a chance to install the dev beta on my machine (but I assume that's basically the same as STP). This is holding up a TinyMCE release, we're just going to have to list it as a known issue Better replication case, using pure ContentEditable instead of prosemirror: https://codepen.io/tinymce/pen/wvQveOX?editors=1000 Is this a regression? Or the problem reproduces in older versions of Safari as well as newer versions? I'm not sure if we have any easily available older OS versions, I'll start grabbing some archives and see if I can track it down. bisect-builds is telling me `No archives found` for both mac-ventura configurations 🤔 We found an old mac running 10.15.7, Safari Version 15.5 (15613.2.7.1.9, 15613) and it does replicate there. So probably not a regression - we just hadn't tested our new feature well enough. Our engineers found an interesting workaround, wrapping the contents of the <summary> tag in a span before deleting with execCommmand: https://codepen.io/spocke78/pen/vYQYWoJ?editors=1010 So we're implementing that for our release: https://github.com/tinymce/tinymce/pull/8803 |