Je kód v pořádku?
Potřeboval jsem pozměnit kód a s někým jsem se dohodl, že mi to udělá zdarma... jenom bych jsem rád věděl, zda je vše ok, jestli v kódu není nějaký problém nebo něco chybného..
Mockrát děkuji
KÓD:
<?php if($theme_scroller_display != "Disable") { ?>
<div id="scroller">
<?php if($theme_scroller_title) { ?><span id="scroller-title">Novinky<span class="scroller-bullet">››</span></span><?php } ?>
<?php if($theme_scroller_display == "Scrolling") { ?>
<marquee id="scroller-marquee" direction="left" behavior="scroll" onMouseover="this.scrollAmount=0" onMouseout="this.scrollAmount=8">
<?php } else { ?>
<span id="scroller-marquee">
<?php } ?>
<?php if($theme_scroller_text) { ?><span id="scroller-text"><?php echo do_shortcode($theme_scroller_text); ?></span><?php } ?>
<?php query_posts("cat=$theme_scroller_cats&posts_per_pa ge=$theme_scroller_posts"); if (have_posts()) : while (have_posts()) : the_post(); ?><span class="scroller-bullet">•</span><a href="<?php the_permalink(); ?>"><?php the_title(); ?></a><?php endwhile; endif; wp_reset_query(); ?>
<?php if($theme_scroller_display == "Scrolling") { ?>
</marquee>
<?php } else { ?>
</span>
<?php } ?>
</div>
<?php } ?>
Vyzerá to ok, ale bez kontextu ťažko povedať.
Ak the_permalink nepoužíva htmlspecialchars, tak by som to tam pridal.
Atribúty onMouseover a onMouseout by som písal lowercase.
Elementu marquee by som sa snažil vyhnúť, ale ak je to požadované, tak ok.
Shorttagy sú moja osobná preferencia, takže k tomu nebudem nič písať.