RESOLVED DUPLICATE of bug 78881 44300
Web Inspector: elements css: single click to edit properties and values 
https://bugs.webkit.org/show_bug.cgi?id=44300
Summary Web Inspector: elements css: single click to edit properties and values
Adam
Reported 2010-08-19 15:42:20 PDT
It seems it would be much simpler to edit the properties and values with a single click rather than a double click. Since while editing a page, those values could be changed a whole lot over and over again, double clicking becomes quite the pain. The selector should remain double clicked to edit since it isn't edited nearly as much. If there it's a matter of all one or the other, single click for all would be preferable.
Attachments
Patch (10.17 KB, patch)
2011-11-17 08:28 PST, Alexander Pavlov (apavlov) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Joseph Pecoraro
Comment 1 2010-08-19 16:04:18 PDT
Thanks for filing the bug. We've talked about this in the past. In fact, there is another bug that is very similar: <http://webkit.org/b/36965> Web Inspector: start editing DOM and styles on click-and-pause.
Pavel Feldman
Comment 2 2010-08-19 23:01:55 PDT
I think Tim implemented it as double-click years ago because he was avoiding non-standard behavior. The idea was that it was either a text area with single click editing or a number of double-click actions. Later, I tried implementing click-and-pause, but it did not really improve the experience. Now the interesting part. We are currently working on changing the way we edit styles entirely (model-wise, not ui-wise). We are improving css parser so that it does not drop unrecognized properties (-moz, ones with syntax errors and such). As a result, we will be editing pretty much formatted original source code, not today's pure webcore model data. As a part of the change, we are likely to start committing entire style bodies on changes to style properties (instead of today's individual properties commits). This brings us much closer to editing style in a single text area.
Alexander Pavlov (apavlov)
Comment 3 2011-11-11 06:27:41 PST
*** Bug 62715 has been marked as a duplicate of this bug. ***
Alexander Pavlov (apavlov)
Comment 4 2011-11-17 08:28:28 PST
Created attachment 115597 [details] Patch
Alexander Pavlov (apavlov)
Comment 5 2011-11-17 08:38:21 PST
This is a strawman proposal. Please apply to see it in action.
Timothy Hatcher
Comment 6 2011-11-19 12:52:04 PST
I agree with Pavel. If the whole rule feels like a editable text area, then I think single click is fine.
Pavel Feldman
Comment 7 2011-11-29 01:44:33 PST
Comment on attachment 115597 [details] Patch Clearing r? since this change is not about requesting the code review.
Alexander Pavlov (apavlov)
Comment 8 2012-02-20 03:47:14 PST
Fixed in bug 78881. *** This bug has been marked as a duplicate of bug 78881 ***
Note You need to log in before you can comment on or make changes to this bug.