Skip to content
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

Merged
merged 4 commits into from
May 24, 2024

Conversation

okinawaa
Copy link
Member

@okinawaa okinawaa commented May 24, 2024

close #97

Overview

#97 에서 @manudeli 님이 제안주신것처럼, 라이브러리의 신뢰성 및 브랜딩을 높이기 위한 작업입니다.

많은 분들이 직접 PR로 배열을 관리하는 방향을 생각하고 있어요.

image

PR Checklist

  • I read and included theses actions below
  1. I have read the Contributing Guide
  2. I have written documents and tests, if needed.

@okinawaa okinawaa requested a review from raon0211 as a code owner May 24, 2024 07:05
Copy link

vercel bot commented May 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
es-hangul ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 24, 2024 10:31am

Copy link

changeset-bot bot commented May 24, 2024

⚠️ No Changeset found

Latest commit: f12245f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov-commenter
Copy link

codecov-commenter commented May 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.51%. Comparing base (7bec39c) to head (f12245f).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #100   +/-   ##
=======================================
  Coverage   99.51%   99.51%           
=======================================
  Files          13       13           
  Lines         206      206           
  Branches       46       46           
=======================================
  Hits          205      205           
  Misses          1        1           

@okinawaa okinawaa requested a review from manudeli May 24, 2024 07:08
Comment on lines 1 to 20
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,
}}
/>
);
}
Copy link
Member

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
@okinawaa okinawaa merged commit 8d4dad0 into main May 24, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: trusted by 토스, 회사1, 회사2
3 participants