Please validate Jetpack

  • First, I was not a fan of jetpack before, but I’m actually really liking this latest version 3.0. However, someone really needs to validate their code. I put in a gallery and carousal into my demo site I am building my my next theme, and not only does my page not validate due to these two plugins, your stylesheet for both of these are being loaded in the content (body) part of the source code (I’m shocked people).

    So with that being said, are their plans to validate the code and enqueue the stylesheets in the <head> and not the body?

    https://www.ads-software.com/plugins/jetpack/

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author Brandon Kraft

    (@kraftbj)

    Code Wrangler

    Howdy,

    Yes, there’s a plan to update all outputted HTML to validate as HTML 5, though it is at the early stages. We’re going to tackle front-end HTML first (since /wp-admin/ is still at HTML 4) and take it module by module.

    To confirm we’re on the same page, which stylesheets are you seeing enqueued into the body as I’m seeing the stylesheets for Tiled Galleries and Carousel in the head.

    Cheers.

    Thread Starter Styled Themes

    (@gejay)

    Awesome to hear…thanks. As for the enqueued styles, you can check out my demo site here, but look at the source code down around line 602

    https://demo.styledthemes.com/preferential/

    id=’jetpack-carousel-css’
    id=’tiled-gallery-css’

    Plugin Author Brandon Kraft

    (@kraftbj)

    Code Wrangler

    I’m on the same page now. I wasn’t seeing it before because this happens when the tiled gallery is being used as a widget in my testing. If the gallery is part of the post content, then it is in the head as expected.

    I’ll look at shifting things around a bit and ensuring nothing breaks through the changes.

    Cheers!

    Plugin Author Brandon Kraft

    (@kraftbj)

    Code Wrangler

    Actually, looking a bit closer, I’m not sure if we’ll be able to do this well. The wp_head hook fires before the sidebars, so we can’t just shift them up to the head.

    The alternative would be to pass the stylesheets to the head simply if the module is active, without regard to if they’re actually used or not, which may be more of a negative than simply letting them enqueue, as they are needed, in the footer.

    The share daddy plugin needs an update, too.

    Error: The scrolling attribute on the iframe element is obsolete. Use CSS instead.
    Error: The frameborder attribute on the iframe element is obsolete. Use CSS instead.
    Error: The allowtransparency attribute on the iframe element is obsolete. Use CSS instead.

    Plugin Author Brandon Kraft

    (@kraftbj)

    Code Wrangler

    We’ll look at all front-facing modules. Thanks!

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Please validate Jetpack’ is closed to new replies.