Přidat otázku mezi oblíbenéZasílat nové odpovědi e-mailemVyřešeno 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">&rsaquo;&rsaquo;</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">&#8226;</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 } ?>
Předmět Autor Datum
Vyzerá to ok, ale bez kontextu ťažko povedať. Ak the_permalink nepoužíva htmlspecialchars, tak by so… poslední
los 08.01.2013 18:47
los

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ť.

Zpět do poradny Odpovědět na původní otázku Nahoru