Devolución TP 3

[andrezrv]

Muy bueno :) Hay algunos detalles a revisar, pero en general está muy bien.

Esto es lo que encontré para corregir:

• Para cargar estilos y scripts sería más ordenado hacerlo en archivos separados (por ejemplo, styles.php y scripts.php) en lugar de hacerlo desde init.php.
• Los estilos y scripts, en el caso del panel de administración, deberían cargarse usando la acción admin_enqueue_scripts (acá se está haciendo en equipo_init). De lo contrario WordPress muestra un error. Para poder ver los errores y alertas, asegurate de tener define( 'WP_DEBUG', true ); en el archivo wp-config.php.
• Ojo con lo de usar Bootstrap en el panel de administración, porque puede sobreescribir varios estilos que WordPress tiene por defecto, y no quedar del todo bien.
• No sé si entiendo el botón al final de la meta box; ¿tiene algún propósito específico?
• En el menú de settings, la nueva opción de color scheme no se está guardando. Eso es porque hay algunos errores en la función choose_color_callback(). La función correcta sería esta:

function choose_color_callback() {
    $settings = get_option( 'equipo_settings' );
    $color = isset( $settings['color'] ) ? $settings['color'] : '';

    ?>
        <select name="equipo_settings[color]">
            <option value="red" <?php selected( $color, 'red' ); ?> >Red</option>
            <option value="green" <?php selected( $color, 'green' ); ?>>Green</option>
            <option value="blue" <?php selected( $color, 'blue' ); ?>>Blue</option>
        </select>
    <?php
}

Notá estos cambios:

• El tag form ya no hace falta, porque ya lo abriste en equipo_settings_page_callback().
• El atributo name del tag select tiene el mismo nombre que la opción donde se guardan las settings del plugin ('equipo_settings'), y entre corchetes el nombre de la setting ('color’). Eso permite que WordPress guarde el valor dentro de un array cuando se actualice la opción 'equipo_settings'. Ese valor se puede recuperar de la manera que se está haciendo en la función al definir las variables $settings y $color.
• Notá cómo se usa la función selected() para marcar una opción como seleccionada. No es algo que estuviera en la consigna, pero te va a venir bien para ahorrar código en el futuro :)

[vdonna]

Ahh gracias, no sabia lo de styles.php y scripts.php, no tenia en claro como incluirlo.

Si vi que con bootstrap me modificaba el estilo de algunas partes, entonces no conviene utilizar bootstrap?

El botón tiene un propósito, que no logre que funcionara jajaja. Quería lograr que cuanto toquen ahí se pueda agregar otro perfil nuevo para completar, sin tener que poner add new y poder ir agregando todos los perfiles.

Sii esa era mi duda con lo del color, no me quedaba bien en claro como lograba guardarlo con settings.

Graciass!!

[andrezrv]

Conviene usar Bootstrap más que nada en la parte pública de WordPress, y lo ideal sería usarlo solamente en themes. Como el admin ya tiene sus estilos definidos, incluir Bootstrap ahí puede llegar a generar algunos conflictos visuales.

Con respecto a lo del botón, la idea es buena, pero lamentablemente no hay muchas maneras de escaparle al "Add new" en WordPress para crear nuevos posts :( (al menos no desde la interfaz de usuario).

[vdonna]

Geniaal graciass!!