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

feat: 앨범 상세 페이지 추가 #34

Merged
merged 13 commits into from
Oct 3, 2023
Merged

Conversation

peacemaker474
Copy link
Member

@peacemaker474 peacemaker474 commented Sep 29, 2023

🔥 연관 이슈

close: #

📝 작업 요약

  • 앨범 상세 페이지 추가

🔎 작업 상세 설명

  • 앨범 상세 페이지 추가
  • 앨범 상세 페이지에 따른 라우터 추가 및 일부 로직 변경
    • 공통 Header, Footer 영역 내 '/details' 포함 여부에 따른 조건부 렌더링 처리
    • /album/:id에서 /album/details로 라우터 변경
  • host_03-2 앨범 상세 이미지 및 미니 앨범 영역 구현에 필요한 Swiper 추가
    • 미니 앨범 영역 구현
  • 앨범 상세 페이지 Footer 내 아이콘 추가 및 날짜 형식 변환 로직 추가
    • 날짜 형식 변환 로직 (수정 필요)
    • 해당 이미지가 메세지를 포함하고 있는 지에 따른, 조건부 메세지 아이콘 추가 필요
  • Header 영역 스타일 수정
  • 이미지 내 메세지 유무에 따른 메시지 아이콘 조건부 렌더링 추가

🌟 논의 사항

  • 현재 보여지고 있는 앨범에 대한 정보가 필요해서, 일단은 동작하기 위해서 전역변수로 억지로 처리했는데 이 부분은 조금 더 고민해봐야 할 것 같습니다. 이게 필요했던 이유는, 앨범 상세 영역 페이지 하단에 날짜 및 관련 이미지 정보를 통해 추후, 메세지에 대한 부분을 서버 요청해야 되기 때문입니다.
  • 앨범 상세 페이지 내 전체적인 뷰 영역 및 미니 앨범 영역 내 Swiper의 CenteredSlides가 제대로 동작하지 않는 것 같은데, 해결해야 될 것 같습니다.😭
  • 메세지 SVG가 공통되는데, 공통 Footer 하단과 메세지 영역에서 active에 따른 색상이 달라졌는데, 일단 급하게 하나 추가했습니다. 관련 어떻게 처리하는 게 좋을까요?

@peacemaker474 peacemaker474 added ⚙ Setting 개발 환경 세팅 🎨 HTML&CSS 마크업 & 스타일링 ✨ Feature 기능 개발 FE labels Sep 29, 2023
@peacemaker474 peacemaker474 self-assigned this Sep 29, 2023
@peacemaker474 peacemaker474 added this to the 2차 스프린트 milestone Sep 29, 2023
@peacemaker474 peacemaker474 added the 🔨 Refactor 코드 리팩토링 label Oct 1, 2023
@leedawnn
Copy link
Contributor

leedawnn commented Oct 3, 2023

  • 현재 보여지고 있는 앨범에 대한 정보가 필요해서, 일단은 동작하기 위해서 전역변수로 억지로 처리했는데 이 부분은 조금 더 고민해봐야 할 것 같습니다. 이게 필요했던 이유는, 앨범 상세 영역 페이지 하단에 날짜 및 관련 이미지 정보를 통해 추후, 메세지에 대한 부분을 서버 요청해야 되기 때문입니다.
  • 앨범 상세 페이지 내 전체적인 뷰 영역 및 미니 앨범 영역 내 Swiper의 CenteredSlides가 제대로 동작하지 않는 것 같은데, 해결해야 될 것 같습니다.😭
  • 메세지 SVG가 공통되는데, 공통 Footer 하단과 메세지 영역에서 active에 따른 색상이 달라졌는데, 일단 급하게 하나 추가했습니다. 관련 어떻게 처리하는 게 좋을까요?

Swiper의 CenteredSlides 관련해서 이것저것 해봤는데 모르겠네요.. 😇 그리고 공통 푸터 메세지 svg랑 메세지 영역 svg는 moment Blue로 색상이 똑같은 것 같은데, 달라진 게 어떤 걸까요?

@peacemaker474
Copy link
Member Author

  • 현재 보여지고 있는 앨범에 대한 정보가 필요해서, 일단은 동작하기 위해서 전역변수로 억지로 처리했는데 이 부분은 조금 더 고민해봐야 할 것 같습니다. 이게 필요했던 이유는, 앨범 상세 영역 페이지 하단에 날짜 및 관련 이미지 정보를 통해 추후, 메세지에 대한 부분을 서버 요청해야 되기 때문입니다.
  • 앨범 상세 페이지 내 전체적인 뷰 영역 및 미니 앨범 영역 내 Swiper의 CenteredSlides가 제대로 동작하지 않는 것 같은데, 해결해야 될 것 같습니다.😭
  • 메세지 SVG가 공통되는데, 공통 Footer 하단과 메세지 영역에서 active에 따른 색상이 달라졌는데, 일단 급하게 하나 추가했습니다. 관련 어떻게 처리하는 게 좋을까요?

Swiper의 CenteredSlides 관련해서 이것저것 해봤는데 모르겠네요.. 😇 그리고 공통 푸터 메세지 svg랑 메세지 영역 svg는 moment Blue로 색상이 똑같은 것 같은데, 달라진 게 어떤 걸까요?

host_03-2 하단에 메세지 아이콘이 검정색이고, 메세지를 클릭했을 때 moment Blue로 색상을 변경시켜야 하는데, 일단은 분리해서 추가시켜 놓은거라서, 합치는 작업도 필요할 것 같아요.

@peacemaker474 peacemaker474 merged commit 7c65ddf into develop Oct 3, 2023
1 check passed
@peacemaker474 peacemaker474 deleted the feature/album_details branch October 3, 2023 07:23
@peacemaker474 peacemaker474 linked an issue Oct 3, 2023 that may be closed by this pull request
18 tasks
peacemaker474 added a commit that referenced this pull request Nov 21, 2023
* feat: 앨범 상세 페이지 라우터 및 레이아웃 추가

* feat: Swiper 설치 및 앨범 상세 페이지 내 미니앨범 구현

* remove: 안 쓰는 코드 라인 제거

* refactor: details에 따른 헤더 분기처리 및 일부 스타일 수정

* refactor: 현재 상세 페이지 앨범 적용 및 스타일 수정

* style: Footer 상세 페이지에 대한 조건부 추가

* 상세 페이지 Footer 영역 및 관련 아이콘 추가

* style: SVG 일부 에러 수정 및 상세 페이지 앨범 반응형 추가

* style: 앨범 영역 정중앙 스타일 추가

* refactor: formatDate 타입 가드 추가

* style: 앨범 상세 페이지 전체적인 레이아웃 스타일 수정

* style: 헤더 영역 스타일 수정

* refactor: 앨범 타입 변경 및 메세지 여부에 따른 조건부 렌더링 추가
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE ✨ Feature 기능 개발 🎨 HTML&CSS 마크업 & 스타일링 🔨 Refactor 코드 리팩토링 ⚙ Setting 개발 환경 세팅
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] 추후 반영할 사항
2 participants