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)
DevTools
Inspector
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
Component: Developer Tools: Animation Inspector → Developer Tools: Inspector
Updated•8 years ago
|
Severity: normal → enhancement
Priority: -- → P3
Comment 2•7 years ago
|
||
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]
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•