Prefer existing framework/library abstractions for both types and layout styling to avoid redundant code and one-off visual tweaks.
Example:
"use client";
import Image, { type ImageProps } from "next/image";
type SidebarLogoProps = ImageProps; // reuse library type
export function SidebarLogo(props: SidebarLogoProps) {
return (
<div className="mb-2 md:mb-0"> {/* spacing controlled at container */}
<Image {...props} />
</div>
);
}
Apply this during review by checking: (1) are we duplicating types already provided by the dependency? (2) are spacing changes implemented in a maintainable place (container/layout + explicit responsive behavior) rather than ad hoc on an inner div?
Enter the URL of a public GitHub repository