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

Error display for counted row on text table(grid) #160

Closed
AnnieHwang opened this issue Sep 12, 2018 · 3 comments
Closed

Error display for counted row on text table(grid) #160

AnnieHwang opened this issue Sep 12, 2018 · 3 comments
Assignees
Labels
bug Something isn't working @chart Chart Component and Widget p2 Priority 2
Milestone

Comments

@AnnieHwang
Copy link
Contributor

Describe the bug
텍스트그리드의 연산행 중 항목수(Count)를 선택하면, 모든 값이 동일하게 컬럼 ROW 갯수가 반환됩니다.

Expected behavior
텍스트그리드의 연산행 중 항목수(Count)를 선택하면, 값이 존재하는 유효한 ROW의 수를 반환되어야 합니다.

Screenshots
image

@AnnieHwang AnnieHwang added bug Something isn't working @dashboard Component Name : Dashboard / Chart awaiting feedback need to feedback labels Sep 12, 2018
@esezin
Copy link

esezin commented Sep 12, 2018

@AnnieHwang
항목수가 잘못 표기되고 있습니다. 위 스크린샷 기준으로 첫번 째 항목인 ‘core infra 본부’의 항목수는 ‘1’이 맞습니다.

이와 관련된 이슈가 있습니다.
현재 연산자의 선택과 무관하게, 연산행의 레이블은 총합(Grand Total)로 되어 있습니다. 따라서 연산자를 합계가 아닌 것으로 변경해도 (특별히 사용자가 레이블을 수동으로 수정하지 않을 경우) 계속 ‘총합’으로 나와서 혼동을 줄 여지가 있습니다.
따라서, 아래와 같이 진행하는 것이 어떨까 합니다.

  1. 연산자 선택에 따라, ‘레이블 변경’의 초기 값이 해당 연산자 이름으로 변경 (연산자 : ‘항목수’ —> 레이블 : ‘항목수’)
  2. 단 사용자가 레이블을 수동으로 수정한 경우, 해당 이름으로 고정
    2.1. 사용자가 레이블을 삭제한 경우, 초기 값으로 복귀
    2.2. 복귀된 초기 값은 현재의 선택된 연산자와 동일

@kyungtaak
Copy link
Contributor

@esezin 저는 괜찬은것 같습니다만, @metatron-app/design 팀 내부적으로 결론을 내셨는지요?

@AnnieHwang
Copy link
Contributor Author

@kyungtaak 네 해당내용은 내부적으로 논의한 내용입니다.

@kyungtaak kyungtaak added @chart Chart Component and Widget and removed @dashboard Component Name : Dashboard / Chart awaiting feedback need to feedback labels Nov 11, 2018
@kyungtaak kyungtaak added this to the 3.2.0 milestone Dec 14, 2018
@kyungtaak kyungtaak added the p2 Priority 2 label Jan 7, 2019
eltriny pushed a commit that referenced this issue Jan 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working @chart Chart Component and Widget p2 Priority 2
Projects
None yet
Development

No branches or pull requests

4 participants