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

[FIX/#62] 인턴 공고 컴포넌트 #65

Merged
merged 2 commits into from
Jul 12, 2024
Merged

Conversation

boiledeggg
Copy link
Member

⛳️ Work Description

  • 인턴 공고 컴포넌트 로직 수정

📸 Screenshot

로직 수정이라 스샷은 없습니다!

📢 To Reviewers

  • 컴포넌트의 크기 설정 방식을 조금 수정하였습니다!!
    내부 컴포넌트 크기를 토대로 외부 레이아웃의 크기가 정해지도록 구현했습니다.
  • 스크랩 색상이 붙은 박스 로직도 조금 수정되었습니다!!

@boiledeggg boiledeggg added FIX 🔨 버그 및 오류 해결 석준💜 석준 labels Jul 12, 2024
@boiledeggg boiledeggg self-assigned this Jul 12, 2024
Copy link
Contributor

@arinming arinming 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 +54 to +55
.height(intrinsicSize = IntrinsicSize.Max)
.padding(10.dp),
Copy link
Contributor

Choose a reason for hiding this comment

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

intrinsicSize 속성은 첨ㅂ네요!!!!

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 +53 to +56
.fillMaxWidth()
.height(intrinsicSize = IntrinsicSize.Max)
.padding(10.dp),
verticalAlignment = Alignment.Bottom
Copy link
Member

Choose a reason for hiding this comment

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

크,, 결국 해낸 석준오빠 멋있다!!! intrinsicSize이제 마스터 했네여

Comment on lines +133 to +139
@Composable
fun TwoLineHeightText(
text: String,
style: TextStyle,
) {
val twoLineHeight = with(LocalDensity.current) {
(style.lineHeight.toDp() * 3) - style.fontSize.toDp()
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 +158 to +159
title = "[Someone] 콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인콘텐츠 마케터 대학생 마케터 대학생 인턴 채용 공고",
dateDeadline = "3",
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
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.

우왕 최고에요!!

style: TextStyle,
) {
val twoLineHeight = with(LocalDensity.current) {
(style.lineHeight.toDp() * 3) - style.fontSize.toDp()
Copy link
Member

Choose a reason for hiding this comment

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

이거 생각해낸거 진짜 대박이에요..!!

@boiledeggg boiledeggg merged commit 614856a 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
FIX 🔨 버그 및 오류 해결 석준💜 석준
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 인턴 공고 컴포넌트
4 participants