New issue
Advanced search Search tips

Issue 681627 link

Starred by 1 user
Status: Verified
Owner:
gambard@chromium.org
Closed: Jan 2017
Cc:
ios-bugs@chromium.org
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

Move "Read Later" as the 3rd choice in contextual menu

Project Member Reported by olivierrobin@chromium.org, Jan 16 2017 
Open in new tab is the main action of the contextual menu.
"Read Later" should come after the two open tabs entry.
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 17 2017 

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

commit 5fd4034984b118b932a891b849320c147f47e702
Author: gambard <gambard@chromium.org>
Date: Tue Jan 17 09:17:53 2017

Move "Read Later" to the third place in the context menu

BUG= 681627 

Review-Url: https://codereview.chromium.org/2631393002
Cr-Commit-Position: refs/heads/master@{#444010}

[modify] https://crrev.com/5fd4034984b118b932a891b849320c147f47e702/ios/chrome/browser/ui/browser_view_controller.mm

Comment 2 by gambard@chromium.org, Jan 17 2017

Status: Fixed (was: Assigned)

Comment 3 by pmadalla@chromium.org, Jan 24 2017

Status: Verified (was: Fixed)
Verified the issue on the latest canary 58.0.2991.0 canary tested on iPhone7+(iOS 10).
Read later is moved as the 3rd choice in context menu, looks good
►

Sign in to add a comment