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

Improved chart y axis properties #2

Closed
kyungtaak opened this issue Aug 12, 2018 · 9 comments
Closed

Improved chart y axis properties #2

kyungtaak opened this issue Aug 12, 2018 · 9 comments
Assignees
Labels
@dashboard Component Name : Dashboard / Chart enhancement Request Change and Feature Enhancement
Milestone

Comments

@kyungtaak
Copy link
Contributor

Is your feature request related to a problem? Please describe.

  • 기존 "auto scale" 용어가 혼란스러우며, 동작이 제대로 되지 않은 문제가 있습니다.
  • 축교자점과 축 스케일 설정간 사용자 액션에 대해 명확하지 않은 부분이 있어 재검토가 필요합니다.

Describe the solution you'd like

  • "auto scale" 용어 fmf '데이터 범위로 축 고정 (Fit to Data Scale)'로 변경, 이에 따른 차트 동작 재 검토
  • 배타적인 3가지(축범위 설정, 교차점, 로그 스케일) 옵션에 대한 동작 UX 재검토
    : 축 교자첨 설정은 기본값이 '0'이며, 별도 ON/OFF UI없이 사용자가 값을 입력하면 '설정 활성화' + '값 입력'이 동시에 처리되는 형태 (task의 사용 depth를 줄이려는 취지)
    : 배타적으로 동작한다는 UX를 명확하게 전달하기 위해서 ON/OFF 옵션 추가 (하나의 옵션이 활성화 시, 나머지 2개의 옵션은 비활성화 처리)

Describe alternatives you've considered

  • Line/Bar/Scatter/Combine Chart 적용 검토

Additional context

@kyungtaak kyungtaak self-assigned this Aug 12, 2018
@kyungtaak kyungtaak added enhancement Request Change and Feature Enhancement @dashboard Component Name : Dashboard / Chart labels Aug 12, 2018
@juheeko9
Copy link
Contributor

@kyungtaak f#2-improve_yaxis_properties 브랜치 생성하였습니다.

@juheeko9
Copy link
Contributor

@kyungtaak

  • auto scale용어 변경하였습니다.
  • 축범위 설정, 교차점 ui 변경하였습니다.
  • log scale의 경우 미개발된 사항이므로 완료후 같이 적용하겠습니다.

@esezin
Copy link

esezin commented Aug 17, 2018

@judy-exntu

  • '축 교차점' 레이블이 '축 교차점 설정'으로 변경되어야 합니다.
  • '축 교차점'의 ON/OFF는 축 교차점의 설정을 사용/사용하지 않음을 선택하는 개념이기 때문에, 만약 값을 변경한 후, OFF를 선택하면(즉, 교차점 설정을 하지 않겠다면) 기본값인 '0'으로 변경되면서 OFF되는 것이 옳은 동작입니다.
  • 축 범위 설정 >데이터 범위로 축 고정 사용 시, 최댓값/최솟값 입력창은 비활성화 되어야 합니다.
  • 최댓값 입력 이후, 삭제해도 값이 그대로 남아 있는 bug가 있습니다.

dave-taeho pushed a commit that referenced this issue Aug 17, 2018
@esezin
Copy link

esezin commented Aug 17, 2018

@judy-exntu

  • 축 범위 설정의 MIN, MAX 초기값을 불러올 수 있는 방법이 없다면, 초기값의 텍스트 필드를 비워 놓는 형태로 진행 부탁드립니다.
  • 데이터 증감을 고려한 시나리오는 삭제합니다.

@kyungtaak

  • 라인 차트에서 MIN, MAX 변경시 발생한 이슈는 내부적으로 좀 더 살펴본 결과, 원래의 기획의도대로 처리 가능할 것으로 보입니다. (중간에 라인이 끊기는 형태가 아닌, 라인의 첨단이 가려지는 형태)
  • 시간은 조금 소요될 것으로 보입니다만, 차주 초에는 적용 가능할 것으로 생각합니다.

dave-taeho pushed a commit that referenced this issue Aug 20, 2018
dave-taeho added a commit that referenced this issue Aug 20, 2018
@dave-taeho
Copy link
Contributor

dave-taeho commented Aug 20, 2018

@kyungtaak @esezin
아래 항목들에 대해 수정되어 확인 요청드립니다.

  • '축 교차점' 레이블이 '축 교차점 설정'으로 변경
  • '축 교차점'의 ON/OFF 사용성 변경
  • 축 범위 설정 >데이터 범위로 축 고정 사용 시, 최댓값/최솟값 입력창은 비활성화
  • 최댓값 입력 이후, 삭제해도 값이 그대로 남아 있는부분 수정
  • 축 교차점을 벗어난 라인차트 점위치 수정

@esezin
Copy link

esezin commented Aug 23, 2018

@DAVE-EXNTU @judy-exntu
금일 스프린트 회의에서 논의된 내용을 아래와 같이 정리합니다.

  1. ‘Fit to data scale’의 레이아웃 행 간격을 MIN, MAX와 동일하게 조정 (현재 간격이 너무 좁음)
  2. MIN / MAX의 최초 값으로 ‘데이터의 min,max 값’을 다시 넣는 것으로
  3. Fit to data scale에 echart 스케일 옵션 적용
  4. 레이블 변경 : Set Axis Range —> Set Axis Scale

그외의 UX issue(아래)는 PMS에 별도로 이슈를 생성하여 진행하겠습니다.

  1. ‘Fit to data scale’과 ‘MIN/MAX 입력’간 우선순위 조정
  2. Set Axis scale을 ON하면, 바로 어떠한 변경이 생기는 것이 좋겠음 (Fit to data scale을 기본 체크로 적용하면 어떨지?)
  3. MIN/MAX에 대한 범위 조절 등 다른 UI 고려

juheeko9 added a commit that referenced this issue Aug 23, 2018
juheeko9 added a commit that referenced this issue Aug 23, 2018
@kyungtaak
Copy link
Contributor Author

@esezin @judy-exntu 위의 코멘트에 피드백드립니다.

  • 포커스 관련 내용이 빠진것 같습니다. (이건은 전체 적용이 필요한가요?)
  • 2번은 바로 적용해도 좋을것 같은데요. 검토 부탁드립니다.
  • F#2 improve yaxis properties #30 PR 이슈에 오류 리포팅 드린 내용도 확인 부탁드립니다.

@esezin
Copy link

esezin commented Aug 24, 2018

@kyungtaak
네, 포커스 관련 (세트로 구성된 입력 필드의 경우 focus-out event로도 Submit 허용) 내용은 일괄적용이 필요한 부분이라 반영되어 있지 않습니다.

@judy-exntu @DAVE-EXNTU
그리고 현재 리뷰 중인데요.
논의되었던 내용은 다 올바르게 적용 된 것으로 확인했습니다만,
위 논의 내용에는 없는, 몇 가지 필요한 것들이 보여서 아래와 같이 정리합니다.

  • MIN/MAX의 기입력된 데이터 값(place holder)에 천단위 구분자 ','적용 (e.g. 123456 -> 123,456) : 포커스와 마찬가지로 일괄적용 필요
  • MIN/MAX의 초기입력 상태에서, 숫자가 아닌 일반 문자 입력 시, '0'으로 반환됨 > '0'이 아닌 초기값 + 포커스인 상태여야 함

dave-taeho added a commit that referenced this issue Aug 27, 2018
@dave-taeho
Copy link
Contributor

@kyungtaak
Min / Max Placeholder의 수치를 아래와 같이 적용하고 있던 부분을 제거하였습니다.
계산식 : ( min - ( max - min ) * 0.05 )

@esezin
MIN/MAX의 초기입력 상태에서, 숫자가 아닌 일반 문자 입력 시, '0'으로 반환됨 => 처리하였습니다.

dave-taeho added a commit that referenced this issue Aug 27, 2018
kyungtaak pushed a commit that referenced this issue Aug 27, 2018
kyungtaak pushed a commit that referenced this issue Aug 27, 2018
ufoscw pushed a commit that referenced this issue Aug 28, 2018
ufoscw pushed a commit that referenced this issue Aug 28, 2018
ufoscw pushed a commit that referenced this issue Aug 28, 2018
ufoscw pushed a commit that referenced this issue Aug 28, 2018
@kyungtaak kyungtaak added this to the 3.1.0 milestone Aug 29, 2018
brandon-wonjune pushed a commit that referenced this issue Aug 29, 2018
brandon-wonjune pushed a commit that referenced this issue Aug 29, 2018
brandon-wonjune pushed a commit that referenced this issue Aug 29, 2018
brandon-wonjune pushed a commit that referenced this issue Aug 29, 2018
brandon-wonjune pushed a commit that referenced this issue Aug 29, 2018
brandon-wonjune pushed a commit that referenced this issue Aug 29, 2018
brandon-wonjune pushed a commit that referenced this issue Aug 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@dashboard Component Name : Dashboard / Chart enhancement Request Change and Feature Enhancement
Projects
None yet
Development

No branches or pull requests

6 participants