Provide feedback on successful fetch but no new suggestions. |
||||||||
Issue descriptionWhen the user initiates a fetch on the Chrome Home suggestions surface and it fails (eg they have no connectivity) we show a Snackbar with the feedback "Can't get suggestions". However, if the user initiates a fetch and it succeeds but there are no new relevant suggestions for the user, we don't show anything. We should show some distinct feedback, eg "No new suggestions". Shimi, do you approved of this string?
,
Oct 18 2017
@srahim, do you approve of the string?
,
Oct 18 2017
,
Oct 26 2017
,
Oct 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5942cc1510f27e73f11f62fcbefc08fc74c1ffc5 commit 5942cc1510f27e73f11f62fcbefc08fc74c1ffc5 Author: Peter E Conn <peconn@chromium.org> Date: Thu Oct 26 13:52:04 2017 🏡 Display Snackbar when a user triggered fetch returns no results. Display a different Snackbar if the user triggers a fetch, it succeeds but there are no new suggestions. Bug: 663376, 775478 Change-Id: Ib85cb397bf612fb570046aea2e7a7e6e125d535c Reviewed-on: https://chromium-review.googlesource.com/708754 Commit-Queue: Peter Conn <peconn@chromium.org> Reviewed-by: Bernhard Bauer <bauerb@chromium.org> Reviewed-by: Michael van Ouwerkerk <mvanouwerkerk@chromium.org> Cr-Commit-Position: refs/heads/master@{#511810} [modify] https://crrev.com/5942cc1510f27e73f11f62fcbefc08fc74c1ffc5/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/ActionItem.java [modify] https://crrev.com/5942cc1510f27e73f11f62fcbefc08fc74c1ffc5/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/SectionList.java [modify] https://crrev.com/5942cc1510f27e73f11f62fcbefc08fc74c1ffc5/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSection.java [modify] https://crrev.com/5942cc1510f27e73f11f62fcbefc08fc74c1ffc5/chrome/android/java/src/org/chromium/chrome/browser/snackbar/Snackbar.java [modify] https://crrev.com/5942cc1510f27e73f11f62fcbefc08fc74c1ffc5/chrome/android/java/strings/android_chrome_strings.grd [modify] https://crrev.com/5942cc1510f27e73f11f62fcbefc08fc74c1ffc5/chrome/android/junit/src/org/chromium/chrome/browser/ntp/cards/SuggestionsSectionTest.java [modify] https://crrev.com/5942cc1510f27e73f11f62fcbefc08fc74c1ffc5/tools/metrics/histograms/enums.xml
,
Nov 7 2017
Hey Shimi! We have a small change to make to Zine. We can't fetch new articles after the user hits "more" (no internet connection) we show a snackbar for feedback. The current string is "Unable to get suggestions". Would you prefer for us to use something else, or does that work for now? Cheers, Rachel
,
Nov 15 2017
Hi Rachel, if the reason we can't fetch more articles is no internet connection, I'd prefer we say "No internet connection". If we need a generic "we don't know what the error is", then how about "Can't get articles"? I prefer "articles" here instead of "suggestions." We may have more suggested articles lined up, we just can't show them. Contrast with when we run out of suggestions to show, we have "no more suggestions."
,
Nov 15 2017
,
Nov 15 2017
Also, I agree w/#1: However, if the user initiates a fetch and it succeeds but there are no new relevant suggestions for the user, we don't show anything. We should show some distinct feedback, eg "No new suggestions".
,
Nov 17 2017
Great, thanks Shimi! Pete - does this cover all the use cases we'd need here? +dgn - nicolas, would you mind pointing us at the strings doc you prepared a while back? I'd like to make sure we're keeping consistency across our strings.
,
Nov 17 2017
,
Nov 17 2017
I'm not sure I understand your distinction between Articles and Suggestions, I figure: No Internet: Can't get suggestions Internet, nothing new: No new suggestions Is a bit more consistent then No Internet: Can't get articles Internet, nothing new: No new suggestions
,
Nov 17 2017
re #c10: Do you mean this document? https://docs.google.com/document/d/1LUHuImA04-VGArgHX6I-1rBAyQ5HDPFgcnEnZgtGDL0/
,
Dec 18 2017
Any update here?
,
Dec 19 2017
This was implemented on 26th Oct (making it into M64) with the strings: Can’t get suggestions No new suggestions It was left open to get an OK for the string choice.
,
Dec 21 2017
Is rachelis@ the right owner for this blocker? rachelis@ are you working on this issue? Please update.
,
Jan 5 2018
Is there anything pending here? Please note that M64 Stable is very soon, and this is marked a blocker.
,
Jan 6 2018
I emailed rachelis@ and I hope to hear from them soon.
,
Jan 10 2018
#15 is OK with me and should not block M64 Stable launch.
,
Jan 10 2018
Based on #15 and 19 |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by peconn@chromium.org
, Oct 17 2017To give you more context, here is a video of the proposed change. The first part of the video shows existing functionality when a fetch fails ("Unable to get suggestions" has since been changed to "Can't get suggestions."). Later in the video we see a fetch that is successful (the server returns a response), but provides no new suggestions and the feedback that the Snackbar could provide.15.3 MB
15.3 MB Download