I like the change that made CSS keys and values to different text fields. However, it made it hard to delete new rules. Repro: 1. Double-click "Element style" 2. "color" : "red" but don't hit enter 3. Change your mind, hit backspace Expected: I'm able to backspace all the way to the left, deleting everything i wrote Actual: Only "red" gets deleted. The caret should move back the the key field if backspace is pressed and the caret is on the left edge of the value field.
Thanks for the report Nico. I think many others would expect the opposite, that backspacing would be limited to just the value. However, you have a very good point. If the user actually types out the ":" and is jumped editing the property to value, then they would also expect that behavior going backwards. The user flow I would have used didn't work as I expected: <http://webkit.org/b/51480> Web Inspector: Tabbing from new rule with empty value should create a new property
Created attachment 77324 [details] [PATCH] Suggested solution I believe this behavior is only useful for new properties, as blurring a changed value of an existing property results in the property being committed (effectively removing the property if the value is empty), which may be far from what the user could expect. Does that sound reasonable?
Comment on attachment 77324 [details] [PATCH] Suggested solution Good solution. Thanks! r=me
Comment on attachment 77324 [details] [PATCH] Suggested solution Clearing flags on attachment: 77324 Committed r74562: <http://trac.webkit.org/changeset/74562>
All reviewed patches have been landed. Closing bug.
http://trac.webkit.org/changeset/74562 might have broken SnowLeopard Intel Release (Tests)