Summary: | Web Inspector: REGRESSION: Elements: pasting in the Styles sidebar adds a text node to the DOM tree | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, hi, inspector-bugzilla-changes, joepeck, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Devin Rousso
2019-07-08 14:10:27 PDT
(In reply to Devin Rousso from comment #0) > 1. copy "foo" Actually, you need to copy something that would instead be rendered as a CSS property (or multiple): "foo:". Created attachment 373664 [details]
Patch
Comment on attachment 373664 [details]
Patch
rs=me
Should this be the case for other events then? WI._copy?
Comment on attachment 373664 [details] Patch (In reply to Joseph Pecoraro from comment #3) > Should this be the case for other events then? WI._copy? I'd say yes, but in the interest of avoiding regressions, I'd rather not. Also, the default functionality of a "copy" won't modify the DOM like "paste" would, so there's less "risk" there of multiple things acting on the same "copy". Comment on attachment 373664 [details] Patch Clearing flags on attachment: 373664 Committed r247241: <https://trac.webkit.org/changeset/247241> All reviewed patches have been landed. Closing bug. |