Fix MaskedAvatar and PublisherAvatar sizing issues, improved loading & fallback states (#5238)

## Problem solved

Short description of the bug fixed or feature added

<!-- start pr-codex -->

---

## PR-Codex overview
This PR primarily focuses on updating the `PublisherAvatar` component usage across several files, replacing the `boxSize` prop with a `className` prop for styling. Additionally, it enhances the `Img` component with improved loading behavior.

### Detailed summary
- Replaced `boxSize` with `className` in `PublisherAvatar` components in:
  - `apps/dashboard/src/components/explore/publisher/index.tsx`
  - `apps/dashboard/src/pages/profile/[profileAddress].tsx`
  - `apps/dashboard/src/components/contract-components/publisher/publisher-header.tsx`
  - `apps/dashboard/src/pages/community/ambassadors.tsx`
- Updated `MaskedAvatar` component to use `className` instead of `boxSize`.
- Modified `MaskedAvatar` component's rendering logic to use a new `Img` component.
- Enhanced `Img` component with a loading state and error handling using a `ref`.

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

<!-- end pr-codex -->

Author: MananTank

apps/dashboard/src/@/components/blocks/Img.tsx

@@ -1,6 +1,7 @@
 /* eslint-disable @next/next/no-img-element */
 "use client";
-import { useState } from "react";
+import { useRef, useState } from "react";
+import { useIsomorphicLayoutEffect } from "../../lib/useIsomorphicLayoutEffect";
 import { cn } from "../../lib/utils";
 
 type imgElementProps = React.DetailedHTMLProps<
@@ -25,19 +26,39 @@ export function Img(props: imgElementProps) {
   const { className, fallback, skeleton, ...restProps } = props;
   const defaultSkeleton = <div className="animate-pulse bg-accent" />;
   const defaultFallback = <div className="bg-muted" />;
+  const imgRef = useRef<HTMLImageElement>(null);
+
+  useIsomorphicLayoutEffect(() => {
+    const imgEl = imgRef.current;
+    if (!imgEl) {
+      return;
+    }
+    if (imgEl.complete) {
+      setStatus("loaded");
+    } else {
+      function handleLoad() {
+        setStatus("loaded");
+      }
+      imgEl.addEventListener("load", handleLoad);
+      return () => {
+        imgEl.removeEventListener("load", handleLoad);
+      };
+    }
+  }, []);
 
   return (
     <div className="relative">
       <img
         {...restProps}
-        onLoad={() => {
-          setStatus("loaded");
-        }}
+        // avoid setting empty src string to prevent request to the entire page
+        src={restProps.src || undefined}
+        ref={imgRef}
         onError={() => {
           setStatus("fallback");
         }}
         style={{
           opacity: status === "loaded" ? 1 : 0,
+          ...restProps.style,
         }}
         alt={restProps.alt || ""}
         className={cn(
@@ -48,6 +69,7 @@ export function Img(props: imgElementProps) {
 
       {status !== "loaded" && (
         <div
+          style={restProps.style}
           className={cn(
             "fade-in-0 absolute inset-0 overflow-hidden transition-opacity duration-300 [&>*]:h-full [&>*]:w-full",
             className,

apps/dashboard/src/components/contract-components/publisher/publisher-header.tsx

@@ -54,7 +54,7 @@ export const PublisherHeader: React.FC<PublisherHeaderProps> = ({
             >
               <PublisherAvatar
                 alt="Publisher avatar"
-                boxSize={14}
+                className="size-14"
                 address={ensQuery.data?.ensName || wallet}
               />
             </Link>

apps/dashboard/src/components/explore/publisher/index.tsx

@@ -30,7 +30,7 @@ export const ContractPublisher: React.FC<ContractPublisherProps> = async ({
     >
       <PublisherAvatar
         isPending={false}
-        boxSize={5}
+        className="size-5"
         address={addressOrEns || ""}
       />
 

apps/dashboard/src/components/hackathon/Judges.tsx

@@ -35,7 +35,7 @@ export const Judges = ({ TRACKING_CATEGORY }: JudgesProps) => {
   ];
 
   return (
-    <div className="relative flex flex-col gap-8">
+    <div className="container relative flex max-w-[1200px] flex-col gap-8">
       <Heading size="title.2xl">Judges</Heading>
       <SimpleGrid
         columns={{ base: 1, lg: 4 }}
@@ -46,10 +46,9 @@ export const Judges = ({ TRACKING_CATEGORY }: JudgesProps) => {
         {judges.map((judge) => (
           <Flex key={judge.name} flexDir="column" gap={2} alignItems="center">
             <MaskedAvatar
-              boxSize={40}
               src={judge.image}
               alt={judge.name}
-              className="rounded-full"
+              className="size-52"
             />
             <Heading size="title.sm" mt={4} textAlign="center">
               {judge.name}

apps/dashboard/src/components/hackathon/JudgesEarn.tsx

@@ -27,7 +27,7 @@ export const JudgesEarn = () => {
   ];
 
   return (
-    <div className="relative flex flex-col gap-8">
+    <div className="container relative flex max-w-[1200px] flex-col gap-8">
       <Heading size="title.2xl">Judges</Heading>
       <SimpleGrid
         columns={{ base: 1, lg: 4 }}
@@ -38,10 +38,9 @@ export const JudgesEarn = () => {
         {judges.map((judge) => (
           <Flex key={judge.name} flexDir="column" gap={2} alignItems="center">
             <MaskedAvatar
-              boxSize={40}
               src={judge.image}
               alt={judge.name}
-              className="rounded-full"
+              className="size-52"
             />
             <Heading size="title.sm" mt={4} textAlign="center">
               {judge.name}

apps/dashboard/src/pages/community/ambassadors.tsx

@@ -599,7 +599,7 @@ const Ambassadors: ThirdwebNextPage = () => {
                   <MaskedAvatar
                     src={ambassador.profileImage}
                     alt={ambassador.name}
-                    boxSize={20}
+                    className="size-20"
                   />
                   <Heading size="title.sm">{ambassador.name}</Heading>
                 </Flex>
@@ -619,7 +619,7 @@ const Ambassadors: ThirdwebNextPage = () => {
           >
             <Flex flexDir="column" gap={0}>
               <Heading as="h2" size="display.md" mb={4} textAlign="center">
-                Ready to
+                Ready to{" "}
                 <Heading
                   size="display.md"
                   as="span"

apps/dashboard/src/pages/profile/[profileAddress].tsx

@@ -116,7 +116,7 @@ const UserPage: ThirdwebNextPage = (props: UserPageProps) => {
             <Flex gap={{ base: 4, md: 8 }} align="center" w="full">
               <PublisherAvatar
                 address={ens.data?.ensName || props.profileAddress}
-                boxSize={30}
+                className="size-30"
               />
               <Flex direction="column" gap={0}>
                 <Heading

apps/dashboard/src/tw-components/masked-avatar/index.tsx

@@ -1,36 +1,30 @@
-import { SkeletonContainer } from "@/components/ui/skeleton";
+import { Img } from "@/components/blocks/Img";
 import { cn } from "@/lib/utils";
 import hexagon from "./hexagon.png";
 
 export interface MaskedAvatarProps {
   src: string;
   isPending?: boolean;
   alt?: string;
-  boxSize?: number;
   className?: string;
 }
 
 export const MaskedAvatar: React.FC<MaskedAvatarProps> = ({
   src,
   alt,
-  boxSize = 12,
   className,
 }) => {
   return (
-    <SkeletonContainer
-      className={cn(className, `size-${boxSize}`)}
+    <Img
+      className={cn("size-12 object-cover", className)}
+      src={src}
+      alt={alt}
       style={{
         WebkitMaskImage: `url("${hexagon.src}")`,
         WebkitMaskSize: "cover",
         mask: `url("${hexagon.src}")`,
         maskSize: "cover",
       }}
-      skeletonData={undefined}
-      loadedData={src}
-      render={(v) => (
-        // eslint-disable-next-line @next/next/no-img-element
-        <img className="object-cover" src={v} alt={alt || ""} />
-      )}
     />
   );
 };