WordPress.org

Ready to get started?Download WordPress

Forums

Hey Cookie
css for ie being brought in for Firefox, Safari, and Chrome. (11 posts)

  1. Fishback
    Member
    Posted 2 years ago #

    The [if lt IE9] condition is working, disallowing ie css from coming in at the top of the template, but then another line further down allows the ie.css to come in which, for example, causes larger footer text than desired in Firefox, Safari, and Chrome. Two links from the demo site for Hey Cookie included.

    https://skitch.com/fishback/8a5k3/source-of-http-heycookie-demo.calobeedoodles.com

    https://skitch.com/fishback/8a5ks/source-of-http-heycookie-demo.calobeedoodles.com

    http://wordpress.org/extend/themes/hey-cookie/

  2. WPyogi
    Volunteer Moderator
    Posted 2 years ago #

    If you post a link to your site, someone can probably help you with this.

  3. Fishback
    Member
    Posted 2 years ago #

    To clarify, I am using Caorline's own demo site to point out what seems to be an error in ie.css handling.

    http://heycookie-demo.calobeedoodles.com/

    A second call is being made to ie.css that is not suppressed in other browsers, and because it comes in last minute, it overrides default style.css in those other browsers.

    This Firefox snapshot shows how the header h1.title has a 28px attribute which is *only* an ie.css attribute.

    https://skitch.com/fishback/82rbn/hey-cookie-ie-css-example

  4. Caroline Moore
    Member
    Theme Author

    Posted 2 years ago #

    I don't have ready access to my computer at the moment, but I'm pretty sure this is temporarily intentional behavior while I work on making the theme responsive. I could be wrong, tho. I started a new job this week and am currently traveling/training, so probably won't have time to delve into this theme for awhile, but I'll make a note of this for the next time I do. Thanks!

  5. Fishback
    Member
    Posted 2 years ago #

    I dug a little further. It seems to be that the conflicting ie style call is being brought in at the end of the header by:

    <?php wp_head(); ?>

    Will dig further. Congrats on new job! I'm experimenting with Hey Cookie over here... GraphicJoke.com

  6. Caroline Moore
    Member
    Theme Author

    Posted 2 years ago #

    Yes, check functions.php for the style registrations. :)

  7. Fishback
    Member
    Posted 2 years ago #

    Looks like functions.php. There is an "if" statement commented out and then by default ie.css is being brought in, even with no-ie browsers. So ie is overriding default style.css with non-ie browsers as well as overrides my own theme options css since those come in prior to this call.

    /*if( isset( $hey_cookie_responsive ) && $hey_cookie_responsive == 1 ) {
    wp_register_style( 'hcmedia', get_template_directory_uri() . '/media.css' );
    wp_enqueue_style( 'hcmedia' );
    }
    else {*/
    wp_register_style( 'hcmediaie', get_template_directory_uri() . '/ie.css' );
    wp_enqueue_style( 'hcmediaie' );
    //}

  8. Fishback
    Member
    Posted 2 years ago #

    I gather you are preparing for a responsive option in theme options (not showing yet) and for now are, in effect, defaulting to style inside ie.css. Well at least I know where to customize (since this all happens after the Custom CSS field statements.) I'm good for now!

  9. Fishback
    Member
    Posted 2 years ago #

    PS.
    The PayPal button at the bottom of theme options didn't go anywhere for me.

    <form action="https://www.paypal.com/cgi-bin/webscr" method="post">

    Will go PayPal the manual route!

  10. Fishback
    Member
    Posted 2 years ago #

    [ Moderator Note: Please post code or markup snippets between backticks or use the code button. ]

    Just upgraded to Hey Cookie 1.2.9 and the ie.css problem is still there. In functions.php because the if statement is commented out, in all cases the /ie.css is registered. This is creating conflicts in look with non-IE browsers as well as conflicts with the user custom CSS field since /ie.css is pulled in after the custom css is displayed in the header.

    /*if( isset( $hey_cookie_responsive ) && $hey_cookie_responsive == 1 ) {
    		wp_register_style( 'hcmedia', get_template_directory_uri() . '/media.css' );
    		wp_enqueue_style( 'hcmedia' );
    	}
    else {*/
    		wp_register_style( 'hcmediaie', get_template_directory_uri() . '/ie.css' );
    		wp_enqueue_style( 'hcmediaie' );
    	//}
  11. Caroline Moore
    Member
    Theme Author

    Posted 2 years ago #

    Yep, I haven't had a chance to look at this yet... I'm swamped. When it's fixed, I'll make sure to note it in the readme.

    In the meantime, you can probably revert to an older version of the theme using the SVN repo... the old, pre-meddled-with functions.php and stylesheets are there.

Topic Closed

This topic has been closed to new replies.

About this Theme

About this Topic

Tags