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

improve review summary #446

Open
wbazant opened this issue Sep 2, 2024 · 4 comments
Open

improve review summary #446

wbazant opened this issue Sep 2, 2024 · 4 comments
Labels
bug Unexpected problems or unintended behavior enhancement New feature request good first issue Good for newcomers
Milestone

Comments

@wbazant
Copy link
Collaborator

wbazant commented Sep 2, 2024

I noticed my test review with just a picture is dragging down an overall score, unlike on the live site:

Screenshot from 2024-09-02 11-07-38

Screenshot from 2024-09-02 11-09-10
d

The proposed resolution is to skip empty scores, so that an average of 3, empty, and 5 is 4, not 3.

@wbazant wbazant changed the title skip empty reviews when calculating summary score skip empty reviews when calculating summary score (or remove summary) Sep 15, 2024
@wbazant
Copy link
Collaborator Author

wbazant commented Sep 15, 2024

Relatedly, #416 proposes:

  • Move the review summary to the location overview (or remove)

I'm thinking that maybe the review summary should go away entirely, instead of getting fixed. The new layout allows for plenty of space for reviews, I think locations typically don't have many reviews - maybe 1 or 2, there's no need to summarize them - and it's questionable whether taking an average is the right way of summarizing reviews.

Also, relatedly: 'does this location have reviews, and how many' is in itself a summary, and a very objective one! #364 (comment) proposes to only show overview/reviews tab if there are reviews.

@ezwelty
Copy link
Collaborator

ezwelty commented Oct 3, 2024

@wbazant Nice catch. I don't know, I kinda like the way reviews are summarized at https://fallingfruit.org, especially for fruiting status. A nice example is https://www.fallingfruit.org/locations/1517357. The design of the review summary is more compact than the one currently implemented in the beta, and it makes much more sense to have it in the location/overview tab than in the review tab.

I agree that presently this is useful for only a few locations (only 3000 locations have more than 2 reviews) but hopefully this number will increase in the future?

@wbazant wbazant changed the title skip empty reviews when calculating summary score (or remove summary) improve review summary Oct 4, 2024
@wbazant
Copy link
Collaborator Author

wbazant commented Oct 4, 2024

I agree the review summary is less good over the reviews, and some summary being present in the overview could be good, especially in the half-open drawer: there's currently a needless surprise in the UX when the overview | reviews top ribbon pops up.

Thanks for mentioning 1517357, it is a good example of reviews working well!

Comparing it to https://beta.fallingfruit.org/locations/1517357, I also like the quality scores being done with words instead of with stars, and they would help to make the review summary shorter. HTML mockup:
Screenshot from 2024-10-04 09-16-27

@ezwelty what do you think about the stars for scores? I think they could remain as an input feature if hovering on stars makes the corresponding word appear on the screen, like:
Screenshot from 2024-10-04 08-49-47
Then the reviews list would either use words and stars or just words, and the overview would use just words.

So I would propose changes as follows:

  • on mobile, move the review summary into the overview tab
  • fix the algorithm (remove empty scores when summarising)
  • change the review form and review list to use qualitative scores for 1-5 values
  • change the UI of overview summary to only use words, be visually coherent, and fit on one line on mobile
  • keep the two-column UI on desktop, just check it looks okay (something wrong with the whitespace there currently)

@ezwelty
Copy link
Collaborator

ezwelty commented Oct 7, 2024

@wbazant In your mockup, I would probably drop the number of reviews and show number of reviews in parentheses after each label-value pair. I would also suggest formatting the labels and values differently, as on the old website, for legibility.

Quality Good 3.2/5 (1) · Yield Excellent 5.0/5 (2)
Unripe fruit August (1), September (1) · Ripe fruit September (1)

Regarding how to input and display the ratings... We started with words (Poor, Fair, Good, etc) but these are loaded terms that don't always translate well (between people or languages). It is also hard to numerically average them. So gradually we started to treat them as numbers.

I'm not sure what the best way is, but tempted to suggest using stars/numbers only:

  • Make the review form stars require a click. The current hover is buggy and makes it look like a rating has been set even when it hasn't.
  • In review, display either as 5 stars with selected stars orange (* * * x x), or as a number and an orange star (3 *). Either way, I think the stars should immediately follow their associated label (Quality or Yield) and layout should allow both be on the same line.
  • In overview, show the numeric average and either 5 stars with (partially) selected stars orange or an orange star. Again layout should allow both ratings to sit on the same line.

@wbazant wbazant added bug Unexpected problems or unintended behavior enhancement New feature request good first issue Good for newcomers labels Nov 10, 2024
@wbazant wbazant added this to the Beta release milestone Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected problems or unintended behavior enhancement New feature request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants