Merge pull request #1358 from danskernesdigitalebibliotek/DDFLSBP-690…

…-remove-null-from-pickup-branch Updated the required check to display empty string instead of null.
danskernesdigitalebibliotek · Aug 28, 2024 · de95645 · de95645
2 parents 7e43bd7 + a18e89b
commit de95645
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/apps/patron-page/util/BranchesDropdown.tsx b/src/apps/patron-page/util/BranchesDropdown.tsx
@@ -28,7 +28,7 @@ const BranchesDropdown: FC<BranchesDropdownProps> = ({
         htmlFor="branches-dropdown"
         className="text-body-medium-medium mt-32 mb-8"
       >
-        {`${t("pickupBranchesDropdownLabelText")}${required ? "*" : null}`}
+        {`${t("pickupBranchesDropdownLabelText")}${required ? " *" : ""}`}
       </label>
       <div
         className={clsx(`dropdown mt-8 ${classNames || ""}`, [

diff --git a/src/components/atoms/input/TextInput.tsx b/src/components/atoms/input/TextInput.tsx
@@ -41,7 +41,7 @@ const TextInput: FC<TextInputProps> = ({
         { "dpl-input--invalid": !!validation }
       ])}
     >
-      <label htmlFor={id}>{required ? `${label}*` : label}</label>
+      <label htmlFor={id}>{required ? `${label} *` : label}</label>
       <input
         className="text-body-medium-medium"
         required={required}