BPDaily – Changing default image header size to 970 by 200px

Hi,

I read this thread that is closed:

https://premium.wpmudev.org/forums/topic/i-need-to-make-the-bp-daily-header-taller#post-43497

It seems the files have changed since then, so it is not easy to find what I need to get this done. Of course i found custom-header.php and saw how that can be edited.

define( ‘HEADER_IMAGE’, $headerpath ); // %s is theme dir uri

define( ‘HEADER_IMAGE_WIDTH’, 970 );

define( ‘HEADER_IMAGE_HEIGHT’, 80 );

The issue I have is there header space is still not allowing this now larger image when I change to 200. Any help with that would be appreciated.

WordPress: 3.3.1

Buddypress: 1.5.4

BPDaily: 4.1.7

Thanks,

John

  • aecnu
    • WP Unicorn

    Greetings johnpent,

    Thank you for being a WPMU Dev member!

    Just to clarify what is going on for resolution …..

    When you change the following to 200 it does not change the height.

    define( ‘HEADER_IMAGE_HEIGHT’, 80 );

    Are you doing this in the individually uploaded to themes child site?

    Cheers, Joe :slight_smile:

  • johnpent
    • Flash Drive

    I did the change here:

    public_html/wp-content/themes/bp-daily/library/functions/custom-header.php

    I fully understand using child themes, however, I usually test in the main theme, install the original files and make the final changes via child theme.

    define( 'HEADER_IMAGE', $headerpath ); // %s is theme dir uri
    define( 'HEADER_IMAGE_WIDTH', 970 );
    define( 'HEADER_IMAGE_HEIGHT', 200 );

    function bp_dtheme_header_style() { ?>
    <style type=&quot;text/css&quot;>
    #header { background-image: url(<?php header_image() ?>); }
    <?php if ( 'blank' == get_header_textcolor() ) { ?>
    #header h1 a {display: none; }
    #header h1 a:visited {display: none; }
    #header h1 a:hover {display: none; }
    <?php } else { ?>
    #header h1 a {color:#<?php header_textcolor() ?>;}
    #header h1 a:visited {color:#<?php header_textcolor() ?>;}
    #header h1 a:hover {color:#<?php header_textcolor() ?>;}
    <?php } ?>
    </style>
    <?php
    }

    function bp_dtheme_admin_header_style() { ?>
    <style type=&quot;text/css&quot;>
    #headimg {
    position: relative;
    color: #fff;
    background: url(<?php header_image() ?>);
    -moz-border-radius-bottomleft: 6px;
    -webkit-border-bottom-left-radius: 6px;
    -moz-border-radius-bottomright: 6px;
    -webkit-border-bottom-right-radius: 6px;
    margin-bottom: 20px;
    height: 100px;
    padding-top: 25px;
    }

    That is the code as I presently have it.

    Thanks for getting back so quickly.

    John

  • aecnu
    • WP Unicorn

    Greetings johnpent,

    Just checking if this issue was eventually resolved in another thread? Or by yourself separately to us? Or by us over email with you? Or using our live support?

    If so, no need to reply, that’s great news.

    If not, or you have any more questions related to this thread, please feel free to post them below including any new symptoms or errors and tick the ‘Mark as Not Resolved (re-open)’ box below the post area (or else we’ll miss it!)

    Otherwise, happy days, glad you got it sorted :slight_smile:

    Thank you for being a WPMU Dev member!

    Cheers, Joe :slight_smile:

Thank NAME, for their help.

Let NAME know exactly why they deserved these points.

Gift a custom amount of points.