🎨 (core): Retoques de estilo menores

src/core.rs

@@ -109,7 +109,7 @@ impl TypeInfo {
 
 /// Proporciona información de tipo en tiempo de ejecución y conversión dinámica de tipos.
 ///
-/// Este *trait* se implementa automáticamente para **todos** los tipos que implementen [`Any`], de
+/// Este trait se implementa automáticamente para **todos** los tipos que implementen [`Any`], de
 /// modo que basta con traer [`AnyInfo`] al ámbito (`use crate::AnyInfo;`) para disponer de estos
 /// métodos adicionales, o usar el [`prelude`](crate::prelude) de PageTop.
 ///

src/core/component/definition.rs

@@ -18,7 +18,7 @@ pub trait ComponentClone {
 
 /// Define la función de renderizado para todos los componentes.
 ///
-/// Este *trait* se implementa automáticamente en cualquier tipo (componente) que implemente
+/// Este trait se implementa automáticamente en cualquier tipo (componente) que implemente
 /// [`Component`], por lo que no requiere ninguna codificación manual.
 #[async_trait]
 pub trait ComponentRender {

src/core/extension/definition.rs

@@ -8,7 +8,7 @@ use crate::web::Router;
 
 /// Interfaz común que debe implementar cualquier extensión de PageTop.
 ///
-/// Este *trait* es fácil de implementar, basta con declarar una estructura sin campos para la
+/// Este trait es fácil de implementar, basta con declarar una estructura sin campos para la
 /// extensión y sobrescribir los métodos que sean necesarios. Por ejemplo:
 ///
 /// ```rust,no_run

src/core/theme/definition.rs

@@ -19,7 +19,7 @@ use crate::web::http::StatusCode;
 /// error. El contenido de cada región depende del [`Context`](crate::core::component::Context) y de
 /// su nombre lógico.
 ///
-/// Todos los métodos de este *trait* tienen una implementación por defecto, por lo que pueden
+/// Todos los métodos de este trait tienen una implementación por defecto, por lo que pueden
 /// sobrescribirse selectivamente para crear nuevos temas con comportamientos distintos a los
 /// predeterminados.
 ///

src/html/props.rs

@@ -81,9 +81,9 @@ pub enum PropsError {
 ///
 /// Las variantes `*Extra` permiten añadir valores tipados usando una clave. Están pensadas para
 /// ampliar el comportamiento de componentes ya existentes. Como no es posible añadir campos a la
-/// estructura de un componente ya definido, temas y extensiones pueden definir un *trait* con
-/// nuevos métodos que leen y escriben valores extra en [`Props`]. Esos valores se interpretan como
-/// si fueran valores internos del componente para tomar decisiones durante el renderizado.
+/// estructura de un componente ya definido, temas y extensiones pueden definir un trait con nuevos
+/// métodos que leen y escriben valores extra en [`Props`]. Esos valores se interpretan como si
+/// fueran valores internos del componente para tomar decisiones durante el renderizado.
 #[derive(Clone, Debug)]
 pub enum PropsOp {
     /// Establece el identificador del componente normalizando el valor: recorta espacios, convierte
@@ -374,7 +374,7 @@ impl PropsOp {
 /// Las variantes [`SetExtra`](PropsOp::SetExtra) y [`RemoveExtra`](PropsOp::RemoveExtra), usando
 /// [`PropsOp::set_extra()`] y [`PropsOp::remove_extra()`] respectivamente, permiten adjuntar
 /// valores tipados a un `Props`. Son útiles para que temas y extensiones amplíen el comportamiento
-/// de componentes ya existentes mediante *traits* con nuevos métodos que lean y escriban esos
+/// de componentes ya existentes mediante traits con nuevos métodos que lean y escriban esos
 /// valores.
 ///
 /// ```rust

src/lib.rs

@@ -125,8 +125,8 @@ use std::ops::Deref;
 /// referencia a la versión del *crate* que lo usa.
 pub const PAGETOP_VERSION: &str = env!("CARGO_PKG_VERSION");
 
-/// Re-exporta el atributo [`async_trait`](https://docs.rs/async-trait) para implementar *traits*
-/// con métodos `async`.
+/// Re-exporta el atributo [`async_trait`](https://docs.rs/async-trait) para implementar traits con
+/// métodos `async`.
 ///
 /// Si en el ámbito se declara `use pagetop::prelude::*` o `use pagetop::async_trait;` basta con
 /// usar el nombre corto `#[async_trait]`. Otra opción es usar su forma cualificada

src/locale/definition.rs

@@ -15,7 +15,7 @@ static FALLBACK_LANGID: LazyLock<LanguageIdentifier> = LazyLock::new(|| langid!(
 
 /// Representa el identificador de idioma [`LanguageIdentifier`] asociado a un recurso.
 ///
-/// Este *trait* permite que distintas estructuras expongan su idioma de forma uniforme. Las
+/// Este trait permite que distintas estructuras expongan su idioma de forma uniforme. Las
 /// implementaciones deben garantizar que siempre se devuelve un identificador de idioma válido. Si
 /// el recurso no tiene uno asignado, se puede devolver, si procede, el identificador de idioma por
 /// defecto de la aplicación ([`Locale::default_langid()`]).

src/locale/request.rs

@@ -10,7 +10,7 @@ use super::{LangId, LanguageIdentifier, Locale};
 /// `?lang=...`. El comportamiento concreto depende de la política global [`LangNegotiation`]
 /// configurada en la aplicación.
 ///
-/// El idioma resultante se expone a través del *trait* [`LangId`], de modo que pueda usarse
+/// El idioma resultante se expone a través del trait [`LangId`], de modo que pueda usarse
 /// [`RequestLocale`] como cualquier otra fuente de idioma en PageTop.
 ///
 /// [`LangNegotiation`]: crate::global::LangNegotiation

tests/component_children.rs

@@ -37,9 +37,10 @@ impl TestComp {
 
     /// Creates a component with no id, with fixed output text.
     fn text(text: &str) -> Self {
-        let mut c = Self::default();
-        c.text = text.to_string();
-        c
+        Self {
+            text: text.to_string(),
+            ..Default::default()
+        }
     }
 }
 

tests/component_template.rs

@@ -31,7 +31,7 @@ impl Theme for MarkerTheme {
         component: &mut dyn Component,
         _cx: &mut Context,
     ) -> Option<Result<Markup, ComponentError>> {
-        let template = (&*component).downcast_ref::<layout::Template>()?;
+        let template = (*component).downcast_ref::<layout::Template>()?;
         template.template().downcast_ref::<CoreTemplates>()?;
         Some(Ok(html! { "marker-template-output" }))
     }

tests/html_props_extras.rs

@@ -5,7 +5,7 @@ use pagetop::prelude::*;
 #[pagetop::test]
 async fn set_and_read_extra() {
     let props = Props::default().with_prop(PropsOp::set_extra("ext.flag", true));
-    assert_eq!(*props.extra::<bool>("ext.flag").unwrap(), true);
+    assert!(*props.extra::<bool>("ext.flag").unwrap());
 }
 
 #[pagetop::test]
@@ -52,13 +52,13 @@ async fn extra_type_mismatch() {
 #[pagetop::test]
 async fn extra_or_returns_value_when_found() {
     let props = Props::default().with_prop(PropsOp::set_extra("ext.flag", true));
-    assert_eq!(props.extra_or("ext.flag", false), true);
+    assert!(props.extra_or("ext.flag", false));
 }
 
 #[pagetop::test]
 async fn extra_or_returns_default_on_missing() {
     let props = Props::default();
-    assert_eq!(props.extra_or("ext.flag", false), false);
+    assert!(!props.extra_or("ext.flag", false));
 }
 
 #[pagetop::test]
@@ -72,7 +72,7 @@ async fn extra_or_returns_default_on_type_mismatch() {
 #[pagetop::test]
 async fn extra_or_default_returns_type_default_on_missing() {
     let props = Props::default();
-    assert_eq!(props.extra_or_default::<bool>("ext.flag"), false);
+    assert!(!props.extra_or_default::<bool>("ext.flag"));
     assert_eq!(props.extra_or_default::<i32>("ext.count"), 0);
 }
 
@@ -93,7 +93,7 @@ async fn extra_or_else_calls_closure_on_missing() {
 async fn clone_preserves_extras() {
     let props = Props::default().with_prop(PropsOp::set_extra("ext.flag", true));
     let cloned = props.clone();
-    assert_eq!(*cloned.extra::<bool>("ext.flag").unwrap(), true);
+    assert!(*cloned.extra::<bool>("ext.flag").unwrap());
 }
 
 #[pagetop::test]
@@ -103,7 +103,7 @@ async fn multiple_extras_with_different_types() {
         .with_prop(PropsOp::set_extra("ext.count", 42_u32))
         .with_prop(PropsOp::set_extra("ext.label", "hello".to_string()));
 
-    assert_eq!(*props.extra::<bool>("ext.flag").unwrap(), true);
+    assert!(*props.extra::<bool>("ext.flag").unwrap());
     assert_eq!(*props.extra::<u32>("ext.count").unwrap(), 42);
     assert_eq!(props.extra::<String>("ext.label").unwrap(), "hello");
 }