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

Ordinal dimensions, Bar Chart: Values are not classified according to their schema:position #820

Closed
ortnever opened this issue Nov 3, 2022 · 2 comments · Fixed by #821
Closed
Labels
bug Something isn't working

Comments

@ortnever
Copy link

ortnever commented Nov 3, 2022

Values are not classified according to their schema:position in Bar chart although it was the case before (see #580)

Steps to reproduce the behavior:

open and edit : https://test.visualize.admin.ch/en/v/4E2vn95fel3H

or

Open int.visualize.admin.ch, source : int
Search "Red list" and inculde draft Datasets
Select "Red list (latest survey with%)"
Start a visualization
Select Dimension "Status" as color

The order of stati is correct in the filters (on the right side and on the left side) but not in the chart and Legend

Expected behavior :
Values (in this case Stati) have to be classified according to their schema:position in all Charts. Please, could you do the correction for all chart's types at same time (see #580)

Screenshots

image

image

Environment (please complete the following information):

Visualize environment and version: test.visualize.admin.ch v3.11.0 ([a94a977]. Same Problem on INT v3.11.0 ([76d8044]
Browser : Edge

@ortnever ortnever added the bug Something isn't working label Nov 3, 2022
@ptbrowne
Copy link
Collaborator

ptbrowne commented Nov 3, 2022

@ortnever I was surprised because when developing this sorting, I tested this dataset and it worked correctly. I think it's because the order of the sorting was "identifier" then "position" instead of "position" then "identifier". It should be fixed in https://visualization-tool-git-fix-order-position-identifier-ixt1.vercel.app/en?dataSource=Int. Can you confirm ?

@ortnever
Copy link
Author

ortnever commented Nov 3, 2022

@ptbrowne Thanks Patrick. It's correct now and also for Pie Chart. That's great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants