Bug 51476 – Web Inspector: Can't backspace from CSS value to CSS key field

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Nico Weber]

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.

[Joseph Pecoraro]

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

[Alexander Pavlov (apavlov)]

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?

[Joseph Pecoraro]

Comment on attachment 77324 [details]
[PATCH] Suggested solution

Good solution. Thanks! r=me

[WebKit Commit Bot]

Comment on attachment 77324 [details]
[PATCH] Suggested solution

Clearing flags on attachment: 77324

Committed r74562: <http://trac.webkit.org/changeset/74562>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[WebKit Review Bot]

http://trac.webkit.org/changeset/74562 might have broken SnowLeopard Intel Release (Tests)