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

[refactor] Dialog Preview 반복 되는 코드 제거 및 코드 개선 #133

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

easyhooon
Copy link
Collaborator

  • Preview 에서 기존에 만들어둔 컴포저블을 호출하여 반복되는 코드 제거,

  • ConfirmButton 의 Start Padding 이 CancelButton 이 존재할때만 적용되도록 코드 수정

  • @StringRes, @DrawableRes annotation 을 파라미터에 추가함으로서 코드의 안정성을 높임

Preview 에서 기존에 만들어둔 컴포저블을 호출하여 반복되는 코드 제거,

ConfirmButton 의 Start Padding 이 CancelButton 이 존재할때만 적용되도록 코드 수정

@StringRes, @DrawableRes annotation 을 파라미터에 추가함으로서 코드의 안정성을 높임
@easyhooon easyhooon added the design tasks related to design aspects of the project label Apr 8, 2024
@easyhooon easyhooon requested a review from josushell April 8, 2024 03:24
@easyhooon easyhooon self-assigned this Apr 8, 2024
@easyhooon easyhooon added the refactor Code restructure for better readability and efficiency label Apr 8, 2024
Copy link
Member

@josushell josushell left a comment

Choose a reason for hiding this comment

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

굿굿

@easyhooon easyhooon merged commit 75064c0 into develop Apr 8, 2024
1 check passed
@easyhooon easyhooon deleted the refactor/dialog branch April 8, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design tasks related to design aspects of the project refactor Code restructure for better readability and efficiency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants