Update import details panel to comply (mostly) with UX direction.

chrome/app/chromeos_strings.grdp

 <?xml version="1.0" encoding="utf-8"?>
 <!-- ChromeOS-specific strings (included from generated_resources.grd).
      Everything in this file is wrapped in <if expr="chromeos">. -->
 <grit-part>
   <message name="IDS_EMPTY_STRING" desc="Empty string, exist only to make code generic. No translation required.">
   </message>
   <message name="IDS_CHROMEBOOK" desc="The localized name for Chromebook.">
     Chromebook
   </message>
   <message name="IDS_CHROMEBOX" desc="The localized name for Chromebox">
@@ skipping 430 matching lines @@
   </message>
   <message name="IDS_FILE_BROWSER_REMOVE_FOLDER_SHORTCUT_BUTTON_LABEL" desc="Menu item label, unpinning the selected folder from the left nav.">
     Remove shortcut
   </message>
   <message name="IDS_FILE_BROWSER_SHARE_BUTTON_LABEL" desc="Menu item label, showing dialog to share the selected file.">
     Share
   </message>
   <message name="IDS_FILE_BROWSER_CANCEL_SELECTION_BUTTON_LABEL" desc="Label for button that unselects all selected items.">
     Cancel selection
   </message>
-  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_BUTTON_LABEL" desc="Label for button that initiates media backup to the cloud.">
-    Import <ph name="FILE_COUNT">$1<ex>5</ex></ph> files to Google Drive
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_TITLE" desc="Title of the cloud import feature.">

[mtomasz, 2015/02/12 03:22:01]

nit: feature -> dialog?

[Steve McKay, 2015/02/12 23:15:53]

Used in a couple locations. So it is the general purpose feature title.

+    Cloud backup

[mtomasz, 2015/02/12 03:22:01]

nit: What's the final naming? Back up backup or import?

[Steve McKay, 2015/02/12 23:15:53]

This is the final naming. "Cloud backup", but the call to action is "Back Up".

   </message>
-  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_ACTIVE_IMPORT_BUTTON_LABEL" desc="Message to user that there is an active import in progress.">
-    Importing...
+  <message name="IDS_FILE_BROWSER_CLOUD_DESTINATION_FOLDER" desc="Title of the cloud import destination folder.">
+    ChromeOS Cloud backup

[mtomasz, 2015/02/12 03:22:00]

nit: ChromeOS -> Chrome OS
nit: Cloud backup -> cloud backup?

[Steve McKay, 2015/02/12 23:15:53]

That's the title. I dunno why we treat title like this, I don't personally like
it, but that's what UX wants.

"Chrome OS" + "Cloud backup".

   </message>
-  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_EMPTY_SCAN_BUTTON_LABEL" desc="Message to user that no new files were found, so cloud import is not possible.">
-    No new media
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_DESCRIPTION" desc="Short description of the cloud import feature.">
+    Back up your media device’s photos and videos to Google Drive.
   </message>
-  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_INSUFFICIENT_SPACE_BUTTON_LABEL" desc="Message to user that there is not enough space to perform a cloud import.">
-    Insufficient space on your device!
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_START" desc="Tooltip of the toolbar button that starts backup.">
+    Backup your photos and videos.

[mtomasz, 2015/02/12 03:22:01]

nit: Not period for tooltips?

[Steve McKay, 2015/02/12 23:15:53]

Done.

   </message>
-  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_SCANNING_BUTTON_LABEL" desc="Label for button showing that scanning is in progress...prior to being enabled when scanning is complete.">
-    Scanning your media...
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_SHOW_DETAILS" desc="Tooltip for button that shows details panel.">
+    Show Cloud backup details.

[mtomasz, 2015/02/12 03:22:00]

nit: ditto

[Steve McKay, 2015/02/12 23:15:53]

Done.

   </message>
-  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_ITEMS_REMAINING" desc="File Manager status message.">
-    Importing <ph name="FILE_COUNT">$1<ex>5</ex></ph> files...
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_COMMAND" desc="Label of button on details panel that invokes cloud import feature.">
+    Back up
   </message>
-
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_STATUS_SCANNING" desc="Cloud import scanning status message.">
+    <ph name="BEGIN_LINK">&lt;span class='leader'&gt;</ph>

[mtomasz, 2015/02/12 03:22:00]

nit: Do we need the class name in strings?

[Steve McKay, 2015/02/12 23:15:53]

It's better than having to pass it in. To be honest what we need is a real
templating system, because this arrangement is a ROYAL pain in the ass.

+    Scanning your media device...
+    <ph name="END_LINK">&lt;/span&gt;</ph>
+    <ph name="FILE_COUNT">$1<ex>5</ex></ph> new photos found
+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_TOOLTIP_SCANNING" desc="Cloud import scanning tooltip.">
+    Scanning your media device...
+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_STATUS_NO_MEDIA" desc="Cloud import no-new-media-found status message.">
+    <ph name="BEGIN_LINK">&lt;span class='leader'&gt;</ph>All backed up!<ph name="END_LINK">&lt;/span&gt;</ph>

[mtomasz, 2015/02/12 03:22:01]

nit: Remove span?

[Steve McKay, 2015/02/12 23:15:53]

No, that's the end </span> tag. Again, need a real templating system.

+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_TOOLTIP_NO_MEDIA" desc="Cloud import no-new-media-found tooltip.">
+    All backed up!

[mtomasz, 2015/02/12 03:22:00]

nit: Why no media shows "All backed up!"? Could you explain it in desc="..." so
people translating have more context?

[Steve McKay, 2015/02/12 23:15:53]

Done.

+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_STATUS_INSUFFICIENT_SPACE" desc="Cloud import insufficient space status message.">
+    <ph name="BEGIN_LINK">&lt;span class='leader'&gt;</ph>
+    <ph name="FILE_COUNT">$1<ex>5</ex></ph> new photos found
+    <ph name="END_LINK">&lt;/span&gt;</ph>
+    &lt;br&gt;
+    Not enough space on device.
+    &lt;br&gt;
+    Try selecting a few photos to start

[mtomasz, 2015/02/12 03:22:01]

nit: period missing

[Steve McKay, 2015/02/12 23:15:53]

Done.

+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_TOOLTIP_INSUFFICIENT_SPACE" desc="Cloud import insufficient space tooltip.">
+    Not enough space on device.

[mtomasz, 2015/02/12 03:22:00]

nit: No period for tooltips.
nit: on device -> on the device?

[Steve McKay, 2015/02/12 23:15:53]

Done.

+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_STATUS_READY" desc="Cloud import ready status message.">
+    <ph name="BEGIN_LINK">&lt;span class='leader'&gt;</ph>

[mtomasz, 2015/02/12 03:22:01]

I think we are putting to much in strings. If we change the UI, we will have to
send many strings to translation again. Can we split these strings? Especially
the "new photos found" should be a separate string.

[Steve McKay, 2015/02/12 23:15:53]

That makes things ever harder to manage from the controller side. Again, what we
need is a templating system...where that templating system supports automatic
extraction of strings for translation. We just don't have anything in Chrome
like that available to us.

+    <ph name="FILE_COUNT">$1<ex>5</ex></ph> new photos found
+    <ph name="END_LINK">&lt;/span&gt;</ph>
+    &lt;br&gt;
+    Ready to backup to <ph name="BEGIN_LINK">&lt;span class='destination-link'&gt;</ph>Google Drive<ph name="END_LINK">&lt;/span&gt;</ph>

[mtomasz, 2015/02/12 03:22:01]

nit: Do we need a class name in this string?

[Steve McKay, 2015/02/12 23:15:53]

Yes. In order to style this.

Before I submit, I'm going to try (again) using the ::first-line meta
selector...I wasn't getting ideal results when I frist tried. But if that works,
I can drop the markup of the first line from the strings...making them MUCH more
readable.

+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_TOOLTIP_READY" desc="Cloud import ready tooltip.">
+    Ready to backup <ph name="FILE_COUNT">$1<ex>5</ex></ph> new photos
+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_STATUS_IMPORTING" desc="Cloud import active-backup status.">

[mtomasz, 2015/02/12 03:22:01]

nit: People doing translations may not be famililar with the cloud import UI at
all. Can we put more useful context in desc="..."? Eg.
"Message shown when importing is in progress"?

[Steve McKay, 2015/02/12 23:15:53]

Done.

+    <ph name="BEGIN_LINK">&lt;span class='leader'&gt;</ph>
+    Backing up <ph name="FILE_COUNT">$1<ex>5</ex></ph> photos to <ph name="BEGIN_LINK">&lt;span class='destination-link'&gt;</ph>Google Drive<ph name="END_LINK">&lt;/span&gt;</ph>
+    <ph name="END_LINK">&lt;/span&gt;</ph>
+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_TOOLTIP_IMPORTING" desc="Cloud import active-backup tooltip.">
+    Backing up <ph name="FILE_COUNT">$1<ex>5</ex></ph> photos
+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_STATUS_DONE" desc="Cloud import backup is complete status.">
+    <ph name="BEGIN_LINK">&lt;span class='leader'&gt;</ph>
+    <ph name="FILE_COUNT">$1<ex>5</ex></ph> photos backed up to <ph name="BEGIN_LINK">&lt;span class='destination-link'&gt;</ph>Google Drive<ph name="END_LINK">&lt;/span&gt;</ph>
+    <ph name="END_LINK">&lt;/span&gt;</ph>
+  </message>
+  <message name="IDS_FILE_BROWSER_CLOUD_IMPORT_TOOLTIP_DONE" desc="Cloud import backup is complete tooltip.">
+    <ph name="FILE_COUNT">$1<ex>5</ex></ph> photos backed up
+  </message>
   <message name="IDS_FILE_BROWSER_COPY_FILE_NAME" desc="File Manager status message.">
     Copying <ph name="FILE_NAME">$1<ex>movie.avi</ex></ph>...
   </message>
   <message name="IDS_FILE_BROWSER_COPY_ITEMS_REMAINING" desc="File Manager status message. 'Item' is used here as a generic term for file or directory.">
     Copying $1 items...
   </message>
   <message name="IDS_FILE_BROWSER_COPY_TARGET_EXISTS_ERROR" desc="File Manager error message. 'Item' is used here as a generic term for file or directory.">
     Whoops, <ph name="FILE_NAME">$1<ex>movie.avi</ex></ph> already exists. Rename it and try again.
   </message>
   <message name="IDS_FILE_BROWSER_COPY_SOURCE_NOT_FOUND_ERROR" desc="File Manager error message. 'Item' is used here as a generic term for file or directory.">
@@ skipping 5561 matching lines @@
   <message name="IDS_FLAGS_DISABLE_RESOLVE_TIMEZONE_BY_GEOLOCATION_NAME" desc="Title for the flag to disable automatic timezone update on user location change.">
     Disable automatic timezone update by geolocation
   </message>
   <message name="IDS_FLAGS_DISABLE_RESOLVE_TIMEZONE_BY_GEOLOCATION_DESCRIPTION" desc="Description for the flag to disable user setting to automatically update timezone by user geolocation.">
     If set, automatic timezone update by the current geolocation is turned off.
   </message>
   <message name="IDS_OPTIONS_RESOLVE_TIMEZONE_BY_GEOLOCATION_DESCRIPTION" desc="Label for checkbox to allow automatic timezone update by user geolocation.">
     Automatically resolve timezone by geolocation
   </message>
 </grit-part>