Summary: | Web Inspector: Add proper formatting for editing styles in the Visual Panel | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> | ||||||
Component: | Web Inspector | Assignee: | Devin Rousso <hi> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 147563 | ||||||||
Attachments: |
|
Description
Devin Rousso
2015-08-18 15:46:09 PDT
Created attachment 259308 [details]
Patch
Comment on attachment 259308 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=259308&action=review > Source/WebInspectorUI/UserInterface/Views/VisualStylePropertyCombiner.js:178 > + _generateFormattedTextForNewProperty(styleText, propertyValue) { No way to sure this with the other version below? Comment on attachment 259308 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=259308&action=review >> Source/WebInspectorUI/UserInterface/Views/VisualStylePropertyCombiner.js:178 >> + _generateFormattedTextForNewProperty(styleText, propertyValue) { > > No way to sure this with the other version below? Would making _generateFormattedTextForNewProperty a static function inside VisualStylePropertyEditor work? If it was not prefixed with an underscore, sure. Created attachment 259326 [details]
Patch
Comment on attachment 259326 [details] Patch Clearing flags on attachment: 259326 Committed r188630: <http://trac.webkit.org/changeset/188630> All reviewed patches have been landed. Closing bug. |