New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 763390 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Task



Sign in to add a comment

[Contextual Suggestions] Offline icon does not appear when suggestion is downloaded

Project Member Reported by galinap@google.com, Sep 8 2017

Issue description

Currently we show a offline icon when an article suggestion is downloaded, but we don't show any icon when a contextual suggestion is downloaded. It would be nice to show the offline icon for consistency.
 
Summary: [Contextual Suggestions] Offline icon does not appear when suggestion is downloaded (was: [Contextual Suggestions] Offline icon does not appear when suggestion is not downloaded)

Comment 2 by fi...@chromium.org, Sep 11 2017

Labels: zine-triaged
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ec25514c37e94aaff1545a5080516fc1f9e925dd

commit ec25514c37e94aaff1545a5080516fc1f9e925dd
Author: galinap <galinap@google.com>
Date: Wed Sep 13 11:39:41 2017

[Contextual Suggestions] Add offline icon when suggestion is downloaded

This CL adds an offline icon when the user downloads a suggestion from
the context menu. This is done by registering an OfflineModelObserver
in the SuggestionsCarouselAdapter.

Bug:  763390 
Change-Id: I8cf9adf54706927b5f11628c7e330b42eb4f2e94
Reviewed-on: https://chromium-review.googlesource.com/660538
Reviewed-by: Nicolas Dossou-Gbété <dgn@chromium.org>
Commit-Queue: Galia Peycheva <galinap@google.com>
Cr-Commit-Position: refs/heads/master@{#501594}
[modify] https://crrev.com/ec25514c37e94aaff1545a5080516fc1f9e925dd/chrome/android/java/src/org/chromium/chrome/browser/suggestions/ContextualSuggestionsCardViewHolder.java
[modify] https://crrev.com/ec25514c37e94aaff1545a5080516fc1f9e925dd/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsBottomSheetContent.java
[modify] https://crrev.com/ec25514c37e94aaff1545a5080516fc1f9e925dd/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsCarousel.java
[modify] https://crrev.com/ec25514c37e94aaff1545a5080516fc1f9e925dd/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsCarouselAdapter.java

Comment 4 by galinap@google.com, Sep 13 2017

Status: Fixed (was: Assigned)

Sign in to add a comment