Bug 22810
| Summary: | Safari can't remove strikethrough | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Deepak Jindal <deepak.jindal> |
| Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | jindal, jparent, ojan, rniwa |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | 20215, 23366, 23496 | ||
| Bug Blocks: | |||
Deepak Jindal
If you have html like this:
<strike>hello</strike>
Safari can't remove the strike through. It make it:
<strike style=""><span class="Apple-style-span" style="text-decoration: none;">hello</span></strike>
Which all browsers including Safari still show as strike through.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Alexey Proskuryakov
How are you trying to remove the strikethrough? It can certainly be done with DOM manipulations.
Deepak Jindal
My mistake. I should have mentioned that I am trying to remove it using execCommand strikeThrough.
Eric Seidel (no email)
This will be fixed when I fix bug 20215. I'll leave it as a separate bug until then.
Eric Seidel (no email)
I've got a partial patch locally. This is going to require some serious surgery to ApplyStyleCommand though. So this will have to wait until bug 23496 is reviewed and landed.
Ryosuke Niwa
Fixed in http://trac.webkit.org/changeset/46286.