1) Highlight node A
2) Disable a css property
3) Highlight node B
4) Highlight node A
The disabled css property is gone. It should still be available.
Created attachment 572121 [details] [diff] [review]
Attached patch saves disabled properties in the inspector store, but the inspector store doesn't last the lifetime of the window, just as long as the store.
Still needs tests.
Would it make sense for the inspector store to have page-lifetime instead of inspector-is-open lifetime?
the InspectorStore's used to restore the Highlighter state on tab switch. I guess it currently has an "inspector-is-open lifetime". Changing that to page-lifetime might get a bit weird and might change some of the restore behavior.
That feels like it could add a fair bit of complexity to managing the InspectorStore. I'd have to see an implementation, I guess.
Comment on attachment 572121 [details] [diff] [review]
Created attachment 572379 [details] [diff] [review]
Adds a (somewhat grody) test.
Comment on attachment 572379 [details] [diff] [review]
looks good. As for the test, I've seen grodier.