search

ICJIA / arrest_explorer

Distribution of, and interface to, aggregated Illinois arrest data.
https://icjia.illinois.gov/arrestexplorer
1 stars 0 forks source link

Axes should be synchronized for all plots sharing screen. #15

Open ErnstMelchior opened 3 years ago

ErnstMelchior commented 3 years ago

Describe Issue If I go by Arrests by Race By Age Group, bar or line, the graphs use different scales, so it seems like Asians have as many arrests as Whites. That will be briefly confusing and make us look silly. Location of Text or Control

Arrests by race by age group. image

Current Text or Control The scale at left of the graphs

Proposed new text/control Make all graphs scale to the current largest graph so that it is clear that Asians have fewer arrests. If users want more detail, they can just filter by Asians. If the graph has multiple groups, it could be for purposes of comparison, and the comparison, visually, should work.

kgruschow commented 3 years ago

in general the default should be to default option to off, and label what happens if the function is turned on. so it would be nice to change from a "standardize axes" toggle that defaults to on, to relabeling it to be "optimize axes to data range" that defaults to off. this way when the user navigates to find the option, they understand a little bit about why this would be desirable.

it would be nice to see the options to optimize max and min separately.

closely related but optional: if there is a way to add a visual indicator to plots when zero is not the bottom for vertical axis, then it would be desirable to add if it can be done cheaply (e.g. squiggle line)

kgruschow commented 3 years ago

This is in a decent place. I think remove the option to unlock the lower bound, and change the upper bound option to be read "Scale vertical axis per graph" and then we can close this.

miserman commented 3 years ago

8ee9607eb9edf0c25cd2fc0eec04f9e34e8c23c1 makes those changes.