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

로그인 & 디자인 시스템 일부 적용 #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ITlearning
Copy link

💡 Issue

https://yappu-world.atlassian.net/browse/YPW-44

🌱 Key changes

✅ To Reviewers

  • Navigation Routing 관련하여 아직 제작을 못함.
  • 혹시 좋은 방법 있으면 먼저 적용해줘도 좋읍니다~
  • 의존성이나 디자인 시스템 코드는 지속적으로 추가하겠습니다.. (생각보다 양이 많아서 이전 프로젝트 보면서 옮기느라 정신이 없네염)

📸 스크린샷

image

@ITlearning ITlearning requested a review from ShapeKim98 January 7, 2025 15:07
Copy link

@ShapeKim98 ShapeKim98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다! 반성을 많이 하게 되는 pr인것 같네요.. 잘 배워가겠습니다... 저는 아직 한참 모자라요..
네비게이션 라우터는 NavigationStack기반의 ViewModel + Delegate패턴을 주로 썼긴 했는데... 좀 더 고민해볼게여 일본여행 잘 다녀오세여

case border
}

public struct YPButtonStyle: ButtonStyle {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ButtonStyle, TextFieldStyle 보면서 그동안 디자인시스템 헛으로 만들었다는 것을 깨달았습니다... 많이 배우겠슴다,,,

@@ -9,15 +9,20 @@ import SwiftUI

// MARK: - Gray Scale
public extension Color {
static let gray22: Color = .init(uiColor: .init(hex: "#70737C")).opacity(0.22)
static let gray22: Color = .init(hex: "#70737C").opacity(0.22)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 그냥 헥스값으로 하신 이유가 있나여? 좀 힘들었을 것 같은데...

.padding(.vertical, verticalPadding)
.foregroundStyle(foregroundColor)
.background(backgroundColor)
.if(colorStyle == .border) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 신박한데,,,

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.

2 participants