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

[PC-602] 심사 거절 다이얼로그 ui 구현 #68

Merged
merged 9 commits into from
Feb 14, 2025

Conversation

sksowk156
Copy link
Contributor

@sksowk156 sksowk156 commented Feb 13, 2025

1. ⭐️ 변경된 내용

-심사 거절 다이얼로그 ui 구현
-프로필 완성 navigation 연결

2. 🖼️ 스크린샷(선택)

설명

3. 💡 알게된 부분

4. 📌 이 부분은 꼭 봐주세요!

  • 프로필 완성 쪽에 있는 이미지 벡터는 export 한데로 안나와서 일단 뺏는데 뭔가 다른 방식으로 넣어야 할까요?

@sksowk156 sksowk156 added UI/UX 🎨 디자인 시스템, 디자인 리소스 관련 코드 🎨 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 ㅈㅎ찐혁 🌙 훗날 세상을 아우를 남자, sksowk156 labels Feb 13, 2025
@sksowk156 sksowk156 requested a review from tgyuuAn February 13, 2025 17:01
@sksowk156 sksowk156 self-assigned this Feb 13, 2025
Copy link
Member

@tgyuuAn tgyuuAn left a comment

Choose a reason for hiding this comment

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

진혁님 고생하셨습니다~~~

이제 진짜 별로 안남았다...

title: AnnotatedString,
subText: String,
@DrawableRes iconId: Int,
title: String,
Copy link
Member

Choose a reason for hiding this comment

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

AnnotatedString의 부모 Class에 String이 있나요 ?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

AnnotatedString이 내부가

class AnnotatedString internal constructor(
    val text: String,
    internal val spanStylesOrNull: List<Range<SpanStyle>>? = null,
    internal val paragraphStylesOrNull: List<Range<ParagraphStyle>>? = null,
    internal val annotations: List<Range<out Any>>? = null
) : CharSequence {

으로 되어 있어서 String을 부모 클래스로 가지고 있진 않는거 같아요... 그냥 String과 같은 CharSequence를 상속한 거 같은데... 제가 AnnotatedString이었던걸 바꾼 이유는 처음엔 text는 한 색상만 쓰고 subText가 글자색상이 변하는 부분이 있어서 text를 String 으로 바꾸고, subText는 AnnotateString을 적용했었는데, 여러 코드로 수정하다보니 String 그대로 두고 다시 원래대로 변경해두지 않았던 거 같아요... 다시 변경해둘까요?!

@tgyuuAn tgyuuAn added 머지 해도될듯염🌟 현재 코드를 기존 코드에 합쳐도 될 것 같다라고 판단..! 🌟 and removed 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 labels Feb 14, 2025
@sksowk156 sksowk156 merged commit 1e80a8f into develop Feb 14, 2025
1 check passed
@sksowk156 sksowk156 deleted the feature/sksowk156/PC-602 branch February 14, 2025 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX 🎨 디자인 시스템, 디자인 리소스 관련 코드 🎨 ㅈㅎ찐혁 🌙 훗날 세상을 아우를 남자, sksowk156 머지 해도될듯염🌟 현재 코드를 기존 코드에 합쳐도 될 것 같다라고 판단..! 🌟
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants