• Except of the front page there is the id=”content” always used twice in this theme. I think this should be fixed. ??

    All the best,
    Torsten

Viewing 4 replies - 1 through 4 (of 4 total)
  • Thread Starter Torsten Landsiedel

    (@zodiac1978)

    And in the content.php there is an unnecessary closing a tag.

    All the best,
    Torsten

    Moderator Kathryn Presner

    (@zoonini)

    Hi Torsten, nice to see you here. ??

    I’m not seeing a duplicate content ID on the demo here:

    https://selademo.wordpress.com/

    If you could possibly paste the entire line where you see the second ID so I can find it that would be great, thank you!

    I see this on line 180:

    <div id="content" class="site-content">

    And in the content.php there is an unnecessary closing a tag.

    On which line are you seeing an extra closing tag? Which tag are you looking at?

    Thanks for the reports!

    Thread Starter Torsten Landsiedel

    (@zodiac1978)

    Hi Kathryn! ??

    You can’t see it on the front page, but I think on every other page, for example on the full width page:
    https://validator.w3.org/nu/?doc=https%3A%2F%2Fselademo.wordpress.com%2Fpage-templates%2Ffull-width-page%2F

    On line 39 in header.php:
    <div id="content" class="site-content">

    On line 21 in full-width-page.php and line 20 in grid-page.php:
    <div id="content" class="site-main" role="main">

    And for the additional closing tag:

    It is in line 15 in content.php
    <?php the_title( '<h1 class="entry-title"><a href=" ' . esc_url( get_permalink() ) . '" rel="bookmark">', '</a></h1>' ); ?></a>

    The closing tag is in the the_title call and there is one after the call, too. The second one (</a>) at the end of the line is not necessary.

    These files have the same problem:
    content-gallery.php (line 17)
    content-video.php, content-aside.php, content-quote.php, content-link.php (line 15)

    As it is in the else part it is just used on non single posts, for example on the blog page:
    https://validator.w3.org/nu/?doc=https%3A%2F%2Fselademo.wordpress.com%2Fblog%2F

    <?php if ( is_single() ) : ?>
    			<?php the_title( '<h1 class="entry-title">', '</h1>' ); ?>
    		<?php else : ?>
    			<?php the_title( '<h1 class="entry-title"><a href=" ' . esc_url( get_permalink() ) . '" rel="bookmark">', '</a></h1>' ); ?></a>
    		<?php endif; // is_single() ?>

    I hope this will help you to spot the mistakes. ??

    All the best,
    Torsten

    Moderator Kathryn Presner

    (@zoonini)

    Thanks so much for those details, Torsten – I’ve reported the issues to our developers and they should be fixed in the next version of Sela. Good eye!

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘id="content" is used twice’ is closed to new replies.