-
Notifications
You must be signed in to change notification settings - Fork 5
Week3_Day4 회의록
yyjjjj edited this page Dec 4, 2020
·
1 revision
- 이은솔
- 어제는 설계의 중요성을 깨닫는 날이었습니다. 누구의 상태인지 협소하게 보면 안된다는 것을 깨달았습니다.
- 타입스크립트 이슈를 붙잡고 몇시간을 보낸 것 같은데.. 그러면서 타입스크립트를 조금 더 이해하게 되었습니다.
- 이준희
- landscape와 관련된 레이어들 속성 변경하는 거를 90% 완성했습니다.
- 다른 속성을 가진 레이어(line, polygon)을 어떻게 구분해서 속성을 적용해야할지 잘 모르겠습니다. 이렇게 헤매고 있는 게 리뷰에서도 지적받은 것 같아 가슴이 아픕니다.
- 다른 분들이 구현하신 거를 보다 적극적으로 참고해보겠습니다
- landscape와 관련된 레이어들 속성 변경하는 거를 90% 완성했습니다.
- 박은식
- subFeature만 string으로 처리를 한 것이 조금 잘못되었던 것 같습니다. feature들을 스타일링 할 때 사용할 수 있으니 feature별로 subFeature타입을 만드는 것이 좋을 것 같습니다.
- 색상표 정리를 끝냈습니다.
- 위정훈
- mapbox 기본 제공하는 스타일로 변경해보고 dark/light/outdoor 맵스타일 json 데이터를 추출해봣습니다. 위성 사진 지도는 json이 아예 없고, outdoor는 조금 내부 함수가 다른 것 같아요
- 어차피 구글 stylemap을 따라간다면 크게 우려될 건 아닐 것 같아요
- 이연정
- 반복되는 구조에 대한 리팩토링을 진행하며 반복되는 작업일수록 초기 데이터 형태 파악과 구조 설계가 중요하다는 생각이 들었습니다.
- 타입을 세세하게 정해놓은 것을 가져다 사용하며 세세한 타입정의의 편리함을 느꼈습니다.
- land
- 국립공원
- 도로 전체 추가 필요
- 물도 네이버 부분으로 추정되는 부분만 지워지고 OSM 부분은 안 지워지네요
- 어떤부분이 안지워질까요?!
- water-shadow라고 물 그림자 부분이 있었네요. live session에서 .\src\utils\map-styling\water.ts 경로에 추가해뒀습니다
- 어떤부분이 안지워질까요?!
- 이 외에도 인도에도 그림자가 있네요. 원하신다면 어떤 부분이 있는지 이후에 공유해드리겠습니다 (감사합니다!)
-
세부 유형 없는 곳으로 이동하면 에러 발생 -> 기본값을 선택하지 않은 상태로 바꾸기 -> useDetail에서 featureName이 바뀌는 것을 useEffect로 받아서 값을 다시 가져오는거?
-
스타일러에서 작업하다가 다른 카테고리로 이동할 때 한번씩 fill/text 속성 못 찾는다고 에러가 나네요
-
icon 기본 opacity가 0 보다 작다고 에러나네여
- weight 절대값
Error: layers.poi-attraction.paint.icon-opacity: -0.001 is less than the minimum value 0
-
리팩토링 사항
-
맵 스타일링 함수(추후...!)
- 제가 말씀드린건, 레이어 정의/state 또는 map 객체로부터 layer리스트를 구할수있고, 이 레이어 리스트 중 변경 할 대상만 필터링(레이어 타입 또는 기타 정보)해서, 각 레이어를 타입 별 스타일 적용이 가능한지를 질문드렸습니다! 현재 구조를 잘 모르고, 질문드렸지만 의도는 현재 복잡한 구문을 로직/규칙으로 간결하게 가능한지 여부가 궁금해서입니다.
-
맵? 상태로 사용하는거...?
-
element === detailName subElement === subDetailName으로 두개가 혼동되서 사용되는 경우가 많은 것 같습니다. 나중에 다같이 한번 타입이름 정의를 하면서 몸에 익힐 필요가 있어 보여요! > 같이
- detail -> element V
-
DetailType 컴포넌트 분리 > 1 // 준희
-
사이드바 상태 redux로 분리 > 2 // 은식, 정훈
-
세부 유형 없는 곳으로 이동하면 에러 발생 > 2 // 은솔, 연정
-
-
theme관련 hook 만들고 있으면 > 끝나는 대로 붙기?
-
deepcopy