Open Bug 1327742 Opened 8 years ago Updated 2 years ago

Inspector causes too many dom mutations when I edit attributes

Categories

(DevTools :: Inspector, enhancement, P3)

enhancement

Tracking

(Not tracked)

UNCONFIRMED

People

(Reporter: arni2033, Unassigned)

Details

(Keywords: parity-chrome)

Attachments

(1 file)

>>> My Info: Win7_64, Nightly 49, 32bit, ID 20160526082509 STR_1: (replacing attributes manually) 1. Open attached "testcase 1" 2. Open devtools -> inspector, open split console (Escape key) 3. Doubleclick on attribute "xyz", replace text `"asdf"` with `"123"` (w/o outer quotes), press Enter AR: 2 mutations: "asdf" -> null, null -> "123". "null" means the attribute is deleted. ER: (X) 1 mutation: "asdf" -> "123". As if I executed in console `O.setAttribute('xyz',"123");` STR_2: (Ctrl+Z vs Ctrl+Y) 1. Open attached "testcase 1" 2. Open devtools -> inspector, open split console (Escape key) 3. Doubleclick on attribute "xyz", replace text `"asdf"` with `"123"`, press Enter 4. Doubleclick on attribute "xyz", replace text `"123"` with `"asdf"`, press Enter 5. Press Ctrl+Z twice 6. Press Ctrl+Y twice AR: Step 5 - mutations: First "asdf" -> "123", "123" -> "123". Then "123" -> "asdf", "asdf" -> "asdf" Step 6 - mutations: First "asdf" -> null, null -> "123". Then "123" -> null, null -> "asdf" ER: Both (Y) and (Z), or at least one of them (Y) Ctrl+Z and Ctrl+Y should change attribute the same way (Z) None of them should delete attribute or set the same value twice STR_3: (setting the same value) 1. Open attached "testcase 1" 2. Open devtools -> inspector, open split console (Escape key) 3. Doubleclick on attribute "xyz", replace text `xyz="asdf"` with `xyz="123"`, press Enter 4. Doubleclick on attribute "xyz", replace text `xyz="123"` with `xyz = "123"`, press Enter 5. Press Ctrl+Z AR: Step 4 - 2 mutations: "123" -> null, null -> "123" Step 5 - 2 mutations: "123" -> null, null -> "123" ER: Both (U) and (V), or at least one of them (U) Step 4 - no mutations, as if I replaced text `xyz="123"` with ` xyz="123" ` (V) Step 5 - Ctrl+Z should restore old value "asdf", since I haven't in fact modified the value
No longer blocks: 1277113
Component: Untriaged → Developer Tools: Animation Inspector
Component: Developer Tools: Animation Inspector → Developer Tools: Inspector
Severity: normal → enhancement
Priority: -- → P3
Mass bug change to replace various 'parity' whiteboard flags with the new canonical keywords. (See bug 1443764 comment 13.)
Keywords: parity-chrome
Whiteboard: [parity-Chrome]
Product: Firefox → DevTools
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: