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

Issue 729922 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Feature



Sign in to add a comment

Improve Add to Shelf/Desktop dialog on Desktop

Project Member Reported by mgiuca@chromium.org, Jun 6 2017

Issue description

Modernize this dialog and probably remove the editable title and "Open as window" checkbox, at least for PWAs that have already set their own preferences.
 
Cc: ainslie@chromium.org
Cc: dominickn@chromium.org
We should probably look to reuse the existing AddToHomescreenManager/DataFetcher code that exists for Android. This does the proper PWA checking, which isn't yet done on desktop when using add to shelf. That code could be made cross-platform with platform-specific subclasses like the app banner code.

Comment 4 by mgiuca@chromium.org, Jun 16 2017

For now, just remove these two things when behind the new feature flag (name TBA). Eventually, we'll want the PWA check logic, and allow editing when it's a non-PWA, but for now just remove it so we can get a feel for the experience.

Comment 5 by mgiuca@chromium.org, Jun 16 2017

Cc: -calamity@chromium.org mgiuca@chromium.org
Owner: calamity@chromium.org
Components: UI>Browser>WebAppInstalls
Project Member

Comment 7 by bugdroid1@chromium.org, Oct 31 2017

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

commit e849b5187b2fc04e70742394466624c934ccf8cb
Author: Christopher Lam <calamity@chromium.org>
Date: Tue Oct 31 06:03:53 2017

[desktop-pwas] Remove 'Open as window' checkbox from bookmark app dialog.

This CL removes the 'Open as window' checkbox from the bookmark app
dialog when DesktopPWAWindowing is enabled and chooses to open as a
window if the site is an installable PWA and as a tab otherwise.

This requires adding an InstallableManager to the BookmarkAppHelper to
determine if the site is an installable PWA.

Bug:  729922 
Change-Id: I30d6329d52a9fabe314e6a8b76aa46e209cd171f
Reviewed-on: https://chromium-review.googlesource.com/700137
Reviewed-by: Ben Wells <benwells@chromium.org>
Reviewed-by: Dominick Ng <dominickn@chromium.org>
Commit-Queue: calamity <calamity@chromium.org>
Cr-Commit-Position: refs/heads/master@{#512761}
[modify] https://crrev.com/e849b5187b2fc04e70742394466624c934ccf8cb/chrome/browser/banners/app_banner_infobar_delegate_desktop.cc
[modify] https://crrev.com/e849b5187b2fc04e70742394466624c934ccf8cb/chrome/browser/extensions/bookmark_app_helper.cc
[modify] https://crrev.com/e849b5187b2fc04e70742394466624c934ccf8cb/chrome/browser/extensions/bookmark_app_helper.h
[modify] https://crrev.com/e849b5187b2fc04e70742394466624c934ccf8cb/chrome/browser/extensions/bookmark_app_helper_unittest.cc
[modify] https://crrev.com/e849b5187b2fc04e70742394466624c934ccf8cb/chrome/browser/extensions/favicon_downloader.cc
[modify] https://crrev.com/e849b5187b2fc04e70742394466624c934ccf8cb/chrome/browser/extensions/favicon_downloader.h
[modify] https://crrev.com/e849b5187b2fc04e70742394466624c934ccf8cb/chrome/browser/ui/views/extensions/bookmark_app_confirmation_view.cc

Comment 8 by mgiuca@chromium.org, Mar 26 2018

Labels: M-67

Comment 9 by mgiuca@chromium.org, Mar 27 2018

Status: Fixed (was: Assigned)
Looks done to me.

Sign in to add a comment