* SUMMARY Extra whitespace left behind in styles sidebar after clicking in and out. * STEPS TO REPRODUCE 1. Inspect <body> on <http://daringfireball.net> 2. Show Styles > Rules sidebar 3. Click in the first CSSStyleDeclarationTextEditor at the end of the line-height line => newline added to add a new property 4. Click out of the editor => expected newline to be removed, it remains * NOTES - Regressed with r188427 - bug 145679
<rdar://problem/22711632>
Just so I understand, we want to have it such that new CSS styles that are added follow the same indentation of existing CSS, but remove any leading/trailing whitespace and extra newlines in the middle. So it would format body { color: red; background-color: black; } into the style declaration with the following formatting: body { color: red; background-color: black; }
(In reply to comment #2) > Just so I understand, we want to have it such that new CSS styles that are > added follow the same indentation of existing CSS, but remove any > leading/trailing whitespace and extra newlines in the middle. So it would > format > > body { > color: red; > > background-color: black; > > } > > into the style declaration with the following formatting: > > body { > color: red; > background-color: black; > } Yep! In the sidebar I don't expect extra newlines in the middle if the editor is not focused. Preserving newlines in the actual StyleSheet's text would be nice, but I don't think we do that now so its fine if we reformat that a little.
Created attachment 261365 [details] Patch
Comment on attachment 261365 [details] Patch Clearing flags on attachment: 261365 Committed r189909: <http://trac.webkit.org/changeset/189909>
All reviewed patches have been landed. Closing bug.