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

[team-22][FE] 2주차 2번째 PR - Park + BB #205

Open
wants to merge 8 commits into
base: team-22
Choose a base branch
from

Commits on Apr 14, 2022

  1. [BB] column 타이틀 옆 +, x 버튼 이벤트

    * feat: column 제목 영역  +버튼,x버튼 클릭 이벤트
    
    * fix: x버튼 클릭 이벤트 삭제 및 취소 이벤트
    
    * refactor: button요소 tagComponents이용하여 생성
    
    * refactor: 템플릿 리터럴 관련 변수명 변경, 코드 분리
    
    * refactor: 중복함수 제거
    
    - on~~MouseOver, Out과 같은 함수를 handle~과 같이 수정
    
    * refactor: 각 버튼마다 존재했던 handle함수를 하나로 합침
    
    * refactor: 칼럼제목 + 버튼 클릭시 $cardWritable 표시
    
    - 기존 칼럼 헤더에 있던 cardWritable을 Cards내부로 이동
    - 클래스 toggle로 추가 / 제거
    
    * fix: 취소 버튼으로 newCard 닫히지 않던 문제 해결
    
    * refactor: 메서드명의 get~을 create로 변경
    BB-choi authored Apr 14, 2022
    Configuration menu
    Copy the full SHA
    870c3d3 View commit details
    Browse the repository at this point in the history
  2. [FE] 2주차 1번째 PR 반영 리팩터링

    * remove: 사용하지 않는 파일 삭제
    
    * refactor: 상수 -> 대문자로 사용
    
    * refactor: LOG_TYPE 대문자 변경에 따른 uppercase
    
    * refactor: Content components props
    
    * refactor: tag, innerHTML -> template 변수명 변경
    
    * refactor: useRef & DOM API 사용 로직 제거
    
    * refactor: tag, innerHTML -> template 변수명 변경
    
    * refactor: create column element refactor
    healtheloper authored Apr 14, 2022
    Configuration menu
    Copy the full SHA
    c714fb3 View commit details
    Browse the repository at this point in the history
  3. [FE] date utils & actions 시간 차이 표시

    * feat: date utils & get date diff
    
    * feat: create getISODateDiff
    healtheloper authored Apr 14, 2022
    Configuration menu
    Copy the full SHA
    a044296 View commit details
    Browse the repository at this point in the history
  4. [Park] webpack devtool 설정 반대로 되어있던 것 수정

    * fix: webpack devtoom 설정 반대로되어있던 것 수정
    
    * style: console 삭제
    healtheloper authored Apr 14, 2022
    Configuration menu
    Copy the full SHA
    07ee231 View commit details
    Browse the repository at this point in the history
  5. [Park] 카드 더블클릭 시 수정 가능, 수정 요청 보내기

    * feat: update card use double click
    
    * style: input value 없을 때 등록버튼 비활성화
    
    * refactor: inline 으로 css 조작하던 방식 리팩터링
    
    * style: confirm button 안쓰는 클래스 삭제
    healtheloper authored Apr 14, 2022
    Configuration menu
    Copy the full SHA
    0204099 View commit details
    Browse the repository at this point in the history

Commits on Apr 15, 2022

  1. [BB] 새로운 카드 등록

    * refactor: 불분명한 함수명, 변수명 변경
    
    - 카드 등록 버튼 $addButton으로 변경
    - 칼럼header의 +, x버튼 hover 핸들러 handleButtonHover로 변경
    
    * feat: newCard 생성시 등록버튼 비활성 스타일
    
    * feat: post 새로운 카드 등록하기
    
    * fix: 처음 $newCard가 표시되면 submit 버튼 disabled처리
    
    * fix: Fix a typo
    
    * refactor: 함수명, 변수명 등 불확실한 이름을 변경
    
    - id -> columnId
    - putTodo -> createTodo
    
    * fix: handleKeyUpInput에 빠졌던 조건 추가
    
    - else if()의 조건에 !isInputEmpty 추가
    
    * refactor: plusButton, deleteButton 핸들러 리팩토링
    
    - xButton, deleteButton 이름이 혼재 되어있던 것도 deleteButton으로 통합
    
    * refactor: handleNewCardVisibl~을 toggleCardVisible로 변경
    
    * refactor: $plusButton 클릭 이벤트 핸들러 추가
    
    - $plusButton이벤트 핸들러간 네이밍 통일 handle~PlusButton
    BB-choi authored Apr 15, 2022
    Configuration menu
    Copy the full SHA
    e54e436 View commit details
    Browse the repository at this point in the history
  2. [BB] 리뷰 PR관련 리팩토링 - Modal, Card

    * refactor: Cards.js에서 todos 정렬 함수 외부로 분리
    
    - dateUtils 사용하여 분리
    
    * refactor: Card.js 불필요한 함수 삭제
    
    - onMouseOver, onMouseOut 등 불필요한 함수 삭제
    
    * refactor: modal 핸들러 함수 위치 변경
    
    - App에서 state로 관리하도록 변경
        - columns를 App에서 생성하므로 App에서 state로 관리
        - 선택된id를 state로 관리하므로 re-rendering으로 인해 클래스 토글이
          아닌 state로 모달 표시 여부 결정
    
    * feat: 카드 삭제 / 모달 팝업 닫을 때 선택된 id 리셋
    
    * fix: x 버튼 hover시 cursor 범위 오류 해결
    
    * fix: PR confilct 해결
    
    * refactor: SideContent 최신순 정렬
    
    * refactor: PR리뷰 반영
    BB-choi authored Apr 15, 2022
    Configuration menu
    Copy the full SHA
    592c2e5 View commit details
    Browse the repository at this point in the history
  3. [Park] visible 처리 로직, input 값 체크 로직 추가, Webpack alias 설정

    * refactor: visible 처리 로직, input 값 체크 로직 추가
    
    * env: webpack alias 설정
    healtheloper authored Apr 15, 2022
    Configuration menu
    Copy the full SHA
    7125d44 View commit details
    Browse the repository at this point in the history