Title: [239762] trunk/Source/WebInspectorUI
- Revision
- 239762
- Author
- nvasil...@apple.com
- Date
- 2019-01-08 19:29:18 -0800 (Tue, 08 Jan 2019)
Log Message
Web Inspector: Styles: Undo reverts all changes at once
https://bugs.webkit.org/show_bug.cgi?id=177676
<rdar://problem/34745031>
Reviewed by Devin Rousso.
Command-Z used to revert all changes at once because Web Inspector never
set any history checkpoints in the style editor.
* UserInterface/Views/SpreadsheetStyleProperty.js:
(WI.SpreadsheetStyleProperty.prototype.spreadsheetTextFieldDidBlur):
* UserInterface/Views/SpreadsheetTextField.js:
(WI.SpreadsheetTextField.prototype._handleBlur):
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/ChangeLog (239761 => 239762)
--- trunk/Source/WebInspectorUI/ChangeLog 2019-01-09 02:56:15 UTC (rev 239761)
+++ trunk/Source/WebInspectorUI/ChangeLog 2019-01-09 03:29:18 UTC (rev 239762)
@@ -1,5 +1,21 @@
2019-01-08 Nikita Vasilyev <nvasil...@apple.com>
+ Web Inspector: Styles: Undo reverts all changes at once
+ https://bugs.webkit.org/show_bug.cgi?id=177676
+ <rdar://problem/34745031>
+
+ Reviewed by Devin Rousso.
+
+ Command-Z used to revert all changes at once because Web Inspector never
+ set any history checkpoints in the style editor.
+
+ * UserInterface/Views/SpreadsheetStyleProperty.js:
+ (WI.SpreadsheetStyleProperty.prototype.spreadsheetTextFieldDidBlur):
+ * UserInterface/Views/SpreadsheetTextField.js:
+ (WI.SpreadsheetTextField.prototype._handleBlur):
+
+2019-01-08 Nikita Vasilyev <nvasil...@apple.com>
+
Web Inspector: Use prefers-color-scheme instead of prefers-dark-interface
https://bugs.webkit.org/show_bug.cgi?id=193265
Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetStyleProperty.js (239761 => 239762)
--- trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetStyleProperty.js 2019-01-09 02:56:15 UTC (rev 239761)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetStyleProperty.js 2019-01-09 03:29:18 UTC (rev 239762)
@@ -372,7 +372,7 @@
this.remove();
}
- spreadsheetTextFieldDidBlur(textField, event)
+ spreadsheetTextFieldDidBlur(textField, event, changed)
{
let focusedOutsideThisProperty = event.relatedTarget !== this._nameElement && event.relatedTarget !== this._valueElement;
if (focusedOutsideThisProperty && (!this._nameTextField.value.trim() || !this._valueTextField.value.trim())) {
@@ -385,6 +385,9 @@
if (typeof this._delegate.spreadsheetStylePropertyFocusMoved === "function")
this._delegate.spreadsheetStylePropertyFocusMoved(this, {direction: null});
+
+ if (changed && window.DOMAgent)
+ DOMAgent.markUndoableState();
}
spreadsheetTextFieldDidBackspace(textField)
Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetTextField.js (239761 => 239762)
--- trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetTextField.js 2019-01-09 02:56:15 UTC (rev 239761)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetTextField.js 2019-01-09 03:29:18 UTC (rev 239762)
@@ -220,7 +220,8 @@
this._applyCompletionHint();
this.discardCompletion();
- this._delegate.spreadsheetTextFieldDidBlur(this, event);
+ let changed = this._valueBeforeEditing !== this.value;
+ this._delegate.spreadsheetTextFieldDidBlur(this, event, changed);
this.stopEditing();
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes