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

[Team23] 프론트엔드 UI 완성 #45

Merged
merged 22 commits into from
Apr 28, 2021

Commits on Apr 21, 2021

  1. [Add] Carousel

    - 캐러셀 구현
    - 캐러셀 틀을 잡기 위해 구조 살짝 변경 - 컴포넌트 트리 참고
    - products.length가 8의 배수가 아닌 경우를 생각해서 로직을 짰는데 아직 테스트는 안해봄
    adelakim5 committed Apr 21, 2021
    Configuration menu
    Copy the full SHA
    c75cc1d View commit details
    Browse the repository at this point in the history
  2. [Fix] Carousel

    - 4의 배수가 아닐 때 캐러셀 애니메이션 테스트함
    - left 버튼을 누를 때 문제가 있었는데 index 업데이트를 수정하여 고침
    adelakim5 committed Apr 21, 2021
    Configuration menu
    Copy the full SHA
    8e42c5f View commit details
    Browse the repository at this point in the history

Commits on Apr 22, 2021

  1. [Fix] Direction of carousel-button

    - 왼쪽 버튼을 누르면 이전 슬라이드가, 오른쪽 버튼을 누르면 이후 슬라이드가 나오도록 수정
    adelakim5 committed Apr 22, 2021
    Configuration menu
    Copy the full SHA
    1c99905 View commit details
    Browse the repository at this point in the history

Commits on Apr 23, 2021

  1. [Fix] Merge conflict

    - 아델라의 캐로셀 코드를 demo 브랜치에 머지
    deprecated-hongbiii committed Apr 23, 2021
    Configuration menu
    Copy the full SHA
    ccfab6b View commit details
    Browse the repository at this point in the history
  2. [Add] hover ver.1

    - 리렌더링 때문에 깜빡거림
    deprecated-hongbiii committed Apr 23, 2021
    Configuration menu
    Copy the full SHA
    ac9082b View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    27cd014 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    24fae61 View commit details
    Browse the repository at this point in the history
  5. [Refactor] Modal 컴포넌트 분리

    - 공통적으로 사용되는 styled component를 common.jsx에 분리
    - 카드에 있는 정보를 모달에 전달
    deprecated-hongbiii committed Apr 23, 2021
    Configuration menu
    Copy the full SHA
    c3b9535 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    7c86069 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    70a202a View commit details
    Browse the repository at this point in the history

Commits on Apr 26, 2021

  1. Configuration menu
    Copy the full SHA
    e2b8c6e View commit details
    Browse the repository at this point in the history
  2. [Modify] Apply Portal for modal

    - 컴포넌트 구조 변경
        - utils를 구별하려고 했는데 components 안에있는 utils가 rename이 안되어서 외부 utils, 내부 utils로 둠
        - 별로 좋지 않은 느낌인거 같고, 아무래도 이 부분은 팀원들과 함께 논의하면서 진행해야 할거같아서 내꺼에만 저장해놓기!
    - Portal
        - index.html에 global-modal 이라는 id값을 가진 돔 생성
        - Modal이라는 포탈을 만들어서 띄워주기
        - 띄워지는거 확인
    adelakim5 committed Apr 26, 2021
    Configuration menu
    Copy the full SHA
    e3e19fe View commit details
    Browse the repository at this point in the history
  3. [Modify] Locate Portal-modal into Card

    - Card의 자식으로 Portal-modal을 집어넣음
    - 모달이 뜰 때 fetch로 해당 상품의 정보를 가져오는 방식으로 구현
        - 혹시라도 실시간으로 재고 상황이 달라질까 싶어서 매번 fetch 하는 형식으로 구현했는데, 좋은 방법인지는 모르겠음
    - StyledLi 밖에서 setModalState를 해주어야 함
        - 그렇지 않으면 클릭하는 모든 부분이 StyledLi라서 클릭 이벤트가 자꾸 발생해 false로 바꿔도 다시 true가 되어 모달창이 꺼지지 않는 일 발생
    - 현재 useEffect 안쓰고 그냥 fetch를 사용했고, 동작은 하는데, 좋은 방법처럼 느껴지지는 않음
        - 근데 useEffect안에 fetch를 사용하는 것이 원칙일까.. 좀 더 공부해볼 필요가 있는 것 같음
    adelakim5 committed Apr 26, 2021
    Configuration menu
    Copy the full SHA
    2a35a71 View commit details
    Browse the repository at this point in the history
  4. [Refactor] 제니의 퍼블리싱 작업 합치기 & 리팩토링

    - 너무 무거워지는 것 같아서 products라는 폴더 만들고 하위 컴포넌트들을 나눠 저장
    - 제니의 퍼블리싱 작업 합침
        - 근데 작은 사진들을 클릭했을 때 썸네일에 안뜨는 문제 발생함
    - 자잘한 버그가 있는 것 같음. 추후 재확인 필요
    adelakim5 committed Apr 26, 2021
    Configuration menu
    Copy the full SHA
    487e287 View commit details
    Browse the repository at this point in the history

Commits on Apr 27, 2021

  1. [Refactor] Integration

    - 제니의 퍼블리싱 작업을 합침
    - 불필요한 주석 제거
    adelakim5 committed Apr 27, 2021
    Configuration menu
    Copy the full SHA
    2c0ee6d View commit details
    Browse the repository at this point in the history
  2. [Add] Install reallyawesome-jennyrousel

    - JennyCarousel 설치해서 적용
    - 자잘한 css 수정 (title의 margin, section의 width..)
    adelakim5 committed Apr 27, 2021
    Configuration menu
    Copy the full SHA
    add6642 View commit details
    Browse the repository at this point in the history
  3. [Modify] Modal, Carousel

    - 모달 창 크기 수정
    - 모달 창 내 캐러셀 삽입 (with 제니캐러셀)
    - 모든 카테고리 보기 구현
    adelakim5 committed Apr 27, 2021
    Configuration menu
    Copy the full SHA
    81b3fce View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    e2b8e11 View commit details
    Browse the repository at this point in the history
  5. [Refactor] 모달창 JennyCarousel 수정

    - 4개 => 5개로 (개수 지정 가능하도록 수정)
    - RecommendCarouselTitle position absolute를 통해 재배치 및 부모 position relative 주기
    wjddnjswjd12 committed Apr 27, 2021
    Configuration menu
    Copy the full SHA
    3a3dfa5 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    f2a1cc1 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    00242f4 View commit details
    Browse the repository at this point in the history
  8. Merge pull request #18 from dyongdi/feature/jentest

    Feature/jentest
    deprecated-hongbiii committed Apr 27, 2021
    Configuration menu
    Copy the full SHA
    4858938 View commit details
    Browse the repository at this point in the history