[Cloud Print] Implement Cloud Print section |
||||||||||||||||||
Issue descriptionThere are no mocks for Cloud Print.
,
Apr 14 2016
,
Apr 14 2016
,
Apr 20 2016
Yes, this is for MD settings - converting our existing cloud print UI to MD. I was planning on this being just a 1:1 swap, but if there are any updates to the existing UI/feature requests/etc, then it should be noted here.
,
Apr 27 2016
,
Apr 28 2016
Ready for triage Cloud print preview: https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Inner%20Pages/03-Settings/preview/cards#%2FPREVIEW-Printing-cloud.png%3Fz=width Card-specific spec https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Inner%20Pages/03-Settings/specs#%2FSPEC-cards_printers-cloud.png%3Fz=width WebUI subpage spec https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Inner%20Pages/03-Settings/specs#%2FSPEC-settings_structure-subpage.png
,
Apr 28 2016
Thanks Alan! Sorry for the delayed reply on this thread. This convo seemed to have existed in a couple different places which is my mistake. We had talked about getting rid of the printers_subpage and just having the toplevel button go to the web app for Cloud Print. I'm fine with just doing the conversion as you laid it out, but would be curious to see if Tom thinks differently.
,
Apr 29 2016
+bblietz for feedback on having Cloud Print settings become a link to the online management page.
,
May 24 2016
,
May 24 2016
so, just to make sure: we're just linking out to chrome://devices for now, right?
,
May 24 2016
,
May 24 2016
,
May 24 2016
Yes, we should link to the chrome://devices page. Note that the mocks still need to be updated with some additional info ( crbug.com/546838 ).
,
May 24 2016
Please provide updated mocks when they are available.
,
May 25 2016
,
May 25 2016
,
May 25 2016
,
May 25 2016
,
May 25 2016
,
May 27 2016
,
May 28 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/70c95b1e895c633a514b21b6d96fef3abe866ec9 commit 70c95b1e895c633a514b21b6d96fef3abe866ec9 Author: dbeam <dbeam@chromium.org> Date: Sat May 28 04:44:08 2016 MD Settings: implement printing page Adds the section as per the mocks on the bug and a side-nav item/route BUG= 602464 R=dschuyler@chromium.org TBR=michaelpg@chromium.org CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2017283002 Cr-Commit-Position: refs/heads/master@{#396635} [modify] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/app/settings_strings.grdp [modify] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/extensions/api/settings_private/prefs_util.cc [modify] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/resources/settings/advanced_page/advanced_page.html [modify] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/resources/settings/icons.html [add] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/resources/settings/printing_page/printing_page.html [add] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/resources/settings/printing_page/printing_page.js [modify] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/resources/settings/settings_menu/settings_menu.html [modify] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/resources/settings/settings_page/settings_router.js [modify] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/resources/settings/settings_resources.grd [modify] https://crrev.com/70c95b1e895c633a514b21b6d96fef3abe866ec9/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc
,
May 31 2016
,
Jun 1 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bbf308a144b4f4026d86d9826756c0c563c6eae2 commit bbf308a144b4f4026d86d9826756c0c563c6eae2 Author: dbeam <dbeam@chromium.org> Date: Wed Jun 01 12:57:41 2016 MD Settings: closure compile the printing page R=dschuyler@chromium.org BUG= 602464 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation NOTRY=true Review-Url: https://codereview.chromium.org/2025023003 Cr-Commit-Position: refs/heads/master@{#397117} [modify] https://crrev.com/bbf308a144b4f4026d86d9826756c0c563c6eae2/chrome/browser/resources/settings/compiled_resources2.gyp [add] https://crrev.com/bbf308a144b4f4026d86d9826756c0c563c6eae2/chrome/browser/resources/settings/printing_page/compiled_resources2.gyp
,
Jul 31 2016
|
||||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||||
Comment 1 by mshe...@chromium.org
, Apr 11 2016