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

Issue 845563 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 31
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Task

Blocking:
issue 833177


Show other hotlists

Hotlists containing this issue:
Hotlist-1


Sign in to add a comment

Make "Open In..." feature more discoverable

Project Member Reported by eugene...@chromium.org, May 22 2018

Issue description

App Version (from "Chrome Settings > About Chrome"): All
iOS Version: All
Device: All

Steps to reproduce: 
0.) Make sure that UI-refresh flag is not enabled (crbug.com/833177)
1.) Load a PDF page (e.g. https://www.irs.gov/pub/irs-pdf/fw4.pdf)
3.) Tap on PDF page

Observed behavior: 
Open In... toolbar is shown after tapping on PDF page, which is not very discoverable

Expected behavior: 
Always keep Open In... toolbar on PDF page

 
Cc: gambard@chromium.org marq@chromium.org
Labels: M-69
Mardini, Open In... is broken for UI Refresh (crbug.com/833177). Instead of fixing the toolbar positioning, maybe we should just implement this feature for M69?
Agreed. We should just implement this for M69. 
Thanks. 
We should ensure that "always keeping Open In ... toolbar on PDF Page" work smoothly with the show/hide of our bottom toolbar on scroll. 
Labels: -Pri-2 Pri-1
Cc: sdefresne@chromium.org
Owner: eugene...@chromium.org
Status: Assigned (was: Untriaged)
Owner: marq@chromium.org
Assigning to Mark to distribute UI-refresh work.
Blocking: 833177
Gauthier: would this be something you'd take on given it's blocking issue 833177 ?

Comment 9 by marq@chromium.org, Jun 13 2018

Owner: gambard@chromium.org
Labels: Proj-UIRefresh
Labels: -Type-Feature MS-Adaptive-Toolbar medium Type-Task
Labels: -Pri-1 Pri-2
Cc: martijnb@chromium.org
From mardini@ in issue 833177
"Please note that we'd like to change the behaviour slightly to make that "Open In ..." toolbar a bit more discoverable. So we should show it on loading the PDF and on scroll away. I don't think we should have it "always on"."
I think that a nice behaviour would be:

- Show it after the PDF loads, no time out. Keep it visible on scroll. 
- Hide it when pressing anywhere on the PDF. (Exception: When Tools menu is visible, and user dismisses tools menu, open in should stay visible)
- Show it again when pressing anywhere on the pdf. 
Labels: -Pri-2 -M-69 M-71 Pri-1
That sounds reasonable to me. Thank you.

Updating milestone and priority. I'm assuming it's too late for M70. 


Project Member

Comment 16 by bugdroid1@chromium.org, Aug 31

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

commit f3d123417511dd0f812f50e0440da26de0742f6b
Author: Gauthier Ambard <gambard@chromium.org>
Date: Fri Aug 31 15:49:37 2018

[iOS] Increase visibility of OpenIn for pdfs

This CL increases the visibility of the OpenIn toolbar in the pdf pages.
The toolbar is now visible when the page is loaded and stays visible
until the user interacts with the page (tap, scroll...).
It is also appearing when the user scrolls up (same as the bottom
toolbar behavior).

Bug:  845563 
Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: I27b2018288c11cc4500a5f48ed47be4bad1d3848
Reviewed-on: https://chromium-review.googlesource.com/1198770
Reviewed-by: Mark Cogan <marq@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#588034}
[modify] https://crrev.com/f3d123417511dd0f812f50e0440da26de0742f6b/ios/chrome/browser/ui/open_in_controller.mm

Status: Fixed (was: Assigned)

Sign in to add a comment