Bug 38672
Summary: | Web Inspector: reorganize front-end directory into subdirectories | ||
---|---|---|---|
Product: | WebKit | Reporter: | Patrick Mueller <pmuellr> |
Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | bweinstein, joepeck, keishi, pfeldman, pmuellr, rik, yurys |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | All | ||
OS: | All |
Patrick Mueller
There are at this time 100 files in the WebCore/inspector/front-end directory, and 153 files in front-end/images.
Perhaps it's time to reorganize this into subdirectories.
Initial thought is a directory per panel, and another directory for everything else.
Perhaps the panel directories could all live under front-end/panels/ or something.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Pavel Feldman
I am not that concerned with the files being in a single folder. Making us more modular is a much more appealing goal to my taste. Adding explicit dependencies between classes and removing circular dependencies that is.
Timothy Hatcher
Multiple directories isn't that critical. Release builds (for Safari at least) merge into one JS file for speed.
Brian Burg
Closing as invalid, as this bug pertains to the old inspector UI and/or its tests.
Please file a new bug (https://www.webkit.org/new-inspector-bug) if the bug/feature/issue is still relevant to WebKit trunk.