[Mission4/정충일] - Project_Notion_VanillaJS#37
Open
tooooo1 wants to merge 8 commits into3/#4_tooooo1from
Open
Conversation
beni1026
reviewed
Nov 21, 2022
beni1026
left a comment
beni1026
left a comment
There was a problem hiding this comment.
안녕하세요 충일님 ~~ 마지막 코드 리뷰 하러 왔습니다. !!
과제 하시느라 수고 많으셨어요 😎
앞으로도 화이팅 입니다
Comment on lines
+38
to
+47
| switch (className) { | ||
| case 'post-delete': | ||
| onDelete(id); | ||
| return; | ||
| case 'post-create': | ||
| onCreate(id); | ||
| return; | ||
| default: | ||
| break; | ||
| } |
Comment on lines
+21
to
+30
| onDelete: async (postId) => { | ||
| await deletePost(postId); | ||
| this.setState(); | ||
| history.replaceState(null, null, '/'); | ||
| const $page = document.querySelector('.post-edit-page'); | ||
| if ($page) { | ||
| $page.remove(); | ||
| } | ||
| }, | ||
| }); |
| @@ -0,0 +1,55 @@ | |||
| const API_END_POINT = 'https://kdt.roto.codes'; | |||
There was a problem hiding this comment.
API 주소가 노출되지 않도록 .gitignore 파일에 추가하는 게 좋을 것 같아요 ~~
Comment on lines
+87
to
+93
| margin: 0; | ||
| padding: 0; | ||
| border: 0; | ||
| font-size: 100%; | ||
| font: inherit; | ||
| vertical-align: baseline; | ||
| } |
There was a problem hiding this comment.
잘 몰라서 그런데 보통 대부분의 태그에 대해 위와 같은 css처리를 하나요 ??
There was a problem hiding this comment.
@beni1026 css reset이라고 불리워지는데 예전엔 브라우저들마다 스펙이 들쭉날쭉해서 사용했고요, 지금은 써야 한다는 의견도 있고 아니라고 생각하는 의견도 있어요.
sairion
reviewed
Nov 23, 2022
| await fetchDocuments(`/documents/${postId}`, { | ||
| method: 'DELETE', | ||
| }); | ||
| }; |
There was a problem hiding this comment.
fetch 함수들에서 전체적으로 모두 await을 사용하고 있는데, await은 사용부에서 하는 걸 가정하고 그냥 return fetchDocuments... 만 해주는게 어떨까 싶습니다.
Comment on lines
+87
to
+93
| margin: 0; | ||
| padding: 0; | ||
| border: 0; | ||
| font-size: 100%; | ||
| font: inherit; | ||
| vertical-align: baseline; | ||
| } |
There was a problem hiding this comment.
@beni1026 css reset이라고 불리워지는데 예전엔 브라우저들마다 스펙이 들쭉날쭉해서 사용했고요, 지금은 써야 한다는 의견도 있고 아니라고 생각하는 의견도 있어요.
|
전체적으로 잘 구현해주신 것 같은데 완성해보시지요! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 과제 설명
바닐라 JS만을 이용해 노션을 클로닝합니다.
기본적인 레이아웃은 노션과 같으며, 스타일링, 컬러값 등은 원하는대로 커스텀합니다.
👩💻 요구 사항과 구현 내용
기본 요구사항
바닐라 JS만을 이용해 노션을 클로닝합니다.
기본적인 레이아웃은 노션과 같으며, 스타일링, 컬러값 등은 원하는대로 커스텀합니다.
보너스 요구사항
그외 개선하거나 구현했으면 좋겠다는 부분이 있으면 적극적으로 구현해봅니다!
✅ 피드백 반영사항
✅ PR 포인트 & 궁금한 점