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

게시글 옵션 클릭시 바텀시트 띄우기 #366

Merged
merged 9 commits into from
May 12, 2023

Conversation

WhiteHyun
Copy link
Member

📌 PR 요약

🌱 작업한 내용

  • 상세 게시글 화면(BoardDetailViewController)에서 게시글 바텀시트가 띄워지도록 구현했습니다.

🌱 PR 포인트

  • BoardBottomSheetViewController에 initializer를 추가했습니다.
    • accessType: 사용자 타입을 의미합니다. author, host, normal이 존재하며 각각 게시글 저자, 호스트, 일반 유저를 뜻합니다.
    • isPinned: 해당 게시글이 고정되어있는지를 나타냅니다. 고정 여부에 따라 보여지는 텍스트가 달라집니다. 만약 고정되어있는 게시글에서 바텀시트가 띄워지는 경우, 클립보드 고정 해제 문구가 나타나고, 고정되어있지 않는 상태이면 클립보드에 고정이 나타납니다.

📸 스크린샷

host author normal
RPReplay_Final1683717898 Simulator Screen Recording - iPhone 14 Pro - 2023-05-10 at 20 29 54 Simulator Screen Recording - iPhone 14 Pro - 2023-05-10 at 20 30 18

📮 관련 이슈

WhiteHyun added 9 commits May 10, 2023 18:59
바텀시트의 기능이 접근자에 따라 다르므로, enum값을 추가로 구현
- 버튼이 눌리면, 사용자가 host, author, normal인지를 알려주도록 구현했습니다.
클립보드, 메인페이지에 따라 보여지는 클립보드 리스트 뷰의 텍스트가 달라집니다.
…ool)

`상세 게시글(BoardDetailVC)`에 진입할 때도 게시글 바텀시트를 사용하게 되는데, 이 때 클립보드로부터 들어왔는지
메인 페이지로부터 들어왔는 지 알 길이 없으므로 처음부터 게시글 고정 여부를 확인하여 텍스트를 처리하도록 수정했습니다.
바텀시트를 사용자 타입에 맞게 띄워주고, delegate를 연동하여 토스트로 알리도록 구현했습니다.
@WhiteHyun WhiteHyun added 😎 Enhancement New feature or request 🍎 ConfigureUI 화면 프로토타입 UI 구현 D+2 일반적인 범주의 PR이에요. 이틀 이내에 리뷰 해주시면 됩니다 🥳MainPage labels May 10, 2023
@WhiteHyun WhiteHyun requested review from dlrjswns and soobin-k May 10, 2023 11:31
@WhiteHyun WhiteHyun self-assigned this May 10, 2023
Copy link
Contributor

@dlrjswns dlrjswns left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 승현님 !

@WhiteHyun WhiteHyun merged commit 4efe7df into develop May 12, 2023
@WhiteHyun WhiteHyun deleted the feat/335-Board/BottomSheet branch May 12, 2023 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍎 ConfigureUI 화면 프로토타입 UI 구현 D+2 일반적인 범주의 PR이에요. 이틀 이내에 리뷰 해주시면 됩니다 😎 Enhancement New feature or request 🥳MainPage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants