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

[CAT-93] DesignSystem 구축 #17

Merged
merged 8 commits into from
Aug 7, 2024
Merged

[CAT-93] DesignSystem 구축 #17

merged 8 commits into from
Aug 7, 2024

Conversation

devMinseok
Copy link
Member

[#100] FEAT : 로그인 구현

관련된 문서 📄

무엇에 관한 PR 인가요? 🙋

어떤 것을 작업하셨나요? 🛠

  • 작업1
  • 작업2

🌱 PR Point

  • PR Point 1
  • PR Point 2

@devMinseok devMinseok added the enhancement New feature or request label Aug 7, 2024
@devMinseok devMinseok requested a review from Jihyun247 August 7, 2024 04:21
@devMinseok devMinseok self-assigned this Aug 7, 2024
Copy link

linear bot commented Aug 7, 2024

Comment on lines +87 to +88
interactionItem(interaction: Global.Color.black.opacity(Alias.Interaction.hover), name: "Hover")
interactionItem(interaction: Global.Color.black.opacity(Alias.Interaction.pressed), name: "Pressed")
Copy link
Collaborator

Choose a reason for hiding this comment

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

interaction token은 컬러는 안가지고 opacity만 가지는 토큰이라서 이렇게 수정한거야? ? ?

Copy link
Member Author

@devMinseok devMinseok Aug 7, 2024

Choose a reason for hiding this comment

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

엉 opacty는 여러 색상에 대응해서 들어가야해서 Color가 아니라 Double을 반환하도록 했어

Comment on lines +30 to +31
public struct BarButtonDetail<Title: View, LeftIcon: View, RightIcon: View>: View {
@Environment(\.barButtonDetailStyle) private var style
Copy link
Collaborator

Choose a reason for hiding this comment

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

bar button detail 뷰랑 스타일 신기하다 .. ! 필요한 configuration 받아서 resolve 하는 과정 잘봤슴다 ...

@devMinseok devMinseok merged commit c994d77 into develop Aug 7, 2024
1 check passed
@devMinseok devMinseok deleted the feature/CAT-93 branch August 7, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants