# STEPS TO REPRODUCE: 1. inspect any page with lots of images (e.g. <https://apple.com>) 2. select the Images folder in the Sources Tab 3. drag an image over the collection view => "Create Local Override" drop zone shown
Created attachment 383321 [details] Patch
Comment on attachment 383321 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383321&action=review > Source/WebInspectorUI/UserInterface/Views/ResourceCollectionContentView.js:107 > + get contentViewConstructorOptions() > + { > + let contentViewConstructorOptions = super.contentViewConstructorOptions; > + if (this.representedObject.resourceType === WI.Resource.Type.Image) > + contentViewConstructorOptions.disableDropZone = true; > + return contentViewConstructorOptions; > + } Why not make collection content views always have `disableDropZone = true`. This only does it for a resource type of Image?
Comment on attachment 383321 [details] Patch Attachment 383321 [details] did not pass win-ews (win): Output: https://webkit-queues.webkit.org/results/13241720 New failing tests: imported/blink/fast/events/panScroll-crash.html
Created attachment 383340 [details] Archive of layout-test-results from ews215 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews215 Port: win-future Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment on attachment 383321 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=383321&action=review >> Source/WebInspectorUI/UserInterface/Views/ResourceCollectionContentView.js:107 >> + } > > Why not make collection content views always have `disableDropZone = true`. This only does it for a resource type of Image? I'd rather not pass it to constructors that aren't expecting it.
Comment on attachment 383321 [details] Patch Clearing flags on attachment: 383321 Committed r252378: <https://trac.webkit.org/changeset/252378>
All reviewed patches have been landed. Closing bug.
<rdar://problem/57135784>