fix(sprint-6/etapa-2): mover gráficos antes de ComparisonTable en tab ROI
Some checks failed
/ Deploy to Cloudflare Pages (push) Has been cancelled
Some checks failed
/ Deploy to Cloudflare Pages (push) Has been cancelled
SensitivityPanel y CumulativeSavingsChart deben ser adyacentes: el slider afecta el gráfico (doble línea) y el usuario no debe scrollear a través de la tabla de comparación para ver el efecto. Reordenamiento de JSX puro, sin cambios de lógica. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -545,20 +545,8 @@ export function ReportPage() {
|
||||
onReset={() => { setVolumeAdjust(0); setInvestmentAdjust(0) }}
|
||||
/>
|
||||
|
||||
{/* Tabla de comparación actividad por actividad — per-ejecución, no varía con los sliders */}
|
||||
<div className="mb-6">
|
||||
<h2 className="text-base font-semibold text-slate-800 mb-4">
|
||||
Comparación por actividad
|
||||
</h2>
|
||||
<ComparisonTable
|
||||
perActivityActual={result.perActivity}
|
||||
perActivityAutomated={resultAutomated!.perActivity}
|
||||
automatableIds={automatableIds}
|
||||
currency={process.currency}
|
||||
/>
|
||||
</div>
|
||||
|
||||
{/* Gráficos: ahorro acumulado + top ahorros */}
|
||||
{/* Gráficos: ahorro acumulado + top ahorros — adyacente al SensitivityPanel
|
||||
para que el efecto de los sliders sea visible sin scrollear */}
|
||||
<div className="grid grid-cols-1 lg:grid-cols-2 gap-4 mb-6">
|
||||
<div className="bg-white rounded-xl border border-slate-200 shadow-sm p-5">
|
||||
<h3 className="text-xs font-semibold text-slate-600 uppercase tracking-wide mb-4">
|
||||
@@ -584,6 +572,19 @@ export function ReportPage() {
|
||||
</div>
|
||||
</div>
|
||||
|
||||
{/* Tabla de comparación actividad por actividad — per-ejecución, no varía con los sliders */}
|
||||
<div className="mb-6">
|
||||
<h2 className="text-base font-semibold text-slate-800 mb-4">
|
||||
Comparación por actividad
|
||||
</h2>
|
||||
<ComparisonTable
|
||||
perActivityActual={result.perActivity}
|
||||
perActivityAutomated={resultAutomated!.perActivity}
|
||||
automatableIds={automatableIds}
|
||||
currency={process.currency}
|
||||
/>
|
||||
</div>
|
||||
|
||||
{/* Transparencia metodológica (condicional) */}
|
||||
<div className="mb-6">
|
||||
<TransparencySection
|
||||
|
||||
Reference in New Issue
Block a user