Skip to content

[Mission4/이혜준] - Project_Notion_VanillaJS #24

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.

Sign up for GitHub

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

Open
solar3070 wants to merge 48 commits into
base: 3/#4_leehyejun
Choose a base branch
from
Open

[Mission4/이혜준] - Project_Notion_VanillaJS #24

Changes from 1 commit
Commits
Show all changes
48 commits
Select commit Hold shift + click to select a range
eb919f3
feat: PostList 더미데이터 렌더링
solar3070 Nov 14, 2022
1136a51
chore: gitignore 추가
solar3070 Nov 14, 2022
869c953
feat: api 요청 함수 작성
solar3070 Nov 14, 2022
5e652b6
feat: Root Documents 가져오는 api 연결
solar3070 Nov 14, 2022
270651a
feat: PostsPage 생성
solar3070 Nov 14, 2022
1926a68
feat: App 컴포넌트 생성
solar3070 Nov 14, 2022
5ad629f
feat: 버튼 컴포넌트 추가
solar3070 Nov 14, 2022
71610c3
feat: PostsPage에 페이지 추가 버튼 생성
solar3070 Nov 14, 2022
47b7ffe
feat: Editor 컴포넌트 생성
solar3070 Nov 14, 2022
dc81dd1
feat: storage.js 생성
solar3070 Nov 14, 2022
b4219b6
feat: 편집 중 상태를 알리는 콜백 함수 생성
solar3070 Nov 14, 2022
3bb525a
feat: onEditing 콜백 함수 디바운싱 처리
solar3070 Nov 14, 2022
3773b11
feat: PostEditPage 생성
solar3070 Nov 14, 2022
c4770e1
chore: api 헤더 x-username 변경
solar3070 Nov 14, 2022
d1b8220
feat: 라우팅 처리
solar3070 Nov 14, 2022
f51d1de
feat: 임시저장 Confirm 띄우기
solar3070 Nov 14, 2022
98adb9b
feat: postId에 따른 api 처리
solar3070 Nov 14, 2022
f50b874
feat: router.js 생성
solar3070 Nov 14, 2022
8d6365a
feat: 이동 버튼들 추가
solar3070 Nov 14, 2022
2c21a36
design: 문서 리스트 옆에 에디터 창 위치
solar3070 Nov 15, 2022
dccbb2e
chore: 필요없는 버튼 제거
solar3070 Nov 15, 2022
2d97090
feat: 페이지 추가 버튼 클릭시 문서 바로 생성
solar3070 Nov 15, 2022
eac8759
feat: 문서 삭제 기능 구현
solar3070 Nov 15, 2022
6b014c9
feat: 하위 문서 생성 기능 구현
solar3070 Nov 15, 2022
55024bd
feat: 제목 수정시 문서 리스트에도 실시간 반영
solar3070 Nov 15, 2022
c2e0a8e
design: 노션 레이아웃 스타일링
solar3070 Nov 15, 2022
7e81611
feat: 뒤로 가기, 앞으로 가기 구현
solar3070 Nov 16, 2022
7f58224
feat: 특정 포스트에서 새로고침할 때 리스트도 같이 뜨도록 수정
solar3070 Nov 16, 2022
125fdab
feat: 포스트 재클릭시 내용 날아감 방지
solar3070 Nov 16, 2022
3626b12
fix: 깜빡임 현상 방지 위해 코드 복구
solar3070 Nov 16, 2022
d235f4f
feat: 문서 수정 낙관적 업데이트
solar3070 Nov 16, 2022
49a1129
feat: 문서 삭제시 루트 URL로 이동
solar3070 Nov 16, 2022
43c58c8
feat: 문서 삭제시 루트 URL 이동 방어 코드 추가
solar3070 Nov 16, 2022
0c848c3
feat: 페이지 추가 버튼 클릭시 문서 트리 갱신
solar3070 Nov 16, 2022
543653c
feat: 특정 포스트에서 새로고침시 문서 트리 로드
solar3070 Nov 16, 2022
4189852
refactor: api 호출 함수들 생성
solar3070 Nov 16, 2022
44b612e
feat: 문서 리스트 렌더링하는 함수 생성
solar3070 Nov 16, 2022
c06f7e3
feat: 현재 편집 중인 문서의 하위 문서 링크 렌더링
solar3070 Nov 16, 2022
d712485
fix: js파일 확장자 붙이기
solar3070 Nov 16, 2022
49c04ac
design: 간단한 스타일링
solar3070 Nov 16, 2022
99b073d
feat: 하위 페이지 토글 기능 구현
solar3070 Nov 17, 2022
486fc28
design: 클릭 문서 액티브 스타일링
solar3070 Nov 17, 2022
c979e6a
feat: 헤더와 파비콘 추가
solar3070 Nov 18, 2022
9d596b5
design: 문서 제목 글자 초과시 ... 처리
solar3070 Nov 18, 2022
720e580
chore: 컬러 코드 변수 파일로 빼기
solar3070 Nov 18, 2022
d1bb616
chore: 클래스 이름 수정
solar3070 Nov 18, 2022
a19d4b3
feat: 헤더 타이틀 클릭시 루트로 이동
solar3070 Nov 18, 2022
3c87f90
chore: netlify 배포 404 에러 해결
solar3070 Nov 18, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/pages/PostEditPage.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,11 @@ export default function PostEditPage({ $target, initialState }) {
}
Comment on lines +38 to +39
Copy link

@fly1chop fly1chop Nov 21, 2022

Choose a reason for hiding this comment

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

저도 이 부분 어떻게 처리할지 고민 많이 했는데 커스텀 이벤트를 활용하는것이 좋네요..!! 이것도 나중에 변수로 만들어내면 더 깔끔할것같다는 생각입니다~


removeItem(postLocalSaveKey);

this.setState({
postId: post.id + "",
Copy link

@chunwookJoo chunwookJoo Nov 21, 2022

Choose a reason for hiding this comment

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

저는 이거 일일히 toString()으로 사용했는데 이렇게 하면 훨씬 간결해지네요 👍

post,
});
Comment on lines +43 to +46
Copy link

@fly1chop fly1chop Nov 21, 2022

Choose a reason for hiding this comment

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

editing 할때마다 this.setState => editor.setState하게되어 저장하면 form focus가 없어집니다. 제 생각에는 editing할때는 setState 없이 api 업데이트만 하고 page reload, push state할때만 setState 하면 더 좋을것같습니다~

Copy link
Member Author

@solar3070 solar3070 Nov 25, 2022

Choose a reason for hiding this comment

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

오 안그래도 이 부분 고민했었는데 감사합니다! 더 고민해보겠습니다

}, 1000);
},
});
Expand Down