WordPress.org

Plugin Directory

Changeset 368921


Ignore:
Timestamp:
04/05/11 03:59:23 (3 years ago)
Author:
gluten
Message:

Simplify options page and remove behavior placing selected option first in list.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • galleria-galleria/trunk/galleria-galleria.php

    r368920 r368921  
    222222                            <?php 
    223223                                $selected = $options['design']; 
    224                                 $p = ''; 
    225                                 $r = ''; 
    226224 
    227225                                foreach ( $defaults['design'] as $option ) { 
    228226                                    $label = $option['label']; 
    229                                     if ( $selected == $option['value'] ) // Make default first in list 
    230                                         $p = "\n\t<option style=\"padding-right: 10px;\" selected='selected' value='" . esc_attr( $option['value'] ) . "'>$label</option>"; 
    231                                     else 
    232                                         $r .= "\n\t<option style=\"padding-right: 10px;\" value='" . esc_attr( $option['value'] ) . "'>$label</option>"; 
     227                                    $value = esc_attr( $option['value'] ); 
     228                                     
     229                                    echo "<option style='padding-right: 10px;' value='$value'"; 
     230                                    if ( $selected == $option['value'] ) { 
     231                                        echo ' selected=selected '; 
     232                                    } 
     233                                    echo ">$label</option>"; 
    233234                                } 
    234                                 echo $p . $r; 
    235235                            ?> 
    236236                        </select> 
     
    250250                            <?php 
    251251                                $selected = $options['transition']; 
    252                                 $p = ''; 
    253                                 $r = ''; 
    254  
     252                                 
    255253                                foreach ( $defaults['transition'] as $option ) { 
    256254                                    $label = $option['label']; 
    257                                     if ( $selected == $option['value'] ) // Make default first in list 
    258                                         $p = "\n\t<option style=\"padding-right: 10px;\" selected='selected' value='" . esc_attr( $option['value'] ) . "'>$label</option>"; 
    259                                     else 
    260                                         $r .= "\n\t<option style=\"padding-right: 10px;\" value='" . esc_attr( $option['value'] ) . "'>$label</option>"; 
     255                                    $value = esc_attr( $option['value'] ); 
     256                                     
     257                                    echo "<option style='padding-right: 10px;' value='$value'"; 
     258                                    if ( $selected == $option['value'] ) { 
     259                                        echo ' selected=selected '; 
     260                                    } 
     261                                    echo ">$label</option>"; 
    261262                                } 
    262                                 echo $p . $r; 
    263263                            ?> 
    264264                        </select> 
     
    308308                            <?php 
    309309                                $selected = $options['image']; 
    310                                 $p = ''; 
    311                                 $r = ''; 
    312  
     310                                 
    313311                                foreach ( $defaults['image'] as $option ) { 
    314312                                    $label = $option['label']; 
    315                                     if ( $selected == $option['value'] ) // Make default first in list 
    316                                         $p = "\n\t<option style=\"padding-right: 10px;\" selected='selected' value='" . esc_attr( $option['value'] ) . "'>$label</option>"; 
    317                                     else 
    318                                         $r .= "\n\t<option style=\"padding-right: 10px;\" value='" . esc_attr( $option['value'] ) . "'>$label</option>"; 
     313                                    $value = esc_attr( $option['value'] ); 
     314                                     
     315                                    echo "<option style='padding-right: 10px;' value='$value'"; 
     316                                    if ( $selected == $option['value'] ) { 
     317                                        echo ' selected=selected '; 
     318                                    } 
     319                                    echo ">$label</option>"; 
    319320                                } 
    320                                 echo $p . $r; 
    321321                            ?> 
    322322                        </select> 
Note: See TracChangeset for help on using the changeset viewer.