Bug 20216
| Summary: | execCommand RemoveFormat removes links | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Deepak Jindal <jindal> |
| Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | jparent, justin.garcia, rniwa, tony |
| Priority: | P2 | Keywords: | GoogleBug, NeedsReduction |
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | 43017 | ||
| Bug Blocks: | |||
Deepak Jindal
RemoveFormat shouldn't remove anchor tags.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Tony Chang
It looks like in WebKit, "removeformat" means remove all tags (lists, tables, anchors, everything). In Firefox and IE, it means remove text formatting, but keep structural formatting.
There's no equivalent command in TextEdit, so I'm not sure what's supposed to happen.
Julie, should we try to mach Firefox and IE here? There are plenty of test cases where the current behavior is expected (e.g., removing lists).
Here's a test case:
http://www.plexode.com/cgi-bin/eval3.py#ht=%3Cdiv%20id%3D%22e%22%20contenteditable%3E%3Cspan%20style%3D%22color%3Ared%22%3E%3Ca%20href%3D%22http%3A%2F%2Fwww.google.com%2F%22%3EHello%3C%2Fa%3E%20world%3C%2Fspan%3E%0A%3Cul%3E%0A%3Cli%3Eone%0A%3Cli%3Etwo%0A%3C%2Ful%3E%0A%3Cinput%3E%0A%3Ctextarea%3Ehi%3C%2Ftextarea%3E%0A%3Ctable%20border%3D1%3E%3Ctr%3E%3Ctd%3Efoo%3C%2Ftd%3E%3C%2Ftr%3E%3C%2Ftable%3E%0A%3C%2Fdiv%3E&ohh=1&ohj=1&jt=document.execCommand(%22removeformat%22%2C%20false%2C%20null)&ojh=0&ojj=0&ms=100&oth=0&otj=0&cex=1
Tony Chang
See also:
https://bugs.webkit.org/show_bug.cgi?id=13125
https://bugs.webkit.org/show_bug.cgi?id=17926
Ryosuke Niwa
This bug has been fixed since http://trac.webkit.org/changeset/70283. Since there are already regression tests for this bug, I close this bug.