[TIL] 210908 오늘 공부는 풀리퀘스트, react-modal

오늘은 리액트 모달창 위에 리액트 모달창을 올리는 걸 하려고 했다. 안되길래 찾아보니까 multiple modal은 안된다는 글들이 많고, 무슨 라이브러리를 또 설치해야한다고 해서 바닐라 자바스크립트로 팝업창을 구현하다가

안될리가 없는 것 같아서 z-index를 주어봤다ㅠ 그래도 안되길래 안되나 했는데 overlay (모달창 배경)에 z-index를 주니까 모달1 위로 모달2가 올라왔다. 오.. 모달 2개가 되는구나. 찾아보니까 모달에 z-index 줄 때, content 뿐만 아니라 overlay에도 z-index를 줘야 위로 올라온다고 했다.

결론, react-modal 라이브러리 모달 2개 됨 (3개도 될까,,?)

 

pull request가 무엇인지, 어떻게 쓰는지에 대해 알게되었다. pull request는 원격에서 merge 할 수 있는 것. 
이름 그대로 "내가 작업한게 있는데 검토하고 merge 해주세요!!" 라는 뜻. 리뷰어를 추가할 수 있다. 원본 레포지토리 관리자가 검토 후 merge 할건지, reject 할건지 결정할 수 있다.

merge에는 3가지 옵션이 있고, 그냥 merge commit 남기거나, 하나의 관리할 브랜치에 다시 재정렬 해주는 rebase merge 있고, 다 합쳐서 하나의 커밋만 남기는 squash and merge 가 있다.

아래 블로그에서 자세하게 설명해주는데 이해가 잘 됨
https://meetup.toast.com/posts/122

협업시 코드 충돌을 최소화 할 수 있고, push 권한이 없는 오픈 소스 프로젝트에서 많이 쓰인다고 한다. 일단 push 하고 merge 하기 전에 코드 리뷰를 받고 진행할 수 있으니 충돌은 당연히 적을 것 같고, 나의 경우에는 동료분께서 코멘트를 달아서 보여주셨다. 

깃에서 코멘트를 작성할 수 있고, 내가 잘못 쓴 이중 삼항 연산자 같은 것도 코멘트를 작성해주셔서 리뷰용으로 확인하기 좋았다. 내가 이 기능을 또 언제 쓸 수 있을지는 모르겠다! 

 

오늘은 코드를 제대로 이해하지 못했는지 아예 조건문을 반대로 쓴것이 적발? 되었다. 설명을 듣고보니 그렇다. 왜 이렇게 했지?;;; 없을때 불러와야하는 코드였는데, 있을때 불러오고 있었다. 그래서 화면 깜빡임이 있었고 ㅠㅠㅠ 코드를 100% 이해 못한 상황에서 조건문을 넣으려다 보니 그렇게 된 것 같다.

요새는 틈틈히 코드를 보면서 100% 이해하려고 노력중이다. 80%만 이해하면 안될 것 같다. 솔직히 이 코드가 뭐하는 놈인지 아직도 잘 모르는게 많지만 계속 보고 찍어보면 익숙해지다가 그냥 머리에 들어오지 않을까 기대하고 있다.(? ㅎ)