From a963caaf5bca971162c82fb330ad7f0696f44861 Mon Sep 17 00:00:00 2001
From: Martin <2026226+martin-mfg@users.noreply.github.com>
Date: Mon, 27 Apr 2026 08:34:56 +0200
Subject: [PATCH] small refactoring: use constants in layout logic (#186)

---
 packages/core/src/cards/gist.js |  7 +++++--
 packages/core/src/cards/repo.js | 20 +++++++++++++-------
 2 files changed, 18 insertions(+), 9 deletions(-)

diff --git a/packages/core/src/cards/gist.js b/packages/core/src/cards/gist.js
index b630dad8..c8d6437d 100644
--- a/packages/core/src/cards/gist.js
+++ b/packages/core/src/cards/gist.js
@@ -16,6 +16,7 @@ import {
 
 const ICON_SIZE = 16;
 const CARD_DEFAULT_WIDTH = 400;
+const X_OFFSET = 25;
 const HEADER_MAX_LENGTH = 35;
 
 /**
@@ -62,7 +63,9 @@ const renderGistCard = (gistData, options = {}) => {
   const multiLineDescription = wrapTextMultiline(desc, lineWidth, linesLimit);
   const descriptionLines = multiLineDescription.length;
   const descriptionSvg = multiLineDescription
-    .map((line) => `<tspan dy="1.2em" x="25">${encodeHTML(line)}</tspan>`)
+    .map(
+      (line) => `<tspan dy="1.2em" x="${X_OFFSET}">${encodeHTML(line)}</tspan>`,
+    )
     .join("");
 
   const lineHeight = descriptionLines > 3 ? 12 : 10;
@@ -128,7 +131,7 @@ const renderGistCard = (gistData, options = {}) => {
   card.setHideBorder(hide_border);
 
   return card.render(`
-    <text class="description" x="25" y="-5">
+    <text class="description" x="${X_OFFSET}" y="-5">
         ${descriptionSvg}
     </text>
 
diff --git a/packages/core/src/cards/repo.js b/packages/core/src/cards/repo.js
index 6b1afb50..d5bab6f1 100644
--- a/packages/core/src/cards/repo.js
+++ b/packages/core/src/cards/repo.js
@@ -19,6 +19,8 @@ import { createTextNode } from "./stats.js";
 
 const ICON_SIZE = 16;
 const DESCRIPTION_LINE_WIDTH = 59;
+const CARD_DEFAULT_WIDTH = 400;
+const X_OFFSET = 25;
 const DESCRIPTION_MAX_LINES = 3;
 
 /**
@@ -96,8 +98,8 @@ const renderRepoCard = (repo, options = {}) => {
     card_width_input && !isNaN(card_width_input)
       ? card_width_input
       : show.length >= 2
-        ? 430
-        : 400;
+        ? CARD_DEFAULT_WIDTH + 30
+        : CARD_DEFAULT_WIDTH;
 
   const i18n = new I18n({
     locale,
@@ -182,7 +184,9 @@ const renderRepoCard = (repo, options = {}) => {
   const desc = parseEmojis(description || "No description provided");
   const multiLineDescription = wrapTextMultiline(
     desc,
-    Math.round((card_width - 400) / 5.93 + DESCRIPTION_LINE_WIDTH),
+    Math.round(
+      (card_width - CARD_DEFAULT_WIDTH) / 5.93 + DESCRIPTION_LINE_WIDTH,
+    ),
     descriptionMaxLines,
   );
   const descriptionLinesCount = description_lines_count
@@ -190,7 +194,9 @@ const renderRepoCard = (repo, options = {}) => {
     : multiLineDescription.length;
 
   const descriptionSvg = multiLineDescription
-    .map((line) => `<tspan dy="1.2em" x="25">${encodeHTML(line)}</tspan>`)
+    .map(
+      (line) => `<tspan dy="1.2em" x="${X_OFFSET}">${encodeHTML(line)}</tspan>`,
+    )
     .join("");
 
   const extraHeight = Object.keys(STATS).length
@@ -298,19 +304,19 @@ const renderRepoCard = (repo, options = {}) => {
           getBadgeSVG(
             i18n.t("repocard.template"),
             colors.textColor,
-            card_width - 400,
+            card_width - CARD_DEFAULT_WIDTH,
           )
         : isArchived
           ? // @ts-ignore
             getBadgeSVG(
               i18n.t("repocard.archived"),
               colors.textColor,
-              card_width - 400,
+              card_width - CARD_DEFAULT_WIDTH,
             )
           : ""
     }
 
-    <text class="description" x="25" y="-5">
+    <text class="description" x="${X_OFFSET}" y="-5">
       ${descriptionSvg}
     </text>