From dca870edcebbc2e2b28f26958cefa81dff727c31 Mon Sep 17 00:00:00 2001
From: NilsUeter <nils.ueter@gmail.com>
Date: Sun, 6 Apr 2025 11:01:31 +0200
Subject: [PATCH] Add possibility to print charts, sort units

---
 src/10th/Roster.jsx            |  54 +++---
 src/10th/ShortSummaryTable.jsx | 295 +++++++++++++++++++--------------
 src/index.css                  |   2 +
 3 files changed, 193 insertions(+), 158 deletions(-)

diff --git a/src/10th/Roster.jsx b/src/10th/Roster.jsx
index c011ae4..a9df9db 100644
--- a/src/10th/Roster.jsx
+++ b/src/10th/Roster.jsx
@@ -31,25 +31,14 @@ export const Roster = ({
 				position: "relative",
 			}}
 		>
-			<div
-				className="print-display-none flex justify-between"
-				style={{
-					backgroundColor: "var(--primary-color)",
-					color: "#fff",
-					padding: "4px 16px",
-					fontSize: " 1.7em",
-					fontWeight: "800",
-					textTransform: "uppercase",
-					marginBottom: "12px",
-				}}
-			>
-				<span>{name}</span>
-				<span>{cost.points} pts</span>
-			</div>
-
 			{forces.map((force, index) => (
 				<React.Fragment key={index}>
-					<ShortSummaryTable force={force} primaryColor={primaryColor} />
+					<ShortSummaryTable
+						force={force}
+						name={name}
+						points={cost.points}
+						primaryColor={primaryColor}
+					/>
 					<Force
 						force={force}
 						onePerPage={onePerPage}
@@ -64,13 +53,22 @@ export const Roster = ({
 const Force = ({ force, onePerPage, colorUserChoice }) => {
 	const { units, factionRules, rules, catalog } = force;
 	var mergedRules = new Map([...factionRules, ...rules]);
+	// units with role "Character" are sorted to the top, then "Battleline", then "Other"
+	const sortedUnits = units.sort((a, b) => {
+		if (a.role === "Character" && b.role !== "Character") return -1;
+		if (a.role !== "Character" && b.role === "Character") return 1;
+		if (a.role === "Battleline" && b.role !== "Battleline") return -1;
+		if (a.role !== "Battleline" && b.role === "Battleline") return 1;
+		return a.name.localeCompare(b.name);
+	});
+
 	return (
 		<div
 			style={{
 				display: "contents",
 			}}
 		>
-			{units.map((unit, index) => (
+			{sortedUnits.map((unit, index) => (
 				<Unit
 					key={unit.name + index}
 					index={index}
@@ -226,7 +224,7 @@ const Unit = ({
 					: "",
 			}}
 		>
-			<div className="flex justify-end gap-3">
+			<div className="flex justify-end gap-3 pb-0.5">
 				<label
 					className="print-display-none"
 					style={{
@@ -390,15 +388,13 @@ const Unit = ({
 					}}
 				>
 					{hasImage && <ImgEditor image={image} name={name} />}
-					<div className="absolute right-[1px] top-[2px] flex gap-1">
+					<div className="absolute right-[1px] top-[3px] flex items-center gap-1.5">
 						{hasImage && !bgRemoved && (
 							<button
-								className="button print-display-none"
+								className="button print-display-none border-none bg-[#f0f0f0e6] hover:bg-[#f0f0f0]"
 								style={{
-									border: "1px solid #999",
 									padding: "1px 4px",
 									fontSize: "0.8rem",
-									backgroundColor: "#f0f0f0",
 								}}
 								onClick={async () => {
 									// reset uploadRef
@@ -414,12 +410,10 @@ const Unit = ({
 						)}
 						{!hasImage && (
 							<a
-								className="button print-display-none"
+								className="button print-display-none border-none bg-[#f0f0f0e6] hover:bg-[#f0f0f0] hover:text-[#111113]"
 								style={{
-									border: "1px solid #999",
 									padding: "1px 4px",
 									fontSize: "0.8rem",
-									backgroundColor: "#f0f0f0",
 								}}
 								href={`https://www.google.com/search?udm=2&q=${name}+warhammer+40k+miniature`}
 								target="_blank"
@@ -428,12 +422,10 @@ const Unit = ({
 							</a>
 						)}
 						<label
-							className="button print-display-none"
+							className="button print-display-none border-none bg-[#f0f0f0e6] hover:bg-[#f0f0f0]"
 							style={{
-								border: "1px solid #999",
 								padding: "1px 4px",
 								fontSize: "0.8rem",
-								backgroundColor: "#f0f0f0",
 							}}
 						>
 							<input
@@ -465,12 +457,10 @@ const Unit = ({
 						</label>
 						{hasImage && (
 							<button
-								className="button print-display-none"
+								className="button print-display-none border-none bg-[#f0f0f0e6] hover:bg-[#f0f0f0]"
 								style={{
-									border: "1px solid #999",
 									padding: "1px 4px",
 									fontSize: "0.8rem",
-									backgroundColor: "#f0f0f0",
 								}}
 								onClick={() => {
 									setImage(undefined);
diff --git a/src/10th/ShortSummaryTable.jsx b/src/10th/ShortSummaryTable.jsx
index 12e79bb..ed41782 100644
--- a/src/10th/ShortSummaryTable.jsx
+++ b/src/10th/ShortSummaryTable.jsx
@@ -1,4 +1,4 @@
-import React from "react";
+import React, { useState } from "react";
 import { Bar } from "react-chartjs-2";
 import {
 	Chart as ChartJS,
@@ -20,7 +20,8 @@ ChartJS.register(
 	Title,
 );
 
-export const ShortSummaryTable = ({ force, primaryColor }) => {
+export const ShortSummaryTable = ({ force, primaryColor, name, points }) => {
+	const [hide, setHide] = useState(true);
 	const { units, factionRules, rules, catalog } = force;
 
 	const sortedUnits = units.sort((a, b) => {
@@ -104,140 +105,182 @@ export const ShortSummaryTable = ({ force, primaryColor }) => {
 		.sort((a, b) => a.save - b.save);
 
 	return (
-		<div
-			className="print-display-none my-4 -mt-4 flex flex-wrap items-start border-2 border-[var(--primary-color)]"
-			style={{
-				background: `url(${cardBackground})`,
-				backgroundSize: "cover",
-			}}
-		>
-			<div className="mt-4 table w-full border-collapse overflow-hidden md:w-[50%] md:flex-1">
-				<div className="table-header-group bg-[var(--primary-color)] font-bold uppercase text-white">
-					<div className="table-row">
-						<div
-							className="table-cell border border-[var(--primary-color)] px-4 py-1"
-							style={{
-								fontSize: "1.1em",
-								color: "#fff",
-								fontWeight: 600,
-							}}
-						>
-							Unit Name
-						</div>
-						<div
-							className="table-cell border border-[var(--primary-color)] px-4 py-1 text-right"
-							style={{
-								fontSize: "1.1em",
-								color: "#fff",
-								fontWeight: 600,
-							}}
-						>
-							Cost (pts)
-						</div>
-						<div
-							className="table-cell border border-[var(--primary-color)] px-4 py-1 text-right"
-							style={{
-								fontSize: "1.1em",
-								color: "#fff",
-								fontWeight: 600,
-							}}
-						>
-							Wounds
-						</div>
-					</div>
+		<>
+			<div className="flex justify-end gap-3">
+				<label
+					className="print-display-none"
+					style={{
+						display: "flex",
+						alignItems: "center",
+						justifyContent: "flex-end",
+						gap: 4,
+						userSelect: "none",
+					}}
+				>
+					<input type="checkbox" onChange={() => setHide(!hide)} />
+					<span className="print-display-none">Don't print this card.</span>
+				</label>
+			</div>
+			<div className={"flex flex-col " + (hide ? "print-display-none" : "")}>
+				<div
+					className="flex justify-between"
+					style={{
+						backgroundColor: "var(--primary-color)",
+						color: "#fff",
+						padding: "4px 16px",
+						fontSize: " 1.7em",
+						fontWeight: "800",
+						textTransform: "uppercase",
+					}}
+				>
+					<span>{name}</span>
+					<span>{points} pts</span>
 				</div>
-				<div className="table-row-group" style={{ lineHeight: 1.05 }}>
-					{sortedUnits.map((unit, index) => {
-						let { name, cost, models } = unit;
-						const count = models?.[0]?.count || 1;
-						return (
-							<div
-								key={name + index}
-								className={`table-row ${index % 2 === 0 ? "" : "bg-[#c4c4c480]"}`}
-							>
-								<div className="table-cell border border-dotted border-[#9e9fa1] px-4 py-1">
-									{count > 1 ? count + "x " : ""}
-									{name}
+				<div
+					className="mb-4 flex flex-wrap items-start border-2 border-[var(--primary-color)] print:flex-nowrap"
+					style={{
+						background: `url(${cardBackground})`,
+						backgroundSize: "cover",
+					}}
+				>
+					<div className="mt-4 table w-full border-collapse overflow-hidden md:w-[50%] md:flex-1">
+						<div className="table-header-group bg-[var(--primary-color)] font-bold uppercase text-white">
+							<div className="table-row">
+								<div
+									className="table-cell border border-[var(--primary-color)] px-4 py-1"
+									style={{
+										fontSize: "1.1em",
+										color: "#fff",
+										fontWeight: 600,
+									}}
+								>
+									Unit Name
 								</div>
-								<div className="table-cell border border-dotted border-[#9e9fa1] px-4 py-1 text-right">
-									{cost.points} pts
+								<div
+									className="table-cell border border-[var(--primary-color)] px-4 py-1 text-right"
+									style={{
+										fontSize: "1.1em",
+										color: "#fff",
+										fontWeight: 600,
+									}}
+								>
+									Wounds
 								</div>
-								<div className="table-cell border border-dotted border-[#9e9fa1] px-4 py-1 text-right">
-									{unit.modelStats?.reduce((sum, stat, index) => {
-										const count = unit.models?.[index]?.count || 1;
-										return sum + (stat.wounds || 0) * count;
-									}, 0)}
+								<div
+									className="table-cell border border-[var(--primary-color)] px-4 py-1 text-right"
+									style={{
+										fontSize: "1.1em",
+										color: "#fff",
+										fontWeight: 600,
+									}}
+								>
+									Cost (pts)
 								</div>
 							</div>
-						);
-					})}
-					<div className="table-row bg-[var(--primary-color)] font-bold text-white">
-						<div className="table-cell border border-[var(--primary-color)] px-4 py-1">
-							Total
 						</div>
-						<div className="table-cell border border-[var(--primary-color)] px-4 py-1 text-right">
-							{sortedUnits.reduce((sum, unit) => sum + unit.cost.points, 0)} pts
-						</div>
-						<div className="table-cell border border-[var(--primary-color)] px-4 py-1 text-right">
-							{totalWoundCount}
+						<div className="table-row-group" style={{ lineHeight: 1.05 }}>
+							{sortedUnits.map((unit, index) => {
+								let { name, cost, models } = unit;
+								const count = models?.[0]?.count || 1;
+								return (
+									<div
+										key={name + index}
+										className={`table-row ${index % 2 === 0 ? "" : "bg-[#c4c4c480]"}`}
+									>
+										<div className="table-cell border border-dotted border-[#9e9fa1] px-4 py-1">
+											{count > 1 ? count + "x " : ""}
+											{name}
+										</div>
+										<div className="table-cell border border-dotted border-[#9e9fa1] px-4 py-1 text-right">
+											{unit.modelStats?.reduce((sum, stat, index) => {
+												const count = unit.models?.[index]?.count || 1;
+												return sum + (stat.wounds || 0) * count;
+											}, 0)}
+										</div>
+										<div className="table-cell border border-dotted border-[#9e9fa1] px-4 py-1 text-right">
+											{cost.points} pts
+										</div>
+									</div>
+								);
+							})}
+							<div className="table-row bg-[var(--primary-color)] font-bold text-white">
+								<div className="table-cell border border-[var(--primary-color)] px-4 py-1">
+									Total
+								</div>
+								<div className="table-cell border border-[var(--primary-color)] px-4 py-1 text-right">
+									{totalWoundCount}
+								</div>
+								<div className="table-cell border border-[var(--primary-color)] px-4 py-1 text-right">
+									{sortedUnits.reduce((sum, unit) => sum + unit.cost.points, 0)}{" "}
+									pts
+								</div>
+							</div>
 						</div>
 					</div>
+
+					<div className="flex w-full flex-col gap-2.5 border-[var(--primary-color)] p-4 pb-2 pt-3.5 md:w-[50%] md:flex-1 md:border-l-2 print:w-[100%]">
+						<ChartComponent
+							data={{
+								labels: groupedChartDataMovement.map(
+									(data) => data.movement + '"',
+								),
+								datasets: [
+									{
+										label: "Total Cost (pts)",
+										data: groupedChartDataMovement.map(
+											(data) => data.totalPoints,
+										),
+										backgroundColor: primaryColor,
+										unitNames: groupedChartDataMovement.map(
+											(data) => data.unitNames,
+										),
+									},
+								],
+							}}
+							title="MOVEMENT OF YOUR UNITS"
+						/>
+
+						<ChartComponent
+							data={{
+								labels: groupedChartDataToughness.map(
+									(data) => data.toughness + "",
+								),
+								datasets: [
+									{
+										label: "Total Cost (pts)",
+										data: groupedChartDataToughness.map(
+											(data) => data.totalPoints,
+										),
+										backgroundColor: primaryColor,
+										unitNames: groupedChartDataToughness.map(
+											(data) => data.unitNames,
+										),
+									},
+								],
+							}}
+							title="TOUGHNESS OF YOUR UNITS"
+						/>
+
+						<ChartComponent
+							data={{
+								labels: groupedChartDataSave.map((data) => data.save + "+"),
+								datasets: [
+									{
+										label: "Total Cost (pts)",
+										data: groupedChartDataSave.map((data) => data.totalPoints),
+										backgroundColor: primaryColor,
+										unitNames: groupedChartDataSave.map(
+											(data) => data.unitNames,
+										),
+									},
+								],
+							}}
+							title="SAVE CHARACTERISTIC OF YOUR UNITS"
+						/>
+					</div>
 				</div>
 			</div>
-
-			<div className="flex w-full flex-col gap-2.5 border-[var(--primary-color)] p-4 md:w-[50%] md:flex-1 md:border-l-2">
-				<ChartComponent
-					data={{
-						labels: groupedChartDataMovement.map((data) => data.movement + '"'),
-						datasets: [
-							{
-								label: "Total Cost (pts)",
-								data: groupedChartDataMovement.map((data) => data.totalPoints),
-								backgroundColor: primaryColor,
-								unitNames: groupedChartDataMovement.map(
-									(data) => data.unitNames,
-								),
-							},
-						],
-					}}
-					title="MOVEMENT OF YOUR UNITS"
-				/>
-
-				<ChartComponent
-					data={{
-						labels: groupedChartDataToughness.map(
-							(data) => data.toughness + "",
-						),
-						datasets: [
-							{
-								label: "Total Cost (pts)",
-								data: groupedChartDataToughness.map((data) => data.totalPoints),
-								backgroundColor: primaryColor,
-								unitNames: groupedChartDataToughness.map(
-									(data) => data.unitNames,
-								),
-							},
-						],
-					}}
-					title="TOUGHNESS OF YOUR UNITS"
-				/>
-
-				<ChartComponent
-					data={{
-						labels: groupedChartDataSave.map((data) => data.save + "+"),
-						datasets: [
-							{
-								label: "Total Cost (pts)",
-								data: groupedChartDataSave.map((data) => data.totalPoints),
-								backgroundColor: primaryColor,
-								unitNames: groupedChartDataSave.map((data) => data.unitNames),
-							},
-						],
-					}}
-					title="SAVE CHARACTERISTIC OF YOUR UNITS"
-				/>
-			</div>
-		</div>
+		</>
 	);
 };
 
diff --git a/src/index.css b/src/index.css
index 4d1d784..b55ad7f 100644
--- a/src/index.css
+++ b/src/index.css
@@ -69,6 +69,8 @@
 
 	button,
 	.button {
+		display: flex;
+		align-items: center;
 		background-color: #f0f0f0;
 		color: #111113;
 		border-radius: 8px;