Skip to content

Commit b4c36a7

Browse files
Merge pull request #2235 from Web3Auth/feat/no-default-logo
do not show a default logo if nothing is set on config
2 parents e0f2aea + d082f27 commit b4c36a7

File tree

1 file changed

+19
-13
lines changed
  • packages/modal/src/ui/components/Login

1 file changed

+19
-13
lines changed

packages/modal/src/ui/components/Login/Login.tsx

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -518,9 +518,14 @@ function Login(props: LoginProps) {
518518
);
519519
};
520520

521+
const headerLogo = [DEFAULT_LOGO_DARK, DEFAULT_LOGO_LIGHT].includes(appLogo) ? "" : appLogo;
522+
521523
const delimiter = (index: number) => {
522524
return (
523-
<div className="w3a--flex w3a--w-full w3a--items-center w3a--gap-x-2" key={`section-delimiter-${index}`}>
525+
<div
526+
className={cn("w3a--flex w3a--w-full w3a--items-center w3a--gap-x-2", headerLogo ? "w3a--my-2" : "w3a--my-4")}
527+
key={`section-delimiter-${index}`}
528+
>
524529
<div className="w3a--h-px w3a--w-full w3a--bg-app-gray-200 dark:w3a--bg-app-gray-500" />
525530
<p className="w3a--text-xs w3a--font-normal w3a--uppercase w3a--text-app-gray-400 dark:w3a--text-app-gray-400">or</p>
526531
<div className="w3a--h-px w3a--w-full w3a--bg-app-gray-200 dark:w3a--bg-app-gray-500" />
@@ -575,8 +580,6 @@ function Login(props: LoginProps) {
575580

576581
const expandedView = () => socialLoginSection(otherRow);
577582

578-
const headerLogo = [DEFAULT_LOGO_DARK, DEFAULT_LOGO_LIGHT].includes(appLogo) ? "" : appLogo;
579-
580583
return (
581584
<div className="w3a--flex w3a--flex-col w3a--items-center w3a--gap-y-4 w3a--p-2">
582585
<div
@@ -585,18 +588,21 @@ function Login(props: LoginProps) {
585588
logoAlignment === "center" ? "" : "w3a--w-full"
586589
)}
587590
>
588-
<figure
589-
className={cn(
590-
"w3a--mx-auto w3a--h-12 w3a--w-[200px]",
591-
logoAlignment === "center" ? "w3a--flex w3a--justify-center w3a--items-center" : "w3a--ml-0 w3a--w-auto"
592-
)}
593-
>
594-
<img src={headerLogo || getIcons(isDark ? "logo-w-dark" : "logo-w-light")} alt="Logo" className="w3a--size-full w3a--object-contain" />
595-
</figure>
591+
{headerLogo && (
592+
<figure
593+
className={cn(
594+
"w3a--mx-auto w3a--h-12 w3a--w-[200px]",
595+
logoAlignment === "center" ? "w3a--flex w3a--justify-center w3a--items-center" : "w3a--ml-0 w3a--w-auto"
596+
)}
597+
>
598+
<img src={headerLogo} alt="Logo" className="w3a--size-full w3a--object-contain" />
599+
</figure>
600+
)}
596601
<p
597602
className={cn(
598-
"w3a--text-lg w3a--font-semibold w3a--text-app-gray-900 dark:w3a--text-app-white",
599-
logoAlignment === "center" ? "w3a--text-center" : "w3a--text-left w3a--w-full w3a--ml-4"
603+
"w3a--text-app-gray-900 dark:w3a--text-app-white",
604+
logoAlignment === "center" ? "w3a--text-center" : "w3a--text-left w3a--w-full w3a--ml-4",
605+
headerLogo ? "w3a--text-lg w3a--font-semibold" : "w3a--text-3xl w3a--font-medium"
600606
)}
601607
>
602608
{t("modal.social.sign-in")}

0 commit comments

Comments
 (0)