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

A sort error occurs if the data grid contains blanks. #1489

Closed
ufoscw opened this issue Feb 21, 2019 · 10 comments
Closed

A sort error occurs if the data grid contains blanks. #1489

ufoscw opened this issue Feb 21, 2019 · 10 comments
Assignees
Labels
bug Something isn't working @workbench Component Name : Workbench
Milestone

Comments

@ufoscw
Copy link
Contributor

ufoscw commented Feb 21, 2019

Describe the bug
A sort error occurs if the data grid contains blanks.
The sort result is incorrect when clicking on the header.
And the arrow coordinates are wrong when clicking the header.

To Reproduce
Steps to reproduce the behavior:

  1. Go to workbench
  2. execute query
select "1", "a"
union select "2", "b"
union select "3", "";
  1. click second column header 3 times.
  2. See 3 different sort result

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@ufoscw ufoscw added bug Something isn't working @workbench Component Name : Workbench labels Feb 21, 2019
@ninezero90hy
Copy link
Contributor

@ufoscw 현재 메타트론 그리드의 헤더를 클릭할 때 화살표 아이콘 변경은 SlickGrid 라이브러리의 기본 동작입니다
아래 첨부한 샘플(SlickGrid 멀티 컬럼 정렬 예제) 확인 부탁드립니다

slickgrid-sort

  • 메타트론 워크벤트 그리드 정렬

metatron-slickgrid-sort

@ufoscw
Copy link
Contributor Author

ufoscw commented Mar 4, 2019

@ninezero90hy
화살표 아이콘 관련해서 리포트한 내용은 화살표의 y좌표가 클릭전과 다르게 움직이는 내용입니다.
올려주신 gif에서 클릭 후 화살표 y좌표가 "No." 컬럼에 비해 상단에 붙어 버리네요.

@ninezero90hy
Copy link
Contributor

@ninezero90hy
화살표 아이콘 관련해서 리포트한 내용은 화살표의 y좌표가 클릭전과 다르게 움직이는 내용입니다.
올려주신 gif에서 클릭 후 화살표 y좌표가 "No." 컬럼에 비해 상단에 붙어 버리네요.

@ufoscw 아이콘 위치 변화를 말하신 거군요 확인해보겠습니다

@ufoscw
Copy link
Contributor Author

ufoscw commented Mar 4, 2019

@ninezero90hy
참고로 재현방법 3번의
3. click second column header 3 times.
은 마지막 컬럼의 헤더를 클릭했을때를 말합니다.

@ufoscw
Copy link
Contributor Author

ufoscw commented Mar 4, 2019

@ninezero90hy

참고로 재현방법 3번의
3. click second column header 3 times.
은 마지막 컬럼의 헤더를 클릭했을때를 말합니다.

mar-04-2019 11-41-50

@ninezero90hy
Copy link
Contributor

@ninezero90hy

참고로 재현방법 3번의
3. click second column header 3 times.
은 마지막 컬럼의 헤더를 클릭했을때를 말합니다.

mar-04-2019 11-41-50

@ufoscw 브라우저가 혹시 IE인가요?

@ufoscw
Copy link
Contributor Author

ufoscw commented Mar 4, 2019

@ninezero90hy
mac용 크롬입니다.

@ninezero90hy
Copy link
Contributor

ninezero90hy commented Mar 4, 2019

@ninezero90hy
mac용 크롬입니다.

@ufoscw 네 저도 맥/크롬을 사용중인데 재현이 안되네요. 확인해보겠습니다

@ninezero90hy
Copy link
Contributor

@ufoscw

  • 아이콘 위치 변경되는 오류 / 정렬 변경 오류 수정했습니다.
    아래와 같이 나오면 되는지 확인 부탁드립니다
select "1", "a"
union select "2", "b"
union select "", "";

fix-grid

@ninezero90hy
Copy link
Contributor

@ufoscw

  • 아이콘 위치 변경되는 오류 / 정렬 변경 오류 수정했습니다.
    아래와 같이 나오면 되는지 확인 부탁드립니다
select "1", "a"
union select "2", "b"
union select "", "";

fix-grid

@ufoscw 해당 결과를 기준으로 PR 올렸습니다. 확인 부탁드립니다

@eltriny eltriny added this to the 3.2.0 milestone Mar 4, 2019
ninezero90hy added a commit that referenced this issue Mar 4, 2019
alchan-lee pushed a commit that referenced this issue Mar 5, 2019
* #1489 Fix grid override css

* #1489 Fix grid sorting error

* #1489 Fix grid sorting error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working @workbench Component Name : Workbench
Projects
None yet
Development

No branches or pull requests

4 participants