Nameslave
asked 6 years ago

Do I have to manually edit Argo copyright footer in function.php for every update (currently 1.0.7)? http://designwall.com/guide/dw-argo/#footer_text Please at least move it to footer.php like others so I could include it in my child theme. Or better still, esp. for less savvy users, make it a customizable item. Thanks.

4 Answers
DominicStaff
answered 6 years ago

To resolve this issue, you can add the following code to the 2 files: sidebar.php & single-download.php find the line 24 in the sidebar.php file and line 226 in the single-download.php  then replace the following code:

<?php dw_footer(); ?>

With new code:

<footer id="colophon" class="site-footer" role="contentinfo">
    <div class="site-footer-inner">
        <?php wp_nav_menu( array( 
            'theme_location' => 'footer', 
            'menu_class' => 'nav',
            'container' => false,
            'depth' => 1,
            'fallback_cb' =>  create_function( '', '
                ?>
                <ul class="nav">
                    <li><a href="'.site_url().'">'.__('Home',DW_TEXT_DOMAIN).'</a></li>
                    <li><a href="'.admin_url( 'nav-menus.php').'">'.__('Setting your menu', DW_TEXT_DOMAIN).'</a></li>
                </ul>
                <?php
            '),
        ) ); ?>
        <div class="site-info">
            <?php do_action( 'dw_credits' ); ?>
            Copyrightdddd &copy; <?php echo date("Y") ?> by <a href="<?php echo site_url(); ?>"><?php echo esc_attr( get_bloginfo( 'name', 'display' ) ); ?></a>.
            <br>
            <a href="<?php echo esc_url( __( 'http://wordpress.org/', DW_TEXT_DOMAIN ) ); ?>" title="<?php esc_attr_e( 'Semantic Personal Publishing Platform', DW_TEXT_DOMAIN ); ?>"><?php printf( __( 'Proudly powered by %s.', DW_TEXT_DOMAIN ), 'WordPress' ); ?></a>
            <a href="<?php echo esc_url( __( 'http://designwall.com/', DW_TEXT_DOMAIN ) ); ?>" rel="nofollow" title="<?php esc_attr_e( 'Responsive WordPress Themes', DW_TEXT_DOMAIN ); ?>"><?php printf( __( 'Theme by %s', DW_TEXT_DOMAIN ), 'DesignWall' ); ?></a>.
        </div>
    </div>
</footer>

Note: You can include 2file in your child theme.

Nameslave
answered 6 years ago

Thanks for the “workaround”, Dominic. Yes, it would be technically “workable” to modify sidebar.php and single-downlaod.php and include them in a child theme instead of manually editing function.php for every update. But the point is, why hardcode something that should be totally optional esp. for your PAID customers (like “Proudly powered by WordPress. Theme by DesignWall.”) in the first place, when even WordPress’ own theme review guideline explicitly requires against: http://make.wordpress.org/themes/guidelines/guidelines-license-theme-name-credit-links-up-sell-themes/

DominicStaff
answered 6 years ago

Very useful tip, Nameslave. We will consider and discuss about that in the next versions of the theme. Again, thanks for the feedback and look forward to hearing more from you, my friend 🙂  

Nameslave
answered 6 years ago

Thanks, Dominic. And I look forward to the next version update. Honestly, DW will definitely benefit more by complying with the WordPress community’s recommended best practice. Keep up the good work. :thumbs up:

Powered by DW Question & Answer Pro