codeforamerica / criminal_case_search

Prototype app for filtering NYC Arrest/Arraignment data based on flexible criteria.
BSD 3-Clause "New" or "Revised" License
6 stars 2 forks source link

Indicate how many incidents are being shown. #150

Open djoaquin opened 11 years ago

djoaquin commented 11 years ago

e.g. image

djoaquin commented 11 years ago
waltz commented 11 years ago

I vaugely remember there being an overall mockup but I can't find it in the Dropbox. It'd be good for me to be able to see the whole mockup while I'm working. 

On October 8, 2013 at 12:38:16 PM, djoaquin (notifications@github.com) wrote:

Should be plural, "Displaying Incidents" not "Displaying Incident". Should only indicate how many actual incidents are on the page, "Displaying Incidents 1 - 15 of 105 Matching Incidents". Should also appear at the top of search results: "Found 105 Matching Incidents". — Reply to this email directly or view it on GitHub.

djoaquin commented 11 years ago

Here is an updated mockup. Above the search results would be the total number of incidents (what we had before) and below would have an indication of how many results are on the current page of the total matching. "Showing" = "Displaying"

image

waltz commented 11 years ago

Great, thanks. On October 8, 2013 at 1:55:15 PM, djoaquin (notifications@github.com) wrote:

Here is an updated mockup. Above the search results would be the total number of incidents (what we had before) and below would have an indication of how many results are on the current page of the total matching. "Showing" = "Displaying"

— Reply to this email directly or view it on GitHub.

djoaquin commented 11 years ago

"Incident" should be "Incidents" at the bottom of the page.

image

djoaquin commented 11 years ago

To be consistent with the top line, incidents should be lowercase