Summary: | REGRESSION(r188138): Web Inspector: cannot edit next/previous DOM attribute by using tab/shift-tab | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | BJ Burg <bburg> | ||||
Component: | Web Inspector | Assignee: | Matt Baker <mattbaker> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, hi, inspector-bugzilla-changes, mattbaker, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
BJ Burg
2017-02-10 09:42:21 PST
Regressed way back in https://trac.webkit.org/changeset/188138. Tabbing to the next attribute *does* work, as long as the current attribute text changes: --- a/Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js +++ b/Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js @@ -874,6 +875,9 @@ WebInspector.DOMTreeElement = class DOMTreeElement extends WebInspector.TreeElem _attributeEditingCommitted(element, newText, oldText, attributeName, moveDirection) { + if (newText === oldText) + return; + this._editing = false; var treeOutline = this.treeOutline; Created attachment 301249 [details]
Patch
Comment on attachment 301249 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=301249&action=review > Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js:963 > + if (!moveDirection && newText === oldText) What about all of the other functions with moveDirection? _tagNameEditingCommitted and _attributeNumberEditingCommitted? Comment on attachment 301249 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=301249&action=review >> Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js:963 >> + if (!moveDirection && newText === oldText) > > What about all of the other functions with moveDirection? _tagNameEditingCommitted and _attributeNumberEditingCommitted? I didn't notice these. However, it looks like _tagNameEditingCommitted handles moveDirection in `cancel()`, and _attributeNumberEditingCommitted ignores the parameter. Comment on attachment 301249 [details] Patch Clearing flags on attachment: 301249 Committed r212236: <http://trac.webkit.org/changeset/212236> All reviewed patches have been landed. Closing bug. |