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

Design : 첨부파일 제외 상세보기 뷰 리디자인 #143

Merged
merged 8 commits into from
Feb 6, 2024

Conversation

m6z1
Copy link
Collaborator

@m6z1 m6z1 commented Feb 2, 2024

📮 관련 이슈

✍️ 구현 내용

  • 리디자인한 상세보기 뷰 xml 작업
  • 툴바의 타이틀은 대학공지에서 진입 시엔 대학공지 / 학부공지에서 진입 시엔 학부공지가 띄워져야 함으로, 추후 api 연동 시에 viewmodel에서 title 관리해서 데이터 바인딩으로 뷰 구현 예정
  • 아직 회의 진행하지 않아 mvp 에서 제외될 수 있는 '팝업 메뉴 버튼 클릭 시 생성되는 바텀시트 뷰' 는 구현하지 않음

[20240203 23:38 update]

  • 한국에서 가장 긴 이름을 가지신 분의 length는 성을 포함하여 17글자이므로, maxlength제약 추가
  • date의 EndToEnd 제약 추가 및 horizontalbias 값을 주어 텍스트 왼쪽 정렬

📷 구현 영상

스크린샷 2024-02-03 오전 1 06 17

✔️ 확인 사항

  • 컨벤션에 맞는 PR 타이틀
  • 관련 이슈 연결
  • PR 관련 정보 연결 (작업자, 라벨, 마일스톤 등)
  • Github Action 통과

@m6z1 m6z1 added 🎨 DESIGN User interface design 💛명지 labels Feb 2, 2024
@m6z1 m6z1 self-assigned this Feb 2, 2024
android:layout_marginStart="16dp"
android:layout_marginEnd="16dp"
android:layout_marginTop="12dp"
android:hint="안녕하세ㅛ아아아ㅣㅣㄴㅇ럼ㄴ런"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
android:hint="안녕하세ㅛ아아아ㅣㅣㄴㅇ럼ㄴ런"
tools:hint="안녕하세ㅛ아아아ㅣㅣㄴㅇ럼ㄴ런"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ㅠㅠ

Comment on lines 96 to 98
android:id="@+id/detail_date"
style="@style/PretendardMedium14"
android:layout_width="wrap_content"
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
Collaborator Author

Choose a reason for hiding this comment

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

도전해보겠습니다 .

android:gravity="end"
android:hint="2024.02.01"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
android:hint="2024.02.01"
tools:hint="2024.02.01"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ㅠㅠ

android:gravity="end"
android:hint="손명지"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
android:hint="손명지"
tools:hint="손명지"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ㅜㅠ

android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:hint="반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
android:hint="반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ"
tools:hint="반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ반갑습니다반갑스비나아럼ㄴ;렁;ㄹ넘라ㅣ;멀;ㅣㅁ너라ㅓㄴ맒ㄴ"

Copy link
Member

Choose a reason for hiding this comment

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

디자인이 정상적으로 동작하는지 확인하는 경우엔 Lorem Ipsum 을 사용하는게 좀 더 깔끔할 것 같아요 😅

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

공부해보고 적용해보겠습니다🫡


<androidx.recyclerview.widget.RecyclerView
android:id="@+id/detail_image_rcv"
<androidx.constraintlayout.widget.ConstraintLayout
Copy link
Member

Choose a reason for hiding this comment

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

구조를 flat 하게 변경해주세요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이 부분은 나중에 첨부파일의 유무에 따라 바인딩 어댑터로 레이아웃 전체를 visibility 처리를 하려고 했습니다. 구조를 flat 하게 바꾸는 게 나을까요? 🤔

Copy link
Member

Choose a reason for hiding this comment

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

ConstraintLayoutGroup 을 사용하면 뷰의 가시성을 한 번에 조정할 수 있어요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

새로운 지식 습득 ~ 😋

Copy link
Contributor

@huiwoo-jo huiwoo-jo 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

@hoyahozz hoyahozz left a comment

Choose a reason for hiding this comment

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

tools:hint 부분에 무의미한 문자열이 아닌 실제 학교 데이터를 넣으면 좀 더 이해가 쉬울 것 같아요! 이 부분만 수정 후 머지하셔도 될 것 같습니다! 고생하셨습니다 👍👍👍

@m6z1 m6z1 merged commit c8f2fb0 into develop Feb 6, 2024
1 check passed
@m6z1 m6z1 deleted the feature/detail-ui branch February 6, 2024 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 DESIGN User interface design 💛명지
Projects
None yet
Development

Successfully merging this pull request may close these issues.

공지사항 글 상세 보기 뷰 리디자인 작업
3 participants