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

버튼 컴포넌트 디자인 리뉴얼 반영하기 #1579

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

mu-hun
Copy link
Contributor

@mu-hun mu-hun commented Mar 19, 2024

작업하면서 발견한 건데 v2 Button 컴포넌트가 링크로 쓰이는 경우 hover, active 등 인터렉션 스타일 클래스가 누락되네요... 현재 프로덕션 코드 베이스도 포함입니다.

Copy link

graphite-app bot commented Mar 19, 2024

Your org requires the Graphite merge queue for merging into main

Add the label “:rocket:” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

@mu-hun mu-hun added the preview 👀👀👀 label Mar 20, 2024
@withglyph
Copy link

withglyph bot commented Mar 20, 2024

배포가 끝나면 아래 주소에서 프리뷰를 확인할 수 있어요.

프로젝트 프리뷰 URL
penxle.com https://website-pr-1579.withglyph.dev
penxle.io https://company-pr-1579.withglyph.dev
help.penxle.com https://help-pr-1579.withglyph.dev

@mu-hun mu-hun force-pushed the button-renewal branch 2 times, most recently from 2aa327a to 38e5b60 Compare March 20, 2024 12:23
@mu-hun mu-hun marked this pull request as draft March 21, 2024 08:37
@mu-hun mu-hun added preview 👀👀👀 and removed preview 👀👀👀 labels Mar 21, 2024
@mu-hun mu-hun marked this pull request as ready for review March 25, 2024 10:56
@mu-hun mu-hun requested a review from Seohyun-Roh March 26, 2024 11:49
Copy link

graphite-app bot commented Mar 27, 2024

Merge activity

작업하면서 발견한 건데  v2 Button 컴포넌트가 링크로 쓰이는 경우 hover, active 등 인터렉션 스타일 클래스가 누락되네요... 현재 프로덕션 코드 베이스도 포함입니다.
@graphite-app graphite-app bot merged commit 369d87e into main Mar 27, 2024
9 checks passed
@graphite-app graphite-app bot deleted the button-renewal branch March 27, 2024 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview 👀👀👀
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants