129096 2014-02-20 04:13:43 -0800 Web Inspector: gradients in the Styles sidebar aren't editable using the dedicated gradients editor 2017-02-13 00:46:31 -0800 1 1 1 Unclassified WebKit Web Inspector 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 119686 1 graouts graouts bburg graouts inspector-bugzilla-changes webkit-bug-importer oldest_to_newest 982783 0 graouts 2014-02-20 04:13:43 -0800 The patch for https://bugs.webkit.org/show_bug.cgi?id=119686 will provide a new gradients editor but it only works as part of SourceCodeTextEditor and not CSSStyleDeclarationTextEditor. We should add support for such editors in CSSStyleDeclarationTextEditor as well and possibly adopt the CodeMirrorColorEditingController for editing colours as well so that SourceCodeTextEditor and CSSStyleDeclarationTextEditor are more alike. 982784 1 webkit-bug-importer 2014-02-20 04:13:54 -0800 <rdar://problem/16120224> 983208 2 graouts 2014-02-21 08:14:34 -0800 I wonder if we could refactor SourceCodeTextEditor and CSSStyleDeclarationTextEditor to share some CodeMirrorTokenTrackingController and CodeMirrorEditingController logic to implement the basic behaviour implemented in SourceCodeTextEditor across both editors. I think basically we'd have a new helper object which we'd notify when the editor hides, has its content populated or changed, and have basic handlers for tokenTrackingControllerNewHighlightCandidate and tokenTrackingControllerMouseOutOfHoveredMarker that do the work. Then the CodeMirrorEditingController delegates would be implemented by this helper object as well and the editor classes would have little knowledge of CodeMirrorEditingController. 983936 3 timothy 2014-02-24 09:56:50 -0800 A unified controller that SourceCodeTextEditor and CSSStyleDeclarationTextEditor uses to share the editing logic seems good to me. One thing we need to consider is how to support changing the color format if the swatch isn't there. This makes having a menu in HoverMenu for colors appealing, since you could have menu items for the other formats (with the current one having a checkmark). 983945 4 graouts 2014-02-24 10:44:43 -0800 (In reply to comment #3) > One thing we need to consider is how to support changing the color format if the swatch isn't there. This makes having a menu in HoverMenu for colors appealing, since you could have menu items for the other formats (with the current one having a checkmark). Good point, I'll look into implementing that. 1275405 5 bburg 2017-02-10 10:53:46 -0800 I am pretty sure that this is no longer an issue, right? We can edit gradients in the style sidebar. 1276130 6 graouts 2017-02-13 00:46:31 -0800 Yeah, I can confirm that this has been fixed. Very nice!