| Summary: | applyInlineStyleToPushDown may apply style with CSS when pushing down styles even if styleWithCSS is false | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | HTML Editing | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW --- | ||
| Severity: | Enhancement | CC: | ahmad.saleem792, bradavogel, darin, enrica, eric, tony |
| Priority: | P3 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | 55338, 55349, 55452 | ||
| Bug Blocks: | |||
My company's product - Inkling Habitat (inkling.com/habitat) - is blocked on this and would love to have it fixed! This FIXME is still present: https://github.com/WebKit/WebKit/blob/b48ba57fb5ed3acdd4f752c85fdf4f0f9fcd8e8d/Source/WebCore/editing/ApplyStyleCommand.cpp#L1022 Although all dependent bugs have been fixed. |
Currently, ApplyStyleCommand::applyInlineStyleToPushDown sets style attribute to push down styles if the node has block flow style or has children. The following fix me corresponds to this bug: // FIXME: applyInlineStyleToRange should be used here instead. if ((node->renderer()->isBlockFlow() || node->childNodeCount()) && node->isHTMLElement()) { setNodeAttribute(toHTMLElement(node), styleAttr, newInlineStyle->style()->cssText()); return; }