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

[FE] 피드백 #30

Open
crongro opened this issue Feb 6, 2023 · 1 comment
Open

[FE] 피드백 #30

crongro opened this issue Feb 6, 2023 · 1 comment
Labels
Frontend Frontend

Comments

@crongro
Copy link

crongro commented Feb 6, 2023

style 코드를 모아둔 시도는 좋고요. common 에 좀더 많은 부분을 모아서 재사용하시는 시도도 좋을것 같군요.

css 스타일 단위가 다양한데 어느정도 규칙을 정해두고 사용해보세요.

src/scripts 라는 이름은 좀 애매합니다. 다른 디렉토리는 pages/ components/ 등 의미가 있는데말이죠.

파일들이 처음에 대문자로 시작하는것, 소문자로 시작하는것 등 일관성을 신경써주세요.

async 함수는 await 를 사용하지 않는다면 async 일 필요는 없습니다.

ES Classes 의 생성자를 사용하지 않으면 없어도 될 듯.

router.js 를 만들었군요. 좋은시도입니다. 완전히 동작하게 하는건 아주 어려운일이던데..노력해보세요!

@hoonystory98
Copy link
Contributor

귀중한 의견 남겨주셔서 정말 감사드립니다.
css는 scss로의 변경 작업 간에 분리 단위에 대해 다시 고민해보겠습니다.
scripts는 이벤트를 테스트해보기 위해 만들었습니다. 브랜치에 둘 이유가 없으니 꼭 지우겠습니다.
네이밍 컨벤션 좀 더 신경쓰겠습니다.
async, constuctor() 등은 확인한 뒤 불필요한 부분 삭제하도록 하겠습니다.
라우팅은 아직 작동방식에 대한 확실한 이해를 더 필요로 한다고 생각합니다. 개발 기간동안 잘 파악하여 적용할 수 있도록 해보겠습니다.

@hoonystory98 hoonystory98 added the Frontend Frontend label Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Frontend
Projects
None yet
Development

No branches or pull requests

2 participants