UB Custom Favicons + JetPack Site Icon module

Hi WPMUdev,

I hope your day is going well!

I have a Q continuing from a Live Chat w/ @Tyler Postle re. Ultimate Branding…

I was initially looking at removing the blavatar images from the My Sites submenu… but then realized that the UB Custom Multisite Favicons module would neatly whitelabel this element as part of its function.

The issue then became the topic of this thread: UB CMF module does NOT allow for the JetPack Site Icon module to work for individual subsites.

My goal:

1) To use UB CMF module to whitelabel the platform by replacing the default images used.

2) Then allow individual subsites to use JetPack’s Site Icon module to set their own favicon instead of the one set by UB.

Make sense?

How can we make it so that UB is sensitive to JetPack’s Site Icon module being used?

You can see this live right at tivism.com – UB CMF is in use displaying the heart image, JetPack’s Site Icon module is active at primary and should be displaying a little laptop image instead… support access is active if you want to see the settings :slight_smile:

Kind Regards,

Max

  • Vinod Dalvi
    • WP Unicorn

    Hi Max,

    I hope you are well today and thank you for your question.

    I tried to access admin area of your site using support staff access but got the following error message.

    Sorry, this is an invalid access token. You must ask the user to grant access.

    Could you please revoke and regrant it?

    It would be also useful if you just share some screenshots of it.

    Kind Regards,

    Vinod Dalvi

  • wp.network
    • The Bug Hunter

    Hi @Vinod Dalvi

    1) Support Access: Revoked & Re-Activated as requested…

    2) Screenshots:

    [attachments are only viewable by logged-in members] [attachments are only viewable by logged-in members] [attachments are only viewable by logged-in members]

    UB CMF is in use displaying the heart image, JetPack’s Site Icon module is active at primary and should be displaying a little laptop image instead…

    Hopefully this clarifies the issue.

    Again, my goal is to a) use UB Custom Multisite favicons to whitelabel the platform, while also b) allowing UB CMF to be ‘overridden’ by JetPack’s Site Icon module for individual subsites (ideally, including primary too).

    Kind Regards, Max

  • wp.network
    • The Bug Hunter

    I am thinking that perhaps part of the issue here is that WPMUdev has only one Ultimate Branding plugin and that this plugin has been build with open networks like edublogs in mind… I do not generally run fully open networks (in fact, I have never run an open network of my own), instead I use WPMS in more of a ‘closed network’ model…

    Thinking about this, I can see how possibly the above issue can be seen as a positive – but only in an open network context – in a closed network wherein clients are much more trusted (and are paying sig. fees for service), the above issue (UB not allowing individual site’s to subsequently set unique favicon/blavatar) is just a negative as far as I can see.

    Unless this is resolvable, I consider the UB CMF module only suitable for (some) open networks – & imho, that’d fall pretty far short of ‘ultimate’ whitelabling… so, hopefully we can identify some ways to move forward here :slight_smile:

    I will look/google/think on this more, may even try some experimenting… of course, will post with anything noteworthy.

    Kind Regards, Max

  • wp.network
    • The Bug Hunter

    of everything I’ve seen so far via google, the following snippet looks like it can tell me alot about what is happening by default with the blavatar in My Sites submenu, and some ways to mess with it… will see how far I can get in looking at core files & finding some improved understanding… :slight_smile:

    /*
    * Change My Sites Menu Names to Domain.com as the menu name
    */
    function change_site_names() {
    global $wp_admin_bar;
    $blue_wp_logo_url = includes_url('images/wpmini-blue.png');
    $blavatar = '<img src="' . esc_url($blue_wp_logo_url) . '" alt="' . esc_attr__( 'Blavatar' ) . '" width="16" height="16" class="blavatar"/>';
    foreach ( (array) $wp_admin_bar->user->blogs as $blog ) {
    $menu_id = 'blog-' . $blog->userblog_id;
    $blogname = ucfirst( $blog->domain );
    $wp_admin_bar->add_menu( array(
    'parent' => 'my-sites-list',
    'id' => $menu_id,
    'title' => $blavatar . $blogname,
    'href' => get_admin_url( $blog->userblog_id ) )
    );
    }
    }
    add_action( 'wp_before_admin_bar_render', 'change_site_names' );

    Cheers, Max

  • wp.network
    • The Bug Hunter

    …doing some more careful testing after I noticed confusing results, will post as I go :slight_smile:

    All browser caches totally cleared, local DNS cache cleared.

    CloudFlare cache cleared, development mode active.

    Sites tested w/ UB CMF module used to assign ‘heart’ image:

    primary: tivism.com – using JetPack Site Icon module ‘laptop’ image

    subsite: tivism.net – using JetPack Site Icon module ‘desktop’ image

    subsite: tivism.org – NOT using JetPack Site Icon module, should be UB ‘heart’ image

    For IE11

    Logged Out:

    tivism.com – JetPack image as favicon

    [attachments are only viewable by logged-in members]

    tivism.net – JetPack image as favicon

    [attachments are only viewable by logged-in members]

    tivism.org – UB CMF image as favicon

    [attachments are only viewable by logged-in members]

    Logged In:

    tivism.com – JetPack image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

    tivism.net – JetPack image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

    tivism.org – UB CMF image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

  • wp.network
    • The Bug Hunter

    For Chrome

    Logged Out:

    tivism.com – UB CMF image as favicon

    [attachments are only viewable by logged-in members]

    tivism.net – UB CMF image as favicon

    [attachments are only viewable by logged-in members]

    tivism.org – UB CMF image as favicon

    [attachments are only viewable by logged-in members]

    Logged In:

    tivism.com – UB CMF image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

    tivism.net – UB CMF image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

    tivism.org – UB CMF image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

  • wp.network
    • The Bug Hunter

    For Firefox

    Logged Out:

    tivism.com – Genesis default image as favicon

    [attachments are only viewable by logged-in members]

    tivism.net – Genesis default image as favicon

    [attachments are only viewable by logged-in members]

    tivism.org – Genesis default image as favicon

    [attachments are only viewable by logged-in members]

    Logged In:

    tivism.com – JetPack image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

    tivism.net – JetPack image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

    tivism.org – UB CMF image as favicon, UB CMF image as blavatar in My Sites submenu

    [attachments are only viewable by logged-in members]

  • wp.network
    • The Bug Hunter

    Summary:

    For IE11 –

    Logged Out:

    tivism.com – JetPack image as favicon (Good, as desired)

    tivism.net – JetPack image as favicon (Good, as desired)

    tivism.org – UB CMF image as favicon (Good, as desired)

    Logged In:

    tivism.com – JetPack image as favicon (Good, as desired), UB CMF image as blavatar in My Sites submenu (Bad, NOT as desired)

    tivism.net – JetPack image as favicon (Good, as desired), UB CMF image as blavatar in My Sites submenu (Bad, NOT as desired)

    tivism.org – UB CMF image as favicon (Good, as desired), UB CMF image as blavatar in My Sites submenu (Good, as desired)

    For Chrome –

    Logged Out:

    tivism.com – UB CMF image as favicon (Bad, NOT as desired)

    tivism.net – UB CMF image as favicon (Bad, NOT as desired)

    tivism.org – UB CMF image as favicon (Good, as desired)

    Logged In:

    tivism.com – UB CMF image as favicon (Bad, NOT as desired), UB CMF image as blavatar in My Sites submenu (Bad, NOT as desired)

    tivism.net – UB CMF image as favicon (Bad, NOT as desired), UB CMF image as blavatar in My Sites submenu (Bad, NOT as desired)

    tivism.org – UB CMF image as favicon (Good, as desired), UB CMF image as blavatar in My Sites submenu (Good, as desired)

    For Firefox

    Logged Out:

    tivism.com – Genesis default image as favicon (Bad, NOT as desired)

    tivism.net – Genesis default image as favicon (Bad, NOT as desired)

    tivism.org – Genesis default image as favicon (Bad, NOT as desired)

    Logged In:

    tivism.com – JetPack image as favicon (Good, as desired), UB CMF image as blavatar in My Sites submenu (Bad, NOT as desired)

    tivism.net – JetPack image as favicon (Good, as desired), UB CMF image as blavatar in My Sites submenu (Bad, NOT as desired)

    tivism.org – UB CMF image as favicon (Good, as desired), UB CMF image as blavatar in My Sites submenu (Good, as desired)

    tivism.com & tivism.net should be using the JetPack assigned image as favicon/blavatar.

    tivism.org should be using the UB assigned image as favicon/blavatar.

    Ok! Hope this level of detail is actually helpful :slight_smile:

    edit: just took a glance w/ Opera too…

    1) seems like w/ Opera for Windows the UB image is shown as favicon when logged out, didn’t test logged in

    2) however, using Opera on Android, when I use ‘Save for Offline’ the JetPack Site Icon image is shown in my saved pages

    This is not what I’d call a consistent branding experience!

    I’d really like to figure out what is making this behave so unevenly, and I’d like a solution even more :slight_smile:

    Kind Regards, Max

    ps. above testing done on my handy Win8 Transformer Book

  • wp.network
    • The Bug Hunter

    So, I’m thinking that, for now, the first thing to try is simply borrow from Ipstenu

    // No Genesis, you may not have your favicon
    remove_action('genesis_meta', 'genesis_load_favicon');

    to get the Genesis favicon out of the mix… then it should just be beween UB + JetPack and we’ll see what happens in Firefox :slight_smile:

    I am going to introduce the above snippet now for tivism.com – all else remains default / as described above.

    Cheers, Max

  • Tyler Postle
    • Recruit

    Hey there Max,

    Thanks again for stopping by the chat!

    I didn’t realize you were using the UB Favicon module, I thought you meant just having UB activated at all would make the Jetpack favicon module not work.

    In this case, yes Jetpack favicon and UB favicon do conflict. Adding favicons per site is on our to do list and I think that will solve your issue here as then you won’t need Jetpack favicon at all correct? :slight_smile:

    Per site favicons is a feature that definitely gets my +1 and I’ll add your support to the feature request ticket too. I see there are quite a few posts on our forums about it so I’ll mention that.

    In the meantime, perhaps just using JetPack Favicon would be an alternative if you don’t want to have only 1 network wide favicon :slight_smile:

    Appreciate your detailed post. I do always worry that I missed something though :p if I missed anything that wasn’t an issue of the conflict between using the 2 different favicon modules then let us know and we can look into getting that sorted out in the meantime!

    Look forward to hearing back on this Max.

    Cheers,

    Tyler

  • wp.network
    • The Bug Hunter

    :disappointed: well, I did learn a few things, but the screenshots were kinda fiddly to do… oh well :slight_smile:

    Adding favicons per site is on our to do list and I think that will solve your issue here as then you won’t need Jetpack favicon at all correct? :slight_smile:

    +1 for sure, would solve my issue as long as its as easy for a user as the JetPack Site Icon process

    In the meantime, perhaps just using JetPack Favicon would be an alternative if you don’t want to have only 1 network wide favicon :slight_smile:

    Not an option.

    That would not meet goal number 1: Whitelabel the platform

    I will look at other ways to get this done for now

    Thanks @Tyler Postle

    Kind Regards, Max

  • Tyler Postle
    • Recruit

    Hey Max,

    Hope you’re doing well!

    Ah, right, they wouldn’t your favicon to begin with :slight_frown: Yes, I think adding the ability to have a per site favicon to UB would be the best solution here. I remember this question coming up a little while back and it was the same situation then, one or the other.

    Hopefully we can get that feature added soon :slight_smile:

    Let us know if any further questions come up.

    Cheers,

    Tyler

  • Vinod Dalvi
    • WP Unicorn

    Hi Max,

    Thank you for your patience.

    There is already a feature request created for this feature and this feature may be added in future version of plugin but We don’t publish ETAs to prevent disappointment if a deadline is missed(which in plugin development quite a frequent occurrence!).

    The feature is not implemented in the plugin.

    Thanks,

    Vinod Dalvi

  • wp.network
    • The Bug Hunter

    Hey WPMUDEV… any update on this?

    Per site favicons is a feature that definitely gets my +1 and I’ll add your support to the feature request ticket too. I see there are quite a few posts on our forums about it so I’ll mention that.

    In the meantime, perhaps just using JetPack Favicon would be an alternative if you don’t want to have only 1 network wide favicon :slight_smile:

    Not an option.

    That would not meet goal number 1: Whitelabel the platform

    Ah, right, they wouldn’t your favicon to begin with :slight_frown: Yes, I think adding the ability to have a per site favicon to UB would be the best solution here. I remember this question coming up a little while back and it was the same situation then, one or the other.

    Hopefully we can get that feature added soon :slight_smile:

    If this feature has not been added to UB yet, can you please provide a link to the the feature request thread mentioned by @Tyler Postle and @Vinod Dalvi

    Kind Regards, Max

  • Tyler Postle
    • Recruit

    Hey Max,

    Sorry for not updating this thread awhile back. Yes, this feature is now in Ultimate Branding :slight_smile: it was added back in 1.8.1

    If you activate the Custom Multisite Favicons module then go to the settings you will see their is a field for the main site favicon where you can also apply that by default to all subsites then down below that you have options for favicons per site. I've attached a screenshot to illustrate.

    Hope that helps!

    Cheers,

    Tyler

    [attachments are only viewable by logged-in members]

  • wp.network
    • The Bug Hunter

    Thanks @Tyler Postle I was just gettin off my lazy backend to read the changelog…

    Version 1.8.1.1Download

    Fixed: Ability to add favicons using external image url

    Version 1.8.1Download

    Added: Favicon for sub-sites

    Cheers WPMUDEV – thank y’all for getting this done!

    Kind Regards, Max