Bug 178990
| Summary: | Web Inspector: Styles: Editing style attribute prepends line-break | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Nikita Vasilyev <nvasilyev> |
| Component: | Web Inspector | Assignee: | Nikita Vasilyev <nvasilyev> |
| Status: | ASSIGNED | ||
| Severity: | Normal | CC: | inspector-bugzilla-changes, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=178835 | ||
Nikita Vasilyev
Editing the style attribute via the sidebar causes a line-break character to be prepended to the style attribute text in the DOM tree.
This affects both the old and the new styles sidebar.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/35242049>
Nikita Vasilyev
This was introduced in Bug 157869 - REGRESSION (r188730): Web Inspector: Warning icons incorrectly positioned in CSS Rules sidebar.
https://bugs.webkit.org/show_bug.cgi?id=157869#c10:
> This patch is essentially fixes the bug without making too much changes in
> how we handle spacing and indentation in CSS.
> In the future, we shouldn't append or prepend "\n" to inline styles at all.
> This would require more changes to be made, which can
> potentially break things.
Now is a good time to make these changes and add more tests.