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

[UI/#42] 공고 뷰 / 공고 상세 페이지 구현 #63

Merged
merged 14 commits into from
Jul 12, 2024

Conversation

arinming
Copy link
Contributor

⛳️ Work Description

  • 공고 상세 페이지 탑 바, 바텀 바 구현
  • 공고 상세 정보 컴포넌트 구현
  • 스크랩 클릭 구현

📸 Screenshot

42.mp4

📢 To Reviewers

  • 뒤로가기 버튼 리플 효과 해제했읍니다

@arinming arinming added UI 💐 UI 작업 아린💛 아린 labels Jul 12, 2024
@arinming arinming added this to the 1차 스프린트 UI 작업 milestone Jul 12, 2024
@arinming arinming self-assigned this Jul 12, 2024
@arinming arinming linked an issue Jul 12, 2024 that may be closed by this pull request
6 tasks
Copy link
Member

@leeeyubin leeeyubin left a comment

Choose a reason for hiding this comment

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

실력이 날로 늘고 있는데요,,,,,,ㄷㄷㄷㄷ 하산하세요.

Comment on lines 93 to 95
item {
val decimal = DecimalFormat("#,###")

Copy link
Member

Choose a reason for hiding this comment

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

오홍 신기하네요! "#,###"는 상수로 둬도 될 것 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

반영했습니닷!

Comment on lines 178 to 182
}
}


Column(
Copy link
Member

Choose a reason for hiding this comment

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

줄 하나 삭제해주세용

Comment on lines 45 to 47
modifier = Modifier
.width(60.dp)
.height(60.dp)
Copy link
Member

Choose a reason for hiding this comment

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

요기두 aspectRatio로 둘 수 있을 듯??!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

끄앙 다 변경햇더용! 감사해ㅑ여

Comment on lines 28 to 30

@Serializable
data object Intern : MainTabRoute
Copy link
Member

Choose a reason for hiding this comment

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

바텀바 없는 스크린 맞죠...? 그럼 Route로 해주세요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아맞다.........................바보

Copy link
Member

@Hyobeen-Park Hyobeen-Park left a comment

Choose a reason for hiding this comment

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

진짜 이렇게 멋있기 있어요?!??!?!??? 진짜 최고다....

Copy link
Member

@boiledeggg boiledeggg left a comment

Choose a reason for hiding this comment

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

뷰 하나를 반나절만에 끝내버리시네요,, Arin 폼미

tint = if (isScrap) TerningMain else Grey350
)
Text(
text = "${viewCount}회",
Copy link
Member

Choose a reason for hiding this comment

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

리소스화!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

인정합니다.

@arinming arinming merged commit 5399824 into develop Jul 12, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI 💐 UI 작업 아린💛 아린
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[UI] 공고 뷰 / 공고 상세 페이지 구현
4 participants