SiteTableViewController is still coupled to HomePanelDelegate, which is a bit strange considering SiteTableViewController doesn't even use the delegate it owns. I think we can clean this up by: * Moving SiteTableViewController out of Home/ and into Widgets/ * Removing the unused HomePanelDelegate from SiteTableViewController * Making home panels subclass SiteTableViewController directly, implement HomePanel directly, and own their own delegates
Created attachment 8569449 [details] [review] Pull request
Comment on attachment 8569449 [details] [review] Pull request I don't really understand the desire to keep the Search controller delegate separate from the home ones. They're basically the same interface with slightly different method names.