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

[Feature/#119] news detail #125

Merged
merged 18 commits into from
Nov 26, 2024
Merged

[Feature/#119] news detail #125

merged 18 commits into from
Nov 26, 2024

Conversation

chanubc
Copy link
Collaborator

@chanubc chanubc commented Nov 25, 2024

โœ… ๐—–๐—ต๐—ฒ๐—ฐ๐—ธ-๐—Ÿ๐—ถ๐˜€๐˜

  • mergeํ•  ๋ธŒ๋žœ์น˜์˜ ์œ„์น˜๋ฅผ ํ™•์ธํ•ด ์ฃผ์„ธ์š”(mainโŒ/developโญ•)
  • ๋ฆฌ๋ทฐ๊ฐ€ ํ•„์š”ํ•œ ๊ฒฝ์šฐ ๋ฆฌ๋ทฐ์–ด๋ฅผ ์ง€์ •ํ•ด ์ฃผ์„ธ์š”
  • P1 ๋‹จ๊ณ„์˜ ๋ฆฌ๋ทฐ๋Š” ํ•„์ˆ˜๋กœ ๋ฐ˜์˜ํ•ฉ๋‹ˆ๋‹ค.
  • Approve๋œ PR์€ assigner๊ฐ€ ๋จธ์ง€ํ•˜๊ณ , ์ˆ˜์ • ์š”์ฒญ์ด ์˜จ ๊ฒฝ์šฐ ์ˆ˜์ • ํ›„ ๋‹ค์‹œ push๋ฅผ ํ•ฉ๋‹ˆ๋‹ค.

๐Ÿ“Œ ๐—œ๐˜€๐˜€๐˜‚๐—ฒ๐˜€

๐Ÿ“Ž๐—ช๐—ผ๐—ฟ๐—ธ ๐——๐—ฒ๐˜€๐—ฐ๐—ฟ๐—ถ๐—ฝ๐˜๐—ถ๐—ผ๐—ป

  • news ์ƒ์„ธ ํ™”๋ฉด ๊ตฌํ˜„
  • ์ด๋ฏธ์ง€ ์ƒ์„ธ๋ณด๊ธฐ, ๊ตฌ๊ธ€ ํผ ์ด๋™
  • ์ƒํƒœ๋ฐ” ๋กœ์ง ์ˆ˜์ •

๐Ÿ“ท ๐—ฆ๐—ฐ๐—ฟ๐—ฒ๐—ฒ๐—ป๐˜€๐—ต๐—ผ๐˜

KakaoTalk_20241126_043927695.mp4

๐Ÿ’ฌ ๐—ง๐—ผ ๐—ฅ๐—ฒ๐˜ƒ๐—ถ๐—ฒ๐˜„๐—ฒ๐—ฟ๐˜€

์ƒํƒœ๋ฐ”๊ฐ€ detail ํ™”๋ฉด์—์„œ white์—์„œ ๋ณ€ํ•˜์งˆ ์•Š์•„์„œ MainActivity๋กœ ๋กœ์ง ์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค!
fragment์˜ ๋ผ์ดํ”„์‚ฌ์ดํด ์•Œ ์ˆ˜ ๊ฐ€ ์—†๊ตฐ์š”.. ์ œ ์ƒ๊ฐ์—” fragment์—์„œ activity์˜ context์— ์ ‘๊ทผํ•˜์—ฌ ์‹œ์Šคํ…œ ui ์ˆ˜์ •์ด ์ƒ๋ช…์ฃผ๊ธฐ์ƒ ๋‚จ์•„ ์žˆ๋Š”๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค

@chanubc chanubc requested a review from a team November 25, 2024 19:40
@chanubc chanubc self-assigned this Nov 25, 2024
@chanubc chanubc requested review from Eonji-sw and sohyun127 and removed request for a team November 25, 2024 19:40
@chanubc chanubc added โญ [FEAT] ์ƒˆ๋กœ์šด ๊ธฐ๋Šฅ ๊ตฌํ˜„ ๐Ÿป ์ฐฌ์šฐ ์ฐจ์€์šฐ โ˜• [COMPOSE] ์ปดํฌ์ฆˆ ui labels Nov 25, 2024
Copy link
Collaborator

@sohyun127 sohyun127 left a comment

Choose a reason for hiding this comment

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

LGTM ๐Ÿ‘

@@ -220,6 +224,22 @@ class MainActivity : AppCompatActivity(), Navigation {
}
}

private fun updateStatusBarColor(navController: NavController) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

์ง€๊ธˆ ์ƒ๊ฐํ•ด๋ณด๋‹ˆ status bar๋Š” activity์˜ ์ฑ…์ž„์ด๋‹ˆ main activity์—์„œ ๊ด€๋ฆฌํ•˜๋Š”๊ฒŒ ๋” ์ข‹์•„๋ณด์ด๋„ค์š”! ๐Ÿ‘


class NewsDetailFragment : BindingFragment<FragmentNewsDetailBinding>(FragmentNewsDetailBinding::inflate) {
private val args: NewsDetailFragmentArgs by navArgs()
private val notice: NewsInfoModel by lazy { args.newsInfoModel }
Copy link
Collaborator

Choose a reason for hiding this comment

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

์˜น ์ง€์—ฐ ์ดˆ๊ธฐํ™” ํ•˜์…จ๊ตฐ์š”

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

์‚ฌ์‹ค initView()์•ˆ์— ํ–ˆ์–ด๋„ ์ƒ๊ด€์—†๊ธดํ•ด์š”ใ…Ž

@chanubc chanubc merged commit 4d2b8f5 into develop Nov 26, 2024
1 check passed
@chanubc chanubc deleted the feature/#119-news-detail branch November 26, 2024 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
โ˜• [COMPOSE] ์ปดํฌ์ฆˆ ui โญ [FEAT] ์ƒˆ๋กœ์šด ๊ธฐ๋Šฅ ๊ตฌํ˜„ ๐Ÿป ์ฐฌ์šฐ ์ฐจ์€์šฐ
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] : ์ƒ์„ธํ™”๋ฉด ui ๋ฐ ์—ฐ๊ฒฐ
2 participants