econ-dashboard / econ-dashboard.github.io

1 stars 0 forks source link

Create search bar on landing page #27

Open rsyoh-97 opened 3 years ago

rsyoh-97 commented 3 years ago
miguelito34 commented 3 years ago

Notes from discussion with Serdar on 5/5:

rsyoh-97 commented 3 years ago

Notes from Serdar's Slack (5.6.2021)

miguelito34 commented 3 years ago

Something that came to mind, we likely only want to show/search through counties for which we actually have the data. Ideally, we can cover all counties across the US, but we'll need to decide (if it even comes up), how to handle counties for which we're missing some or all of the data.

zstumgoren commented 3 years ago

@miguelito34 That's a very good point about the display of counties in search. I could see handling this in one of two ways:

  1. As part of the preprocessing pipeline, only include counties for which you have data in the JSON file that feeds Fuse or QuickSearch (can't recall which one you settled on)
  2. Include all counties in the dropdown menu/search, but counties that are not supported should be styled visually so as to indicate that they're not available. For example, you could use a lighter gray font for "inactive" counties. You may need some additional annotation beyond that to ensure clarity.

Approach 1 has the downside that users may find it confusing if they can't find their county. You could alleviate this with a note somewhere in the UI that explains that not all counties are yet supported. Alternative 2 avoids this confusion but will require some very clear visual indicator (and possibly textual explanation) that greyed out counties are not yet supported.

Either approach is acceptable, though personally I'd lean towards stategy 2.