-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: es-hangul을 사용하는 회사, 프로젝트 들의 로고를 나열할 수 있는 컴포넌트를 만듭니다. #100
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
import { useIsDarkMode } from '@/hooks/use-is-dark-mode'; | ||
import { ImgHTMLAttributes } from 'react'; | ||
|
||
interface Props extends Omit<ImgHTMLAttributes<HTMLImageElement>, 'src'> { | ||
lightSrc: string; | ||
darkSrc: string; | ||
} | ||
|
||
export function AdoptorImage(props: Props) { | ||
const isDarkMode = useIsDarkMode(); | ||
return ( | ||
<img | ||
src={isDarkMode ? props.darkSrc : props.lightSrc} | ||
{...props} | ||
style={{ | ||
height: 80, | ||
}} | ||
/> | ||
); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nextra가 dark, light 모드를 제공해서 하얀 글씨 혹은 검정 글씨가 보이지 않아서 인 거 같은데 기여자 입장에서는 둘 다 구해야하는 게 어려울 수 있을 거 같아요. 한쪽 모드로만 관리하면 어떨까요?
아래에서 제안하고 싶은 부분 PR로 남겨요
#101
* docs: light 모드로 강제하고 오타 수정 및 컴포넌트 수정, 로고도 알파값 있게 수정 * chore: update * docs: update * docs: 다크모드 다시 만들고 아임웹도 추가 * chore: update
close #97
Overview
#97 에서 @manudeli 님이 제안주신것처럼, 라이브러리의 신뢰성 및 브랜딩을 높이기 위한 작업입니다.
많은 분들이 직접 PR로 배열을 관리하는 방향을 생각하고 있어요.
PR Checklist