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.

Sign up for GitHub

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

Jump to bottom

Fewer Options - 표기 단계 조절(도로) #64

Closed
1 task done
yyjjjj opened this issue Nov 30, 2020 · 0 comments · Fixed by #159 or #166
Closed
1 task done

Fewer Options - 표기 단계 조절(도로) #64

yyjjjj opened this issue Nov 30, 2020 · 0 comments · Fixed by #159 or #166
Labels
must This label is for issue which should be implemented 기본 설정
Milestone
4주차

Comments

@yyjjjj
Copy link
Collaborator

yyjjjj commented Nov 30, 2020 •
edited by Eunsol0410
Loading

  • 표기단계에 따라 해당 요소의 밀도가 조절된다

논의 내용

  • 우리가 구현할 때 기존 로직을 활용하는 것이 전부 일 뿐 새로운 얻을 만한 지식이 없다고 판단하여서 우선순위를 뒤로 미룸

추가 논의 사항

  • 상태가 없어지는 이슈 => 고급설정을 기본설정 하위로 이동
  • 전체 스타일 상태 업데이트 새로운 방식이 필요? 현재는 SET을 여러번 호출하는 방식이 되야함...
@yyjjjj yyjjjj added must This label is for issue which should be implemented 기본 설정 labels Nov 30, 2020
@qkrdmstlr3 qkrdmstlr3 added this to the 3 주차 milestone Nov 30, 2020
@qkrdmstlr3 qkrdmstlr3 modified the milestones: 3 주차, 4주차 Dec 7, 2020
gitgitWi pushed a commit to gitgitWi/Project08-A-Styled-Map-Admin-Tool that referenced this issue Dec 9, 2020
[FEAT] [boostcamp-2020#64,boostcamp-2020#66] 표기단계 조절 위한 hooks 구현
f3b87dd 
## 구현 내용

- 표기단계 range 선택에 따라 해당 layer visibility 조절
  - addlayer / removelayer 방식과 고민한 결과,
  - layer를 추가/삭제하는 방식으로 조절할 경우 리소스 문제 및 존재하지 않는 layer에 속성 적용 오류 발생 예상됨

- 이동하는 depth range 구하는 함수 구현, 여러 단계 이동시에도 한번에 처리할 수 있도록 함

## TODO

- 표기 단계를 1~2단계로 설정하더라도
  - 고급설정 메뉴 -> 돌아가기로 다시 돌아오면 `useState(3)`에 의해 무조건 3으로 지정되는 문제
  - localStorage 등 rendering 과 무관한 곳에 저장한 값을 가져오는 방식으로 해결할 수 있을지??
@gitgitWi gitgitWi linked a pull request Dec 9, 2020 that will close this issue
도로 및 행정구역 표기단계 조절 위한 hooks 구현, 교통 styler 함수 리팩토링 #159
Merged
gitgitWi pushed a commit to gitgitWi/Project08-A-Styled-Map-Admin-Tool that referenced this issue Dec 10, 2020
[FEAT] [boostcamp-2020#64,boostcamp-2020#66] 표기단계 조절 및 테마선택 상태 관리를 re…
fd86630 
…dux로 변경

- 상태 관리를 useState -> react-redux로 변경
  - action/reducer 추가
  - 기존 hooks 수정
gitgitWi pushed a commit to gitgitWi/Project08-A-Styled-Map-Admin-Tool that referenced this issue Dec 10, 2020
[FIX] [boostcamp-2020#64,boostcamp-2020#66] 행정구역 표기단계 조절시 에러 및 전체 스타일…
7794662 
… 조정 에러 해결

- 이전 커밋 (fd86630)에서 행정구역 중 `locality` 속성 관련 문제 해결
  - Element로 labelText 만 갖고 있어 다른 속성 적용함

- 세부 스타일 설정하고 표기단계 조절하는 경우, 전체 스타일 초기화 되는 문제 해결
  - `changeStyle` -> `replaceStyle`로 함수 변경하고
  - `useWholeStyle`에서 전체 저장된 스타일 가져오는 함수 `getWholeStyle` 생성
  - 표기단계 조절시 기존 스타일 iterate로 추가하여 적용함
@gitgitWi gitgitWi linked a pull request Dec 10, 2020 that will close this issue
표기단계 조절 및 테마선택 상태 관리를 redux로 변경 #166
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
must This label is for issue which should be implemented 기본 설정
Projects
None yet
Milestone
4주차
3 participants
@qkrdmstlr3 @yyjjjj @Eunsol0410