Links behind the reading list header are tappable |
||||
Issue descriptionApp Version: 63.0.3205.0 canary iOS Version: 10.3.3, 11.0 Device : iPhone,iPad Precondition : 1- Have few links in Reading list Steps to reproduce: 1. Launch chrome 2. Tap on Menu > Tap on Reading list. 3. Scroll the link upwards. 4. Long tap on Reading list header. Observed results: Context menu is displayed Expected results: Context menu should not be displayed Number of times you were able to reproduce: 5/5 Bug reproducible after clean install: Yes Bug reproducible after clearing cache and cookies: Yes Bug reproducible on Chrome Mobile on Android: NA Bug reproducible on Safari/Firefox: NA Bug reproducible on current stable build (App Version, iOS Version): Yes in M60 Bug reproducible on the current beta channel build (App Version, iOS Version): Yes in M61 and 62 Link to Video : https://drive.google.com/a/google.com/file/d/0B--UpU2GW2EpYUJ5TmNYTVA0QlE/view?usp=sharing
,
Sep 7 2017
,
Sep 12 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2079cb829384606ea06e2fd1956c101ef9da56d9 commit 2079cb829384606ea06e2fd1956c101ef9da56d9 Author: Gauthier Ambard <gambard@chromium.org> Date: Tue Sep 12 10:58:21 2017 Don't trigger long press on ReadingList app bar When the user long press the app bar displayed at the top of the Reading List collection, the context menu corresponding to the item displayed below should not be displayed. Bug: 761988 Change-Id: Iee203182aa896096875b04b886db0818bf6226ac Reviewed-on: https://chromium-review.googlesource.com/660001 Reviewed-by: Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#501232} [modify] https://crrev.com/2079cb829384606ea06e2fd1956c101ef9da56d9/ios/chrome/browser/ui/reading_list/reading_list_collection_view_controller.mm
,
Sep 12 2017
,
Oct 9 2017
Verified the issue on the latest build 63.0.3236.0 canary tested on iPhone7+(11.0.1). Links behind reading list heard are not tappable, works fine. |
||||
►
Sign in to add a comment |
||||
Comment 1 by sczs@chromium.org
, Sep 6 2017Status: Assigned (was: Untriaged)