WordPress.org

Ready to get started?Download WordPress

Forums

Slideshow
[resolved] HTML Errors (3 posts)

  1. kYem
    Member
    Posted 1 year ago #

    There are couple of things, that are easy to fix and make them html valid: frontend_attachement.php Line 55.

    <div class="slideshow_slide slideshow_slide_image">
    				<a <?php echo $anchorTagAttributes; ?>>
    					<img src="<?php echo htmlspecialchars($imageSrc); ?>" alt="<?php echo $alt; ?>" width="<?php echo $imageWidth ?>" height="<?php echo $imageHeight; ?>">
    				</a>
    				<div class="slideshow_description slideshow_transparent">
    					<a>>
    						<?php echo !empty($title) ? '<h2>' . $title . '</h2>' : ''; ?>
    						<?php echo !empty($description) ? '<p>' . $description . '</p>' : ''; ?>
    					</a>
    				</div>
    			</div>

    Should be changed to

    <div class="slideshow_slide slideshow_slide_image">
    				<a <?php echo $anchorTagAttributes; ?>>
    					<img src="<?php echo htmlspecialchars($imageSrc); ?>" alt="<?php echo $alt; ?>" width="<?php echo $imageWidth ?>" height="<?php echo $imageHeight; ?>"/>
    				</a>
    				<div class="slideshow_description slideshow_transparent">
    						<?php echo !empty($title) ? '<h2><a'. $anchorTagAttributes .'>' . $title . '</a></h2>' : ''; ?>
    						<?php echo !empty($description) ? '<p><a'. $anchorTagAttributes .'>' . $description . '</a></p>' : ''; ?>
    				</div>
    			</div>

    That would fix the self closing error on the img tag and the block/inline element errors.

    The last one is in slideshow.php, line 8

    <div class="slideshow_pagination"><div class="slideshow_pagination_center">
    <ul></ul>
    </div></div>

    This will cause the end tag for "ul" which is not finished html error as you actually need li element with it.

    One solution is to create the ul tag inside jQuery and the populate it with list items. It would fix the html problem.

    Now the last one, document type does not allow element "style" here, I assume will be sorted out eventually.

    http://wordpress.org/extend/plugins/slideshow-jquery-image-gallery/

  2. Stefan Boonstra
    Member
    Plugin Author

    Posted 1 year ago #

    Thanks for sharing this information! It would indeed be better to move the anchor tags inside the elements and to have jQuery create the unordered list.

    I'll apply these fixes in version 2.2.8.

  3. Stefan Boonstra
    Member
    Plugin Author

    Posted 1 year ago #

    The style element will be moved to the bottom of the page in version 2.2.8 as well. This will make the stylesheets cahcheable, so this has multiple advantages.

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic