Context Navigation

Changes between Initial Version and Version 1 of Ticket #7996, comment 1

-                      initial
+                      v1
 Swapping `home_url` with `bp_get_root_domain` is a good idea.
+I'm less convinced about adding a `BP_Email` class method for the `email.preheader` part as that only sets an (optional) template token. Given this is single-use, I don't immediately see a good reason why it should be a global function. I think we would be fine to just wrap that filter call in `switch_to_blog_` etc.
+I'm less convinced about adding a `BP_Email` class method for the `email.preheader` part, as this code only sets an (optional) templating token, and I don't believe that fits into the purpose of what the `BP_Email` class represents. Given it's single-use, I don't immediately see a good reason why it should be a global function either. I think we would be fine to just wrap that filter call in `switch_to_blog_` etc.