RESOLVED FIXED 60433
Web Inspector: REGRESSION metrics editing doesn't applies to the element 
https://bugs.webkit.org/show_bug.cgi?id=60433
Summary Web Inspector: REGRESSION metrics editing doesn't applies to the element
Ilya Tikhonovsky
Reported 2011-05-07 00:31:11 PDT
Metrics side bar panel.
Attachments
Add attachment proposed patch, testcase, etc.
Alexander Pavlov (apavlov)
Comment 1 2011-05-07 01:55:37 PDT
What does "does not apply" mean in this context? The element's inline style does not receive the altered box properties? The visual model is not guaranteed to change once these properties are applied (e.g. for position: static elements).
Ilya Tikhonovsky
Comment 2 2011-05-07 02:09:07 PDT
(In reply to comment #1) > What does "does not apply" mean in this context? The element's inline style does not receive the altered box properties? The visual model is not guaranteed to change once these properties are applied (e.g. for position: static elements). the entered value doesn't appear in the element's style property and there is no visual changes in the inspected page. I saw the next error in the inspector's inspector console: Uncaught TypeError: Object #<MouseEvent> has no method 'getPropertyValue' WebInspector.MetricsSidebarPane.editingCommitted DevTools.js:16538 boundDevTools.js:12 editingCommittedDevTools.js:3449 handleEditingResultDevTools.js:3468 keyDownEventListener the line: if (computedStyle.getPropertyValue("box-sizing") === "border-box" && (styleProperty === "width" || styleProperty === "height")) {
Alexander Pavlov (apavlov)
Comment 3 2011-05-07 04:50:24 PDT
Which revision of webkit is this?
Alexander Pavlov (apavlov)
Comment 4 2011-05-07 04:53:15 PDT
Just to clarify, I believe the issue was fixed in r84481 (a parameter added to the this.startEditing.bind() call)
Note You need to log in before you can comment on or make changes to this bug.