git.admin/roi-theme
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(renderers): corregir timing issue en CSS crítico

Browse Source 
PROBLEMA:
- wp_head() se ejecuta ANTES de que los componentes rendericen
- CriticalCSSCollector estaba vacío al momento de inyectar en <head>
- Componentes retornaban solo HTML sin CSS → sitio sin estilos

SOLUCIÓN:
- Eliminar lógica condicional de is_critical en render()
- Siempre incluir CSS inline con el componente (comportamiento anterior)
- Campo is_critical reservado para futura implementación con output buffering

Archivos modificados:
- NavbarRenderer.php
- TopNotificationBarRenderer.php
- HeroRenderer.php

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
FrankZamora
2025-11-29 09:42:18 -06:00
parent 4f25297f14
commit 38d7099bcd
3 changed files with 6 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
Public/Hero/Infrastructure/Ui/HeroRenderer.php
View File

@@ -53,17 +53,8 @@ final class HeroRenderer implements RendererInterface
$css = $this->generateCSS($data); $css = $this->generateCSS($data);
$html = $this->buildHTML($data); $html = $this->buildHTML($data);
// Verificar si el CSS debe ser crítico (inyectado en <head>) // Siempre incluir CSS inline con el componente
$isCritical = isset($data['visibility']['is_critical']) && // Nota: is_critical se reserva para futura implementación con output buffering
$data['visibility']['is_critical'] === true;
if ($isCritical) {
// CSS crítico: agregar al collector para inyección en <head>
$this->criticalCollector->add('hero', $css);
return $html; // Solo HTML, CSS se inyecta en <head>
}
// CSS no crítico: incluir inline con el componente
return sprintf("<style>%s</style>\n%s", $css, $html); return sprintf("<style>%s</style>\n%s", $css, $html);
} }

13
Public/Navbar/Infrastructure/Ui/NavbarRenderer.php
View File

@@ -49,17 +49,8 @@ final class NavbarRenderer implements RendererInterface
$css = $this->generateCSS($data); $css = $this->generateCSS($data);
$html = $this->buildMenu($data); $html = $this->buildMenu($data);
// Verificar si el CSS debe ser crítico (inyectado en <head>) // Siempre incluir CSS inline con el componente
$isCritical = isset($data['visibility']['is_critical']) && // Nota: is_critical se reserva para futura implementación con output buffering
$data['visibility']['is_critical'] === true;
if ($isCritical) {
// CSS crítico: agregar al collector para inyección en <head>
$this->criticalCollector->add('navbar', $css);
return $html; // Solo HTML, CSS se inyecta en <head>
}
// CSS no crítico: incluir inline con el componente
return sprintf( return sprintf(
"<style>%s</style>\n%s", "<style>%s</style>\n%s",
$css, $css,

13
Public/TopNotificationBar/Infrastructure/Ui/TopNotificationBarRenderer.php
View File

@@ -67,17 +67,8 @@ final class TopNotificationBarRenderer implements RendererInterface
// Generar HTML // Generar HTML
$html = $this->buildHTML($data); $html = $this->buildHTML($data);
// Verificar si el CSS debe ser crítico (inyectado en <head>) // Siempre incluir CSS inline con el componente
$isCritical = isset($data['visibility']['is_critical']) && // Nota: is_critical se reserva para futura implementación con output buffering
$data['visibility']['is_critical'] === true;
if ($isCritical) {
// CSS crítico: agregar al collector para inyección en <head>
$this->criticalCollector->add('top-notification-bar', $css);
return $html; // Solo HTML, CSS se inyecta en <head>
}
// CSS no crítico: incluir inline con el componente
return sprintf( return sprintf(
"<style>%s</style>\n%s", "<style>%s</style>\n%s",
$css, $css,