BrowserViewController.swift is approaching 2000 lines of code. This is not healthy.
Now a lot of code in there are delegate methods that are already grouped in class extensions. How about we start to move some of those into files like BrowserViewController-BrowserToolbarDelegate.swift and BrowserViewController-ReaderMode.swift I know that still means we have a giant class but at least things are a bit manageable then. Open for suggestions on how to move larger chunks of logic to smaller classes. Maybe those will then also be easier to unit test? S.
_______________________________________________ mobile-firefox-dev mailing list [email protected] https://mail.mozilla.org/listinfo/mobile-firefox-dev
