From d656c9f87541efe4966b6dddb4128bec6d77b3f6 Mon Sep 17 00:00:00 2001 From: Kalle <38327916+Sendouc@users.noreply.github.com> Date: Sat, 20 Jun 2026 15:03:46 +0300 Subject: [PATCH] Fix analyzer table alignment + row height not being equal --- .../build-analyzer/routes/analyzer.module.css | 6 ++++ .../build-analyzer/routes/analyzer.tsx | 32 ++++++++++--------- 2 files changed, 23 insertions(+), 15 deletions(-) diff --git a/app/features/build-analyzer/routes/analyzer.module.css b/app/features/build-analyzer/routes/analyzer.module.css index 3f3b2593d..f7f4c10a6 100644 --- a/app/features/build-analyzer/routes/analyzer.module.css +++ b/app/features/build-analyzer/routes/analyzer.module.css @@ -255,6 +255,12 @@ align-items: center; } +/* doubled class to outweigh the Table cell padding rule so the popover + button does not make these rows taller than the text-only rows */ +.popoverCell.popoverCell { + padding-block: 0; +} + .statPopover { min-width: 360px; diff --git a/app/features/build-analyzer/routes/analyzer.tsx b/app/features/build-analyzer/routes/analyzer.tsx index bdff6f6b6..a49195bb6 100644 --- a/app/features/build-analyzer/routes/analyzer.tsx +++ b/app/features/build-analyzer/routes/analyzer.tsx @@ -1009,7 +1009,7 @@ function StatChartPopover(props: StatChartProps) { } /> } @@ -1643,19 +1643,21 @@ function DamageTable({ return ( - - {damageIsSubWeaponDamage(val) ? ( - - ) : null}{" "} - {t(typeRowName as any)}{" "} - {damageIsSubWeaponDamage(val) && val.type === "SPLASH" ? ( - <>({t("analyzer:damage.SPLASH")}) - ) : null} + +
+ {damageIsSubWeaponDamage(val) ? ( + + ) : null}{" "} + {t(typeRowName as any)}{" "} + {damageIsSubWeaponDamage(val) && val.type === "SPLASH" ? ( + <>({t("analyzer:damage.SPLASH")}) + ) : null} +
{showDistanceColumn && ( @@ -1679,7 +1681,7 @@ function DamageTable({ )} {showPopovers ? ( - + {renderPopover(val, (val as SubWeaponDamage).subWeaponId) ? (