ryush00 / capstone

1 stars 0 forks source link

메인창_홈버튼_새로고침기능적용 #42

Closed Yang0Jin0Woo closed 2 weeks ago

Yang0Jin0Woo commented 2 weeks ago

작업 내용

스크린샷

리뷰 시 참고사항

ryush00 commented 2 weeks ago

굳이 필요하지 않은 코드 같아요. 이미 root_path( = / )로 링크 걸린 홈 누르면 새로고침되는데, 자바스크립트로 location.reload();를 할 필요가 없을 것 같아요.

ryush00 commented 2 weeks ago

아 PR 내용 지금 확인했습니다. 살펴볼게요.

Yang0Jin0Woo commented 2 weeks ago

넵!!

ryush00 commented 2 weeks ago

40 PR에서 수정 했어요. 해당 브랜치 가져와서 잘 되나 한번 확인해보시고 승인부탁드려요!

image

ryush00 commented 2 weeks ago

홈 누르면 (페이지가 바뀌면) 자바스크립트 코드도 실행이 되는데, 지금 사용되는 라이브러리 (turbo) 방식 상 기존에 설정됐던게 초기화가 안되고 덮어씌어 지는 방식이라서 뭔가 꼬였어요.

image

간단히는 홈 링크에 data-turbo="false"를 추가해주면 되기는 했을탠데 (이게 구현하신 방식이랑 동일합니다), 좀더 근본적인 문제를 해결하고 싶어서 #40 PR에서 수정했어요.

ryush00 commented 2 weeks ago

40 PR에서 수정했으므로, 종결할게요