ThreeSixtyGiving / grantnav

This is a web based search tool for data in the 360 giving data format.
http://grantnav.threesixtygiving.org/
Other
9 stars 5 forks source link

Work out what we want to appear in the search summary box #597

Closed robredpath closed 3 years ago

robredpath commented 3 years ago

We've left a blank space where the search summary needs to go.

Agile suggested a very attractive design with an illustration, which they can develop further for us if we want.

The search summary box serves two purposes for all searches:

For well-refined searches, it also provides genuinely helpful aggregations.

However, there is a significant risk that people take the aggregations at face value when they don't support that, but the search results don't suggest that something is amiss.

We can help by:

robredpath commented 3 years ago

I've started this out of sequence because I think it will take some back-and-forth early on to get it to the point where we can implement.

I've put some ideas into a prototype at https://threesixtygiving.github.io/grantnav/prototypes/search_results_summary.html

robredpath commented 3 years ago

For this sprint, we're going to implement the same summary box as we currently have, with some appropriate copy to warn people about the potential pitfalls.

We'll speak with agile.coop separately to look at how we can get a new design, and when we've got a better idea of timescales, we can implement their work.

Lathrisk commented 3 years ago

Summary boxes also appear on the top of other pages, such as funders, recipients, etc, pages. We might need to decide is this is going to be largely the same summary box that appears on the search page. I am going to assume the same pattern for now.

robredpath commented 3 years ago

@Lathrisk there shouldn't be summary boxes on the funder page (as there aren't already) - but there are other tables, if that's what you mean?

Lathrisk commented 3 years ago

Ok, there is a top-stats box at the top of the funders page that contains the same stats as the summary stats at the top of the search page (with minor differences). I thought the new summary stats box would replace both of these, perhaps incorrectly.

Funders page: grantnav threesixtygiving org_funder_GB-COH-RC000766 (1)

Search page: grantnav-search

robredpath commented 3 years ago

@Lathrisk ooooohhhhh I see, you mean the funder pages (eg http://grantnav.threesixtygiving.org/funder/GB-LAE-GLA ), not the funders page ( http://grantnav.threesixtygiving.org/funders )

I think that keeping the existing format for those tables is fine, at least for now.

robredpath commented 3 years ago

(SUPER confusing terminology, almost set up to trap the unwary.....)

drkane commented 3 years ago

Search summary box for GrantNav

drkane commented 3 years ago

Search summary box is looking great. Some feedback from us - we can meet to talk this through if helpful:

Headings

Charts:

I would aim for as simple charts as possible - would be good to see what it looks like with just the bars and labels for the bars, and remove all the other parts of the charts.

Right hand side stats:

Show highlighted grants

Link to insights

mariongalley commented 3 years ago

Call with David x2 & Michael:

Have agreed to add a limit to the size of search we'll send to Insights. For now we'll set it to 1,000 and increase it if we can fix the insights memory issue. Please can the Insights call to action be conditional:

michaelwood commented 3 years ago

Dev instance of GN has been updated to the search-summary-box branch http://grantnav.dev.default.threesixtygiving.uk0.bigv.io/search

mariongalley commented 3 years ago

Following fixes have been done:

The following are still to do:

Lathrisk commented 3 years ago

Quick note, the work that Michael has deployed to the dev server is WIP, and there are still a few things to be done. I didn't realise it would be deployed or reviewed at this point. I'll update once the work is complete. I expect that will be tomorrow, hopefully morning.

P.S. One issue might require more work than we currently have time for: smallest grant - should be smallest greater than 0

@kindly (David Raznick) is looking into that but if we run out of time we might need to raise this as a separate ticket and come back to it

mariongalley commented 3 years ago

Re: Link to Insights, when we do change it to be conditional can we change the wording from 360Giving Insights to 360Insights? I checked with Katie (Comms) and she confirmed that's how we refer to it on our website and in the brand guidelines.

mariongalley commented 3 years ago

Understood, thanks @Lathrisk :)

Lathrisk commented 3 years ago

@kindly (David Raznick) is looking into that but if we run out of time we might need to raise this as a separate ticket and come back to it

I've written this up as a separate ticket. We're hoping to deploy the most recent changes to dev in the morning.

mariongalley commented 3 years ago

Call with Jared, David R, Michael and David K:

All changes described in this comment above are done and look good, except those raised as separate tickets (#760 #761), neither of which is critical to the release.