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

Issue 597052 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

We should remove a menu item to intentionally offline a link before Async Loading is in experiment

Project Member Reported by petewil@chromium.org, Mar 22 2016

Issue description

To test offline page background loading, I'm adding a new menu item "Save linked page for later".

The menu item will only show if the experimental flag "offline-pages-background-load" is enabled.

We should decide if this is the best way to expose asking to save a page offline once we get more network, and what the best wording to use is (we have a 30 character limit), and if we are OK with users seeing this menu item if they enable the flag.

This item is shown at the bottom of the menu if you do a long press on a link.
 
Status: Assigned (was: Untriaged)
Rachel, please take a look. We are introducing a menu item so that we can test asynch browsing in a simpler way, while you can try to figure out a proper UI.

This is meant to be a temporary hack. We don't intend to ship it, unless UX decision is to do so.

Comment 2 by talo@chromium.org, Mar 30 2016

I think having this is fine to make testing easier, but I think we have to think more about it as an actual feature both in terms of other efforts that Clank folks are thinking about and to prioritize against are other work (since menu items are less readily used.
Yes, +1 to Tal's comment here.

Comment 4 by dim...@chromium.org, Mar 31 2016

Cc: dim...@chromium.org
Labels: -Pri-3 Pri-2
Owner: petewil@chromium.org
Summary: We should remove a menu item to intentionally offline a link before Async Loading is in experiment (was: We should have a menu item to intentionally offline a link)
Changing title and back to Peter as a tracking issue. This menu option stays a temporary tool for development. Lets remove it after we have Async Loading and before we go for visible experiment.
Labels: -Pri-2 Pri-3
No need to remove this for the current milestone, it is hidden behind a flag, lowering pri to 3.

Comment 6 by dim...@chromium.org, Sep 27 2016

Labels: -Pri-3 M-55 Pri-2
Lets keep the flag, remove the menu item.

Comment 7 by dim...@chromium.org, Oct 12 2016

Labels: -M-55 M-56
Status: Fixed (was: Assigned)
Removed now.

Sign in to add a comment