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

Feature/상세페이지 구현 #24

Merged
merged 21 commits into from
May 7, 2024
Merged

Conversation

Ahnjonghyun87
Copy link
Collaborator

@Ahnjonghyun87 Ahnjonghyun87 commented May 6, 2024

close #22

detail.js에서 import 해온 detail 정보에 인물 관련 정보가 없어서

절취선 밑에 따로 api를 가져오는 함수를 gpt4파워로 썼습니다

현재는 상세정보와 인물 정보가 id 1094844로 맞춰진 상태인데

메인화면에서 카드를 클릭해서 가져오는 id 값이 밑의 credit api의 fetch에서도 동일하게 바꿔주는 작업까진 해야 합니다

@1eeyerin 1eeyerin assigned 1eeyerin and Ahnjonghyun87 and unassigned 1eeyerin May 6, 2024
@1eeyerin 1eeyerin added the enhancement New feature or request label May 6, 2024
@Ahnjonghyun87 Ahnjonghyun87 changed the title Feature/create detail Feature/상세페이지 구현 May 6, 2024
css/detail.css Outdated Show resolved Hide resolved
css/detail.css Outdated Show resolved Hide resolved
css/detail.css Outdated Show resolved Hide resolved
css/detail.css Outdated Show resolved Hide resolved
css/detail.css Outdated Show resolved Hide resolved
css/detail.css Outdated Show resolved Hide resolved
css/detail.css Outdated Show resolved Hide resolved
css/detail.css Outdated Show resolved Hide resolved
detail.html Outdated Show resolved Hide resolved
js/detail.js Outdated Show resolved Hide resolved
@Ahnjonghyun87
Copy link
Collaborator Author

.detail-view-content-area .detail-view-box .detail-info-table .genre {
display: inline-block;
color: #fff;
vertical-align: top;
} 220번째줄. 작동을 안해서 삭제했습니다

Copy link
Collaborator

@yeeunseo-dev yeeunseo-dev left a comment

Choose a reason for hiding this comment

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

너무 고생하셨습니다..
새로운 페이지라 난이도가 있었을텐데 아주 깔끔하게 적어주셔서 잘 읽었습니다 :)

background-repeat: no-repeat;
width: 100%;
height: 100%;
z-index: 10;
Copy link
Collaborator

Choose a reason for hiding this comment

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

z-index값이 어떤 역할하는지 궁금합니다! 저는 큰 차이없어서 지우기도 했어서욤

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

::before라는 효과 때문에 썼습니다 저도 웨이브 클론 css하면서 알게된건데 예를들면 얇은 셀로판지 두게 깔아두고 z 값으로 a와 b중 어떤게 더 앞으로 부각될지 조절하는 값이라고 합니다

css/detail.css Outdated

.detail-info-box {
width: 780px;

Copy link
Collaborator

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.

수정했습니다~

Copy link
Collaborator

@ARMJONG ARMJONG left a comment

Choose a reason for hiding this comment

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

JS코드 일일히 주석으로 달아주셔서 이해하기 쉬웠습니다!

@Ahnjonghyun87
Copy link
Collaborator Author

JS코드 일일히 주석으로 달아주셔서 이해하기 쉬웠습니다!

감사합니다 암종현님도 따봉스

@Ahnjonghyun87 Ahnjonghyun87 merged commit e6a5460 into develop May 7, 2024
@Ahnjonghyun87 Ahnjonghyun87 deleted the feature/create-detail branch May 7, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

상세페이지 작성
4 participants