Created attachment 322219 [details] [Animated GIF] Bug Steps: 1. Inspect any element with 2+ CSS properties. 2. Modify both property values one after another. 3. Move focus away from CSS editing field (CodeMirror). 4. Press Command-Z. Expected: Only the last change is reverted. Actual: All changes are reverted. Notes: I reproduced it in the shipped Safari and ToT WebKit.
<rdar://problem/34745031>
Created attachment 358553 [details] Patch
Created attachment 358556 [details] [Video] With patch applied
Comment on attachment 358553 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=358553&action=review r=me, wow I didn't realize how much I wanted this till I tried using it 😅 > Source/WebInspectorUI/UserInterface/Views/SpreadsheetStyleProperty.js:375 > + spreadsheetTextFieldDidBlur(textField, event, hasChanged) This is already implicitly past tense given the function name. I think `changed` is fine enough (it's also past tense).
Created attachment 358661 [details] Patch
Comment on attachment 358661 [details] Patch Clearing flags on attachment: 358661 Committed r239762: <https://trac.webkit.org/changeset/239762>
All reviewed patches have been landed. Closing bug.