Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: code-squad/javascript-amazon
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: code-squad/javascript-amazon
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: choisohyun
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 4 commits
  • 25 files changed
  • 1 contributor

Commits on Feb 13, 2020

  1. [HTML, CSS, JS] 아마존 - 네비게이션UI 미션 제출합니다. (#122)

    * Initial commit
    
    * feat. html, css UI 틀 추가
    
    - 아마존 슬라이드 화면을 클론코딩한 파일 업로드
    - 슬라이드 클릭 시 화면 이동은 아직 구현하지 않음
    - 이미지, 콘텐츠는 dummy 파일 이용함
    
    * feat. html, css UI 틀 추가
    
    - 아마존 슬라이드 화면을 클론코딩한 파일 업로드
    - 슬라이드 클릭 시 화면 이동은 아직 구현하지 않음
    - 이미지, 콘텐츠는 dummy 파일 이용함
    
    * feat. prev,next 버튼 클릭 시 슬라이드 이동 기능 추가
    
    - 슬라이드의 양 옆 화살표 버튼 클릭 시 다음 화면이 나오는 기능을 구현함
    - 상단의 네비게이션 클릭 시 이동은 아직 구현하지 못함
    - 마지막 슬라이드에서 다음 버튼을 누르면 첫 번째 슬라이드가 나옴
    - 처음 슬라이드에서 이전 버튼을 누르면 마지막 슬라이드가 나옴
    
    * refactor. function -> class로 변경
    
    - js 코드를 ES6 Classes로 표현하기 위해 변경함
    - 기능 추가는 없음
    
    * feat. navigation 버튼 클릭 시 슬라이드 이동 기능 추가
    
    - 각각의 navigation 버튼을 클릭하면 해당 버튼의 index를 얻어와 이동함
    - 슬라이드 애니메이션 기능은 아직 추가하지 않음
    
    * docs. Delete image files, duplicate files
    choisohyun authored Feb 13, 2020
    Configuration menu
    Copy the full SHA
    258da47 View commit details
    Browse the repository at this point in the history

Commits on Feb 17, 2020

  1. [미션 제출] Animation (#137)

    * Initial commit
    
    * feat. html, css UI 틀 추가
    
    - 아마존 슬라이드 화면을 클론코딩한 파일 업로드
    - 슬라이드 클릭 시 화면 이동은 아직 구현하지 않음
    - 이미지, 콘텐츠는 dummy 파일 이용함
    
    * feat. html, css UI 틀 추가
    
    - 아마존 슬라이드 화면을 클론코딩한 파일 업로드
    - 슬라이드 클릭 시 화면 이동은 아직 구현하지 않음
    - 이미지, 콘텐츠는 dummy 파일 이용함
    
    * feat. prev,next 버튼 클릭 시 슬라이드 이동 기능 추가
    
    - 슬라이드의 양 옆 화살표 버튼 클릭 시 다음 화면이 나오는 기능을 구현함
    - 상단의 네비게이션 클릭 시 이동은 아직 구현하지 못함
    - 마지막 슬라이드에서 다음 버튼을 누르면 첫 번째 슬라이드가 나옴
    - 처음 슬라이드에서 이전 버튼을 누르면 마지막 슬라이드가 나옴
    
    * refactor. function -> class로 변경
    
    - js 코드를 ES6 Classes로 표현하기 위해 변경함
    - 기능 추가는 없음
    
    * feat. navigation 버튼 클릭 시 슬라이드 이동 기능 추가
    
    - 각각의 navigation 버튼을 클릭하면 해당 버튼의 index를 얻어와 이동함
    - 슬라이드 애니메이션 기능은 아직 추가하지 않음
    
    * docs. Delete image files, duplicate files
    
    * feat. Add Mock Data & html 자동생성파일 완성
    
    data.js - mock data를 임의로 배열로 저장함
    mockData.js - mock data를 html 파일로 순서대로 추가될 수 있도록 함
    
    * feat. Add dummy element for infinite slides
    
    - addDummySlide(function)
     부드럽게 넘어가는 무한 슬라이드를 구현하기 위해 생성함
     slide의 first, last child를 각각 마지막과 처음에 넣어 줌
     추후 파일명, 변수명 등 리팩토링 예정임
    
    * refactor. Rename & change class structure
    
    - index.js -> main.js, navigationBar.js, carousel.js로 클래스 나눔
    - slideSetting.js -> 슬라이드 구현에 필요한 요소를 객체로 따로 저장함
    - sprinkleData.js -> mockData.js에 배열로 있는 데이터를 뿌려주는 역할을
    함
    - navigationBar.js와 carousel.js는 다른 클래스이지만 중복되는 코드가
    존재함
    => 추후 리팩토링 필요
    
    * style. Edit arrow button position
    
    화살표 버튼의 위치가 어색해 width를 100% -> 800px로 변경함
    choisohyun authored Feb 17, 2020
    Configuration menu
    Copy the full SHA
    2d08d9b View commit details
    Browse the repository at this point in the history

Commits on Feb 26, 2020

  1. [미션] Fetch (#155)

    * Initial commit
    
    * feat. html, css UI 틀 추가
    
    - 아마존 슬라이드 화면을 클론코딩한 파일 업로드
    - 슬라이드 클릭 시 화면 이동은 아직 구현하지 않음
    - 이미지, 콘텐츠는 dummy 파일 이용함
    
    * feat. html, css UI 틀 추가
    
    - 아마존 슬라이드 화면을 클론코딩한 파일 업로드
    - 슬라이드 클릭 시 화면 이동은 아직 구현하지 않음
    - 이미지, 콘텐츠는 dummy 파일 이용함
    
    * feat. prev,next 버튼 클릭 시 슬라이드 이동 기능 추가
    
    - 슬라이드의 양 옆 화살표 버튼 클릭 시 다음 화면이 나오는 기능을 구현함
    - 상단의 네비게이션 클릭 시 이동은 아직 구현하지 못함
    - 마지막 슬라이드에서 다음 버튼을 누르면 첫 번째 슬라이드가 나옴
    - 처음 슬라이드에서 이전 버튼을 누르면 마지막 슬라이드가 나옴
    
    * refactor. function -> class로 변경
    
    - js 코드를 ES6 Classes로 표현하기 위해 변경함
    - 기능 추가는 없음
    
    * feat. navigation 버튼 클릭 시 슬라이드 이동 기능 추가
    
    - 각각의 navigation 버튼을 클릭하면 해당 버튼의 index를 얻어와 이동함
    - 슬라이드 애니메이션 기능은 아직 추가하지 않음
    
    * docs. Delete image files, duplicate files
    
    * feat. Add Mock Data & html 자동생성파일 완성
    
    data.js - mock data를 임의로 배열로 저장함
    mockData.js - mock data를 html 파일로 순서대로 추가될 수 있도록 함
    
    * feat. Add dummy element for infinite slides
    
    - addDummySlide(function)
     부드럽게 넘어가는 무한 슬라이드를 구현하기 위해 생성함
     slide의 first, last child를 각각 마지막과 처음에 넣어 줌
     추후 파일명, 변수명 등 리팩토링 예정임
    
    * refactor. Rename & change class structure
    
    - index.js -> main.js, navigationBar.js, carousel.js로 클래스 나눔
    - slideSetting.js -> 슬라이드 구현에 필요한 요소를 객체로 따로 저장함
    - sprinkleData.js -> mockData.js에 배열로 있는 데이터를 뿌려주는 역할을
    함
    - navigationBar.js와 carousel.js는 다른 클래스이지만 중복되는 코드가
    존재함
    => 추후 리팩토링 필요
    
    * style. Edit arrow button position
    
    화살표 버튼의 위치가 어색해 width를 100% -> 800px로 변경함
    
    * feat. launch with express server, get json file fetch
    
    - server와 client를 나눔
    - server에는 app.js와 routes가 있음
    => 라우터 : express로 index.html과 mockData.json 파일을 띄우고 보내는
    역할을 하는 index.js, users.js가 있음
    - 애니메이션에 버그가 있음 => 추후 해결 예정
    
    * refactor & fix. Remove duplicate code, fix some animation bugs
    
    변경 사항
    1. server 단에 있던 html 코드 => client로 이동
    2. 독립적인 두 서버를 연결해 json file fetch받을 수 있도록 처리
    3. fetchData.js에 있던 코드 => main.js로 이동
    - fetch된 데이터 templating 완료 이후에 애니메이션이 적용되야 하기
    때문에 main으로 이동함
    4. navigationBar.js & carousel.js => carouselMovement.js,
    eventHandler.js로 클래스 역할 변경
    - navigationBar와 carousel 클래스 간에 중복되는 동작 메소드가 많아서
    이를 줄이기 위해 carousel을 움직이는 메소드가 있는 클래스인
    carouselMovement와 버튼 클릭 시 이벤트를 추가해 주는 eventHandler로
    변경함
    5. css 변경 중복 코드를 util.js로 묶어서 새로 만듦
    6. 버그 미해결 부분 있음
    - 클래스를 두 개로 나눴을 때 currentIndex가 동기화되지 않아서 다른
    영역을 번갈아 클릭했을 때 이벤트가 제대로 동작하지 않음
    7. templateData.js => template literal 방법 적용 필요
    
    * chore. Delete old work files added during merge
    
    - git 로그 관리 중 머지를 진행했는데, 이전 작업 파일이 생겨 이를 삭제함
    
    * fix & refactor. Change class structure
    
    - carouselMovement & eventHandler => carousel로 클래스 합침
    - 동기화되지 않던 슬라이드 인덱스로 인한 버그 사라짐
    
    * refactor. Use template literal when forming html
    
    - html을 mock Data로 그려줄 때 template literal 방식 사용함
    - 자연스러운 애니메이션을 위한 dummySlide 추가 부분은 DOM API 사용함
    
    * feat. Add Cache json file in localStorage
    
    - localStorage에 서버가 연결되어 있을 동안 캐시되도록 설정함
    
    * refactor. Rename function css => set
    
    - cssTransition과 cssTransform => setTransition, setTransform
    - css라는 이름이 적절하지 않은 것 같아 변경함
    
    * fix. Rename function css => set
    
    - util에서 변경한 이름 동일하게 변경함
    choisohyun authored Feb 26, 2020
    Configuration menu
    Copy the full SHA
    020721d View commit details
    Browse the repository at this point in the history

Commits on Mar 8, 2020

  1. [미션] Search Auto Complete (#171)

    * Initial commit
    
    * feat. html, css UI 틀 추가
    
    - 아마존 슬라이드 화면을 클론코딩한 파일 업로드
    - 슬라이드 클릭 시 화면 이동은 아직 구현하지 않음
    - 이미지, 콘텐츠는 dummy 파일 이용함
    
    * feat. html, css UI 틀 추가
    
    - 아마존 슬라이드 화면을 클론코딩한 파일 업로드
    - 슬라이드 클릭 시 화면 이동은 아직 구현하지 않음
    - 이미지, 콘텐츠는 dummy 파일 이용함
    
    * feat. prev,next 버튼 클릭 시 슬라이드 이동 기능 추가
    
    - 슬라이드의 양 옆 화살표 버튼 클릭 시 다음 화면이 나오는 기능을 구현함
    - 상단의 네비게이션 클릭 시 이동은 아직 구현하지 못함
    - 마지막 슬라이드에서 다음 버튼을 누르면 첫 번째 슬라이드가 나옴
    - 처음 슬라이드에서 이전 버튼을 누르면 마지막 슬라이드가 나옴
    
    * refactor. function -> class로 변경
    
    - js 코드를 ES6 Classes로 표현하기 위해 변경함
    - 기능 추가는 없음
    
    * feat. navigation 버튼 클릭 시 슬라이드 이동 기능 추가
    
    - 각각의 navigation 버튼을 클릭하면 해당 버튼의 index를 얻어와 이동함
    - 슬라이드 애니메이션 기능은 아직 추가하지 않음
    
    * docs. Delete image files, duplicate files
    
    * feat. Add Mock Data & html 자동생성파일 완성
    
    data.js - mock data를 임의로 배열로 저장함
    mockData.js - mock data를 html 파일로 순서대로 추가될 수 있도록 함
    
    * feat. Add dummy element for infinite slides
    
    - addDummySlide(function)
     부드럽게 넘어가는 무한 슬라이드를 구현하기 위해 생성함
     slide의 first, last child를 각각 마지막과 처음에 넣어 줌
     추후 파일명, 변수명 등 리팩토링 예정임
    
    * refactor. Rename & change class structure
    
    - index.js -> main.js, navigationBar.js, carousel.js로 클래스 나눔
    - slideSetting.js -> 슬라이드 구현에 필요한 요소를 객체로 따로 저장함
    - sprinkleData.js -> mockData.js에 배열로 있는 데이터를 뿌려주는 역할을
    함
    - navigationBar.js와 carousel.js는 다른 클래스이지만 중복되는 코드가
    존재함
    => 추후 리팩토링 필요
    
    * style. Edit arrow button position
    
    화살표 버튼의 위치가 어색해 width를 100% -> 800px로 변경함
    
    * feat. launch with express server, get json file fetch
    
    - server와 client를 나눔
    - server에는 app.js와 routes가 있음
    => 라우터 : express로 index.html과 mockData.json 파일을 띄우고 보내는
    역할을 하는 index.js, users.js가 있음
    - 애니메이션에 버그가 있음 => 추후 해결 예정
    
    * refactor & fix. Remove duplicate code, fix some animation bugs
    
    변경 사항
    1. server 단에 있던 html 코드 => client로 이동
    2. 독립적인 두 서버를 연결해 json file fetch받을 수 있도록 처리
    3. fetchData.js에 있던 코드 => main.js로 이동
    - fetch된 데이터 templating 완료 이후에 애니메이션이 적용되야 하기
    때문에 main으로 이동함
    4. navigationBar.js & carousel.js => carouselMovement.js,
    eventHandler.js로 클래스 역할 변경
    - navigationBar와 carousel 클래스 간에 중복되는 동작 메소드가 많아서
    이를 줄이기 위해 carousel을 움직이는 메소드가 있는 클래스인
    carouselMovement와 버튼 클릭 시 이벤트를 추가해 주는 eventHandler로
    변경함
    5. css 변경 중복 코드를 util.js로 묶어서 새로 만듦
    6. 버그 미해결 부분 있음
    - 클래스를 두 개로 나눴을 때 currentIndex가 동기화되지 않아서 다른
    영역을 번갈아 클릭했을 때 이벤트가 제대로 동작하지 않음
    7. templateData.js => template literal 방법 적용 필요
    
    * chore. Delete old work files added during merge
    
    - git 로그 관리 중 머지를 진행했는데, 이전 작업 파일이 생겨 이를 삭제함
    
    * fix & refactor. Change class structure
    
    - carouselMovement & eventHandler => carousel로 클래스 합침
    - 동기화되지 않던 슬라이드 인덱스로 인한 버그 사라짐
    
    * refactor. Use template literal when forming html
    
    - html을 mock Data로 그려줄 때 template literal 방식 사용함
    - 자연스러운 애니메이션을 위한 dummySlide 추가 부분은 DOM API 사용함
    
    * feat. Add Cache json file in localStorage
    
    - localStorage에 서버가 연결되어 있을 동안 캐시되도록 설정함
    
    * refactor. Rename function css => set
    
    - cssTransition과 cssTransform => setTransition, setTransform
    - css라는 이름이 적절하지 않은 것 같아 변경함
    
    * fix. Rename function css => set
    
    - util에서 변경한 이름 동일하게 변경함
    
    * refactor. 'run' function=>instant excution function
    
    - run 함수는 한 번만 실행되기 위한 함수이기 때문에 익명즉시실행함수로
    변경함
    
    * feat. Add search box
    
    - 검색창 html, css 코드 작성
    - 참고 : https://codepen.io/huange/pen/rbqsD, https://www.amazon.com/
    - 자동완성 리스트 나오는 부분은 다음 커밋에 올릴 예정
    
    * feat. Add AutoComplete List View
    
    - html은 뼈대만 남김
    - templateData.js에서 searchBox와 AutoList를 추가해 준다.
    - addAutoList에서 li 태그 부분은 하드코딩으로 수정이 필요하다.
    - 상위 10개까지만 검색되게 하기 위해 처음부터 리스트를 10개로 시작했다.
    
    * style. Modify directory structure
    
    - 폴더 구조 변경
    
    * style. Modify directory structure
    
    - 폴더 구조 변경2
    
    * feat. Event occurs when search box is clicked
    
    - 검색창 클릭 시 클래스 이름 추가되고, 하이라이트되는 이벤트 발생함
    
    * feat. Handle when input search keyword
    
    - 검색 키워드 입력 시 화면 처리하는 기능 추가
    - 자동완성 리스트는 autoComplete에서 처리한 것을 기반으로 될 것임
    - autoComplete는 다음 커밋으로 할 예정
    
    * feat. Generate keyword match list
    
    - controller를 통해 검색할 키워드가 autoComplete로 전해져서 검색하게 됨.
    - controller는 다음 커밋에 올릴 예정
    
    * feat. Controller that dispatches input events
    
    - 입력값을 300ms의 지연 시간을 두고 저장해, searchBox와 autoComplete에
    전달해 실행할 수 있도록 함
    - 하드코딩한 부분은 추후 수정할 예정임
    
    * feat. Add autocomplete result keyboard move event
    
    - 자동완성 결과를 방향키로 이동하기 기능 추가
    - enter 입력 시 해당 위치 결과가 표시되게 하는 기능 추가
    
    * feat. Server to import data
    
    - 목데이터를 가져와 보내주는 서버
    - express/heroku 사용해 호스팅 할 예정임.. (firebase 실패함..)
    
    * feat. Navigation of the current position
    
    - 현재 표시되고 있는 부분을 Navigation의 scale이 변화하면서 강조시킴
    
    * refactor. Duplicate style change function
    
    - visibility, scale 변환 부분에 중복 코드를 줄이기 위해 함수로 따로
    저장함
    
    * refactor. delete duplicate carousel code
    
    - 캐러셀 슬라이드 중복부분 수정
    
    * refactor. delete duplicate search autocomplete code
    
    - 자동완성 중복부분 따로 메소드 빼는 식으로 수정
    
    * docs. Delete old file - carousel.js
    
    * docs. Delete old file - main.js
    
    * docs. Delete old file - templateData.js
    
    * docs. Delete old file - util.js
    choisohyun authored Mar 8, 2020
    Configuration menu
    Copy the full SHA
    c499b78 View commit details
    Browse the repository at this point in the history
Loading