My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 152090: FR: add print button to photo editor in CrOS files app
5 people starred this issue and may be notified of changes. Back to list
 
Project Member Reported by melissad...@chromium.org, Sep 24, 2012

Google Chrome	23.0.1271.2 (Official Build 157709) canary
Platform	2913.3.0 (Official Build) canary-channel lumpy

What steps will reproduce the problem?
Users report that printing images or files from the file manager/file app is not intuitive. 

From one EDU/Enterprise user:

"So, I had a student who wanted to print a picture off of her Chromebook.  You can't print directly out of the download file on the CB, you can't print straight out of Drive, and you can't print out of apps like pixlr express and picmonkey either.  Also, using Picassa web you can't print.  So, I finally found that you can get a print option in pixlr, but does anyone have the answer for native printing off of the CB?"

Please use labels and text to provide additional information.
Sep 24, 2012
#1 angel...@chromium.org
Depending on what version the EDU user is on, they may be running into crosbug.com/22539 which was fixed in R22
Sep 24, 2012
#2 melissad...@chromium.org
Either way, from the current build, there's still not a print icon in File Manager which seems a little silly.

I'm wondering if we could, after making the file manager changes, add one? 

Is it unrealistic? 
Sep 24, 2012
#3 angel...@chromium.org
Updating the title to make this FR more clear
Summary: FR: add print button to photo editor in CrOS files app
Labels: -Feature-Ash -Feature-CloudPrint
Sep 24, 2012
#4 joshwood...@chromium.org
@kenmoore - WDYT? We don't really have UI for this right now. Perhaps in a context menu for those users that want it?
Status: Assigned
Owner: kenmo...@chromium.org
Labels: Mstone-24
Oct 24, 2012
#5 joshwood...@chromium.org
Pri-2 + branch nearing = Bulk punt to M25. 

Please pull this back to M24 if you're actively working on this!
Labels: -Mstone-24 Mstone-25
Dec 10, 2012
#6 joshwood...@chromium.org
Impending branch point + Pri-2 = M25 -> M26
Labels: -Mstone-25 Mstone-26
Dec 13, 2012
#7 joshwood...@chromium.org
(No comment was entered for this change.)
Status: WontFix
Dec 27, 2012
#8 kenmo...@chromium.org
I agree we should add some affordance, it's lame to only support an invisible Ctrl+P to print.

If we add a context menu with Print on it for the photo gallery view, it seems that context menu should also include Slideshow, Edit, Delete and Share.  But then it feels arbitrary that Print is the only menu item there that doesn't have a corresponding icon -- I don't see a good reason not to include it, especially once  Issue 167712  gets fixed so the buttons will never overlap the thumbnails.

So the proposed context menu would be:
  Slideshow
  Edit
  Print
  Delete
  Share

... and the buttons in the bottom right would be in that order left to right also.

@joshwoodward, thoughts?

I've attached assets just in case


icon_print.png
158 bytes   View   Download
icon_print_selected.png
162 bytes   View   Download
icon_print_2x.png
204 bytes   View   Download
icon_print_selected_2x.png
207 bytes   View   Download
Dec 27, 2012
#9 joshwood...@chromium.org
SGTM. M26 is looking quite full so adding it to M27 for now.

@satorux - Who in TOK should be our go-to Feature-Photo person?
Status: Assigned
Owner: sato...@chromium.org
Labels: -Feature-FileManager -Mstone-26 Feature-Photo Mstone-27
Jan 15, 2013
#10 sato...@chromium.org
Let me move Feature=Photo issues to mtomasz@
Owner: mtomasz@chromium.org
Jan 18, 2013
#11 lafo...@google.com
(No comment was entered for this change.)
Labels: -Feature-Photo Feature-FileManager-Photo
Mar 10, 2013
#12 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-UI -Mstone-27 -Feature-FileManager-Photo Cr-UI Cr-Platform-Apps-PhotoEditor M-27
Mar 20, 2013
#13 joshwood...@chromium.org
(No comment was entered for this change.)
Labels: M-28
Mar 20, 2013
#14 joshwood...@chromium.org
(No comment was entered for this change.)
Labels: -M-27
Apr 26, 2013
#15 joshwood...@chromium.org
Let's try to do this as a part of the Photo Editor new UI.
Labels: -Pri-2 -M-28 Pri-1 M-29
Jun 6, 2013
#16 jamescook@chromium.org
This would make me happy - I couldn't figure out how to print a photo last night.

Cc: jamescook@chromium.org
Jun 6, 2013
#17 mtomasz@chromium.org
(No comment was entered for this change.)
Labels: Iteration-84
Jun 14, 2013
#18 joshwood...@chromium.org
We will pick this up in M-30 with photo editor updates.
Labels: -M-29 M-30
Jun 17, 2013
#19 joshwood...@chromium.org
(No comment was entered for this change.)
Labels: -Iteration-84 Iteration-85
Jun 24, 2013
#20 mtomasz@chromium.org
(No comment was entered for this change.)
Status: Started
Jun 26, 2013
#22 mtomasz@chromium.org
(No comment was entered for this change.)
Status: Fixed
Jun 26, 2013
#23 bugdro...@chromium.org
------------------------------------------------------------------------
r208692 | mtomasz@chromium.org | 2013-06-26T12:48:00.720030Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/generated_resources.grd?r1=208692&r2=208691&pathrev=208692
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/chromeos/extensions/file_manager/file_browser_private_api.cc?r1=208692&r2=208691&pathrev=208692
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/file_manager/css/gallery.css?r1=208692&r2=208691&pathrev=208692
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/file_manager/js/photo/gallery.js?r1=208692&r2=208691&pathrev=208692
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/file_manager/js/photo/slide_mode.js?r1=208692&r2=208691&pathrev=208692

Add printing to the Files.app's gallery.

This patch adds the icon for printing in the slide mode. Also, enabled the shortcut (Ctrl+P) to invoke the dialog.

TEST=Open any picture, hit ctrl+p or hit on the button. Printing should be disabled in the mosaic view.
BUG=152090

Review URL: https://chromiumcodereview.appspot.com/17603006
------------------------------------------------------------------------
Jun 27, 2013
#24 patri...@chromium.org
(No comment was entered for this change.)
Labels: VerifyIn-29
Jul 24, 2013
#25 ka...@chromium.org
Verified in
Google Chrome	30.0.1573.2 (Official Build 213041) dev
Platform	4443.0.0 (Official Build) dev-channel
Status: Verified
Sign in to add a comment

Powered by Google Project Hosting