Ok, another thing is that when I moved all the handling of figuring out which page and tab are active out of template logic, I made a point to specify `sidebar_items` in a way that there could be a single `is_active()` method in the base class and not have to override it with complicated per-tab logic.  Where I couldn't do that due to the existing structure not fitting into the model I defined, I've commented that they're hacks, and as I've expanded the complexity of the navigation framework (e.g., not requiring that all subpages of a tab begin with that tab's base URL for the commconnect-only view of HQ changes), I've been trying to remove those hacks.

Given that `UITab.is_active()` already checks all of the urls defined in `sidebar_items` (https://github.com/dimagi/commcare-hq/blob/3d6307286cab72502a56148229cdabab2f52fa10/corehq/apps/hqwebapp/models.py#L138) and you define `cloudcare_app_settings` as one of those urls below (https://github.com/dimagi/commcare-hq/pull/918/files#L40R597) I'm unsure how any of this is necessary.