Отображение всех одиночных сообщений


Я думаю, что есть некоторые запутанные и странные проблемы отступ в этом файле Вордпресс (single.php). Любые предложения по улучшению отступы и читаемости?

<?php
/**
 * The Template for displaying all single posts.
 *
 * @package WordPress
 * @subpackage Starkers
 * @since Starkers 3.0
 */

get_header(); ?>

<?php get_sidebar(); ?>

<div id="content">
    <?php // Set and display custom field
        $intro_image = get_post_meta($post->ID, 'Intro Image', true); ?>
        <div class="block-1">
            <img src="<?php echo $intro_image; ?>" alt="" />
        </div> <?php
    ?>

    <?php // Start The Loop
        if ( have_posts() ) while ( have_posts() ) : the_post();
    ?>  <div class="block-2 padding-top no-overlay">
            <?php the_content(); ?>
        </div><!-- .entry-content -->
    <?php endwhile; // end of the loop. ?>

    <?php //  Display previous and next posts thumbnails ?>
        <div class="block-2 border-top">
        <?php // Set and display custom field
            $mainbar_left_title = get_post_meta($post->ID, 'Mainbar Left Title', true);
            $mainbar_left_image = get_post_meta($post->ID, 'Mainbar Left Image', true); ?>
            <div class="float-left">
                <h2><?php echo $mainbar_left_title; ?></h2>
                <img src="<?php echo $mainbar_left_image ?>" alt="" />
            </div> <?php
        ?>

        <?php // Set and display custom field
            $mainbar_right_title = get_post_meta($post->ID, 'Mainbar Right Title', true);
            $mainbar_right_image = get_post_meta($post->ID, 'Mainbar Right Image', true); ?>
            <div class="float-right">
                <h2><?php echo $mainbar_right_title; ?></h2>
                <img src="<?php echo $mainbar_right_image ?>" alt="" />
            </div> <?php
        ?>
    </div>

    <?php // Set and display custom field
        $mainbar_right_title = get_post_meta($post->ID, 'Mainbar Right Title', true);
        $mainbar_right_image = get_post_meta($post->ID, 'Mainbar Right Image', true); ?>
        <div class="block-3 border-top">
            <h2><?php echo $mainbar_right_title; ?></h2>
            <img src="<?php echo $mainbar_right_image ?>" alt="" />
        </div> <?php
    ?>

    <?php // Start The Loop
        if ( have_posts() ) while ( have_posts() ) : the_post();
    ?>  <div class="block-4 border-top">
            <?php the_content(); ?>
        </div><!-- .entry-content -->
    <?php endwhile; // end of the loop. ?>

    <?php get_sidebar('secondary'); ?>
</div><!-- #content -->

<?php get_footer(); ?>


335
4
задан 18 февраля 2011 в 12:02 Источник Поделиться
Комментарии
2 ответа

Вы могли бы вы PHP альтернативный синтаксис

<?php
/**
* The Template for displaying all single posts.
*
* @package WordPress
* @subpackage Starkers
* @since Starkers 3.0
*/
?>

<?php get_header();
<?php get_sidebar(); ?>

<div id="content">
<?php $intro_image = get_post_meta($post->ID, 'Intro Image', true); ?>
<div class="block-1">
<img src="<?php echo $intro_image; ?>" alt="" />
</div>

<?php if ( have_posts() ? while ( have_posts() ) : the_post()): ?>
<div class="block-2 padding-top no-overlay">
<?php the_content(); ?>
</div>
<?php endwhile; ?>

<div class="block-2 border-top">
<?php $mainbar_left_title = get_post_meta($post->ID, 'Mainbar Left Title', true); ?>
<?php $mainbar_left_image = get_post_meta($post->ID, 'Mainbar Left Image', true); ?>
<div class="float-left">
<h2><?php echo $mainbar_left_title; ?></h2>
<img src="<?php echo $mainbar_left_image ?>" alt="" />
</div>

<?php $mainbar_right_title = get_post_meta($post->ID, 'Mainbar Right Title', true); ?>
<?php $mainbar_right_image = get_post_meta($post->ID, 'Mainbar Right Image', true); ?>
<div class="float-right">
<h2><?php echo $mainbar_right_title; ?></h2>
<img src="<?php echo $mainbar_right_image ?>" alt="" />
</div>
</div>

<?php $mainbar_right_title = get_post_meta($post->ID, 'Mainbar Right Title', true); ?>
<?php $mainbar_right_image = get_post_meta($post->ID, 'Mainbar Right Image', true); ?>
<div class="block-3 border-top">
<h2><?php echo $mainbar_right_title; ?></h2>
<img src="<?php echo $mainbar_right_image ?>" alt="" />
</div>

<?php if ( have_posts() ? while ( have_posts() ) : the_post()): ?>
<div class="block-4 border-top">
<?php the_content(); ?>
</div>
<?php endwhile; ?>

<?php get_sidebar('secondary'); ?>
</div>

<?php get_footer(); ?>

Это следует всем стандартам кодирования WP из того, что я могу сказать и читаемым.
Я также хотел бы предложить изменение разделов такой:

<?php $mainbar_right_title = get_post_meta($post->ID, 'Mainbar Right Title', true); ?>
<?php $mainbar_right_image = get_post_meta($post->ID, 'Mainbar Right Image', true); ?>
<div class="block-3 border-top">
<h2><?php echo $mainbar_right_title; ?></h2>
<img src="<?php echo $mainbar_right_image ?>" alt="" />
</div>

К:

<div class="block-3 border-top">
<h2><?php echo get_post_meta($post->ID, 'Mainbar Right Title', true); ?></h2>
<img src="<?php echo get_post_meta($post->ID, 'Mainbar Right Image', true) ?>" alt="" />
</div>

Хотя это просто личное предпочтение, я нахожу его гораздо более читаемым.

4
ответ дан 20 февраля 2011 в 02:02 Источник Поделиться

<?php // Start The Loop
if ( have_posts() ) while ( have_posts() ) : the_post();
?> <div class="block-2 padding-top no-overlay">
<?php the_content(); ?>
</div><!-- .entry-content -->
<?php endwhile; // end of the loop. ?>

Которые могли бы стать:

<?php // Start The Loop
if (have_posts()) {
while (have_posts()) : the_post(); ?>
<div class="block-2 padding-top no-overlay">
<?php the_content(); ?>
</div><!-- .entry-content -->
<?php endwhile; // end of the loop.
} ?>

Еще немного смущает, но я продолжаю думать об этом.

Еще одна идея, что больше использует теги:

<?php if (have_posts()) { // Start the loop ?>
<? while (have_posts()) : the_post(); ?>
<div class="block-2 padding-top no-overlay">
<?php the_content(); ?>
</div><!-- .entry-content -->
<?php endwhile; // end of the loop. ?>
<?php } ?>

Еще одно изменение раздела:

<?php // Set and display custom field
$mainbar_right_title = get_post_meta($post->ID, 'Mainbar Right Title', true);
$mainbar_right_image = get_post_meta($post->ID, 'Mainbar Right Image', true);
?>
<div class="float-right">
<h2><?php echo $mainbar_right_title; ?></h2>
<img src="<?php echo $mainbar_right_image ?>" alt="" />
</div>

Есть, казалось бы, бесполезны сидя там. Похоже, идея была закрыть блок кода, но это не делать ничего. Я удален, что и изменить отступы, поэтому он не был похож на блока.

2
ответ дан 18 февраля 2011 в 02:02 Источник Поделиться