aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPinapelz <yukais@pinapelz.com>2025-10-08 17:26:35 -0700
committerPinapelz <yukais@pinapelz.com>2025-10-08 17:26:35 -0700
commit7cc950fa7d3cd11ef643fda6174fe125c2bccc17 (patch)
tree08218e23d1654ae3eafd14165f6e4d696411d422
parent5155ef11630affd4220b30b3e835f96caa586d8f (diff)
frontend: remove others keys from well-defined score displays
-rw-r--r--frontend/src/components/displays/DancearoundScoreDisplay.tsx33
-rw-r--r--frontend/src/components/displays/DancerushScoreDisplay.tsx30
2 files changed, 3 insertions, 60 deletions
diff --git a/frontend/src/components/displays/DancearoundScoreDisplay.tsx b/frontend/src/components/displays/DancearoundScoreDisplay.tsx
index 26a54fa..a2d18f8 100644
--- a/frontend/src/components/displays/DancearoundScoreDisplay.tsx
+++ b/frontend/src/components/displays/DancearoundScoreDisplay.tsx
@@ -55,14 +55,13 @@ const DancearoundScoreDisplay: React.FC<ScoreDisplayProps> = ({
clear_status: "Status"
};
- const primaryKeys = ["title", "artist", "song"];
const mainStatKeys = [
"score",
"difficulty",
"lamp",
"diff_lamp",
];
- const expandableKeys = ["judgements", "optional"];
+ const expandableKeys = ["judgements", "optional", "clear_status"];
const localSkipKeys = ["num_players"]
// eslint-disable-next-line @typescript-eslint/no-explicit-any
const formatValue = (value: any, key: string): string => {
@@ -182,22 +181,12 @@ const DancearoundScoreDisplay: React.FC<ScoreDisplayProps> = ({
([key]) => !globalSkipKeys.includes(key) && !localSkipKeys.includes(key),
);
- const primary = entries.filter(([key]) => primaryKeys.includes(key));
const mainStats = entries.filter(([key]) => mainStatKeys.includes(key));
const expandable = entries.filter(([key]) => expandableKeys.includes(key));
- const others = entries.filter(
- ([key]) =>
- !primaryKeys.includes(key) &&
- !mainStatKeys.includes(key) &&
- !expandableKeys.includes(key) &&
- key !== "timestamp",
- );
return {
- primary,
mainStats,
expandable,
- others,
timestamp: score.timestamp,
};
};
@@ -280,7 +269,7 @@ const DancearoundScoreDisplay: React.FC<ScoreDisplayProps> = ({
<div className="grid grid-cols-1 md:grid-cols-2 lg:grid-cols-2 xl:grid-cols-3 gap-6">
{sortedScores.map((score, index) => {
// eslint-disable-next-line @typescript-eslint/no-unused-vars
- const { primary, mainStats, expandable, others, timestamp } =
+ const { mainStats, expandable, timestamp } =
getScoreEntries(score);
const chartIdHash = SHA1(`dancearound${score.title}${score.artist}`).toString();
return (
@@ -337,24 +326,6 @@ const DancearoundScoreDisplay: React.FC<ScoreDisplayProps> = ({
</div>
))}
- {/* Other fields */}
- {others.length > 0 && (
- <div className="mb-4">
- <div className="grid grid-cols-2 gap-2 text-sm">
- {others.map(([key, value]) => (
- <div key={key} className="flex justify-between">
- <span className="text-slate-400">
- {getDisplayName(key)}:
- </span>
- <span className="text-white font-medium">
- {renderValue(value, key)}
- </span>
- </div>
- ))}
- </div>
- </div>
- )}
-
{/* Timestamp */}
<div className="pt-4 border-t border-slate-800/50">
<p className="text-slate-500 text-xs">
diff --git a/frontend/src/components/displays/DancerushScoreDisplay.tsx b/frontend/src/components/displays/DancerushScoreDisplay.tsx
index 908880c..35a3968 100644
--- a/frontend/src/components/displays/DancerushScoreDisplay.tsx
+++ b/frontend/src/components/displays/DancerushScoreDisplay.tsx
@@ -55,7 +55,6 @@ const DancerushScoreDisplay: React.FC<ScoreDisplayProps> = ({
num_players: "Players"
};
- const primaryKeys = ["title", "artist", "song"];
const mainStatKeys = [
"score",
"difficulty",
@@ -186,22 +185,12 @@ const DancerushScoreDisplay: React.FC<ScoreDisplayProps> = ({
([key]) => !globalSkipKeys.includes(key),
);
- const primary = entries.filter(([key]) => primaryKeys.includes(key));
const mainStats = entries.filter(([key]) => mainStatKeys.includes(key));
const expandable = entries.filter(([key]) => expandableKeys.includes(key));
- const others = entries.filter(
- ([key]) =>
- !primaryKeys.includes(key) &&
- !mainStatKeys.includes(key) &&
- !expandableKeys.includes(key) &&
- key !== "timestamp",
- );
return {
- primary,
mainStats,
expandable,
- others,
timestamp: score.timestamp,
};
};
@@ -284,7 +273,7 @@ const DancerushScoreDisplay: React.FC<ScoreDisplayProps> = ({
<div className="grid grid-cols-1 md:grid-cols-2 lg:grid-cols-2 xl:grid-cols-3 gap-6">
{sortedScores.map((score, index) => {
// eslint-disable-next-line @typescript-eslint/no-unused-vars
- const { primary, mainStats, expandable, others, timestamp } =
+ const { mainStats, expandable, timestamp } =
getScoreEntries(score);
const chartIdHash = SHA1(`dancerush${score.title}${score.artist}`).toString();
return (
@@ -341,23 +330,6 @@ const DancerushScoreDisplay: React.FC<ScoreDisplayProps> = ({
</div>
))}
- {/* Other fields */}
- {others.length > 0 && (
- <div className="mb-4">
- <div className="grid grid-cols-2 gap-2 text-sm">
- {others.map(([key, value]) => (
- <div key={key} className="flex justify-between">
- <span className="text-slate-400">
- {getDisplayName(key)}:
- </span>
- <span className="text-white font-medium">
- {renderValue(value, key)}
- </span>
- </div>
- ))}
- </div>
- </div>
- )}
{/* Timestamp */}
<div className="pt-4 border-t border-slate-800/50">
send patches to the email below
yukais@pinapelz.com
include the subject [PATCH repo_name]
pinapelz.com
homepage