Print Preview: Fading in/out printing options.

chrome/browser/resources/print_preview.js

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 var localStrings = new LocalStrings();
 
 // The total page count of the previewed document regardless of which pages the
 // user has selected.
 var totalPageCount = -1;
 
@@ skipping 139 matching lines @@
  */
 function updateControlsWithSelectedPrinterCapabilities() {
   var printerList = $('printer-list');
   var selectedIndex = printerList.selectedIndex;
   if (selectedIndex < 0)
     return;
 
   var selectedValue = printerList.options[selectedIndex].value;
   if (selectedValue == PRINT_TO_PDF) {
     updateWithPrinterCapabilities({'disableColorOption': true,
-                                   'setColorAsDefault': true});
+                                   'setColorAsDefault': true,
+                                   'disableCopiesOption': true});
   } else if (selectedValue == MANAGE_PRINTERS) {
     printerList.selectedIndex = lastSelectedPrinterIndex;
     chrome.send('managePrinters');
     return;
   } else {
     // This message will call back to 'updateWithPrinterCapabilities'
     // function.
     chrome.send('getPrinterCapabilities', [selectedValue]);
   }
 
   lastSelectedPrinterIndex = selectedIndex;
 
   // Regenerate the preview data based on selected printer settings.
   setDefaultValuesAndRegeneratePreview();
 }
 
 /**
  * Updates the controls with printer capabilities information.
  * @param {Object} settingInfo printer setting information.
  */
 function updateWithPrinterCapabilities(settingInfo) {
   var disableColorOption = settingInfo.disableColorOption;
+  var disableCopiesOption = settingInfo.disableCopiesOption;
   var setColorAsDefault = settingInfo.setColorAsDefault;
   var colorOption = $('color');
   var bwOption = $('bw');
 
-  if (disableColorOption)
-    $('color-options').classList.add("hidden");
-  else
-    $('color-options').classList.remove("hidden");
+  if (disableCopiesOption) {
+    fadeOutElement($('copies-option'));
+    $('hr-before-copies').classList.remove('invisible');
+  } else {
+    fadeInElement($('copies-option'));
+    $('hr-before-copies').classList.add('invisible');
+  }
+
+  disableColorOption ? fadeOutElement($('color-options')) :
+      fadeInElement($('color-options'));
 
   if (colorOption.checked != setColorAsDefault) {
     colorOption.checked = setColorAsDefault;
     bwOption.checked = !setColorAsDefault;
     setColor(colorOption.checked);
   }
 }
 
 /**
  * Disables or enables all controls in the options pane except for the cancel
@@ skipping 247 matching lines @@
  * Called when the PDF plugin loads its document.
  */
 function onPDFLoad() {
   if (isLandscape())
     $('pdf-viewer').fitToWidth();
   else
     $('pdf-viewer').fitToHeight();
 
   $('dancing-dots').classList.add('invisible');
 
-  if (!previewModifiable) {
-    $('landscape').disabled = true;
-    $('portrait').disabled = true;
-  }
+  if (!previewModifiable)
+    fadeOutElement($('landscape-option'));
 
   updateCopiesButtonsState();
 }
 
 /**
  * Update the print preview when new preview data is available.
  * Create the PDF plugin as needed.
  * Called from PrintPreviewUI::PreviewDataIsAvailable().
  * @param {number} pageCount The expected total pages count.
  * @param {string} jobTitle The print job title.
@@ skipping 85 matching lines @@
  */
 function checkCompatiblePluginExists() {
   var dummyPlugin = $('dummy-viewer')
   return !!dummyPlugin.onload;
 }
 
 /**
  * Updates the state of print button depending on the user selection.
  * The button is enabled only when the following conditions are true.
  * 1) The selected page ranges are valid.
- * 2) The number of copies is valid.
+ * 2) The number of copies is valid (if applicable).
  */
 function updatePrintButtonState() {
-  $('print-button').disabled = (!isNumberOfCopiesValid() ||
-                                getSelectedPagesValidityLevel() != 1);
+  if (getSelectedPrinterName() == PRINT_TO_PDF)
+    $('print-button').disabled = (getSelectedPagesValidityLevel() != 1);
+  else
+    $('print-button').disabled = (!isNumberOfCopiesValid() ||

[Lei Zhang, 2011/05/27 22:58:01]

nit: add parenthesis.

[dpapad, 2011/05/27 23:16:17]

Done. BTW I thought we prefer to use parenthesis only when required.

[Lei Zhang, 2011/05/27 23:27:30]

Multi-line else statement = required.

[Lei Zhang, 2011/05/27 23:28:55]

Err, I meant to say braces. that is:

if () {
...
} else {
...
}

+                                  getSelectedPagesValidityLevel() != 1);
 }
 
 window.addEventListener('DOMContentLoaded', onLoad);
 
 /**
  * Listener function that executes whenever a change occurs in the 'copies'
  * field.
  */
 function copiesFieldChanged() {
   updateCopiesButtonsState();
   $('collate-option').hidden = getCopies() <= 1;
 }
 
 /**
  * Executes whenever a blur event occurs on the 'individual-pages'
  * field or when the timer expires. It takes care of
  * 1) showing/hiding warnings/suggestions
  * 2) updating print button/summary
  */
 function pageRangesFieldChanged() {
   var currentlySelectedPages = getSelectedPagesSet();
   var individualPagesField = $('individual-pages');
   var individualPagesHint = $('individual-pages-hint');
   var validityLevel = getSelectedPagesValidityLevel();
 
   if (validityLevel == 1) {
     individualPagesField.classList.remove('invalid');
-    hideInvalidHint(individualPagesHint);
+    fadeOutElement(individualPagesHint);
   } else {
     individualPagesField.classList.add('invalid');
     individualPagesHint.classList.remove('suggestion');
     individualPagesHint.innerHTML =
         localStrings.getStringF('pageRangeInstruction',
                                 localStrings.getString(
                                     'examplePageRangeText'));
-    showInvalidHint(individualPagesHint);
+    fadeInElement(individualPagesHint);
   }
 
   resetPageRangeFieldTimer();
   updatePrintButtonState();
   updatePrintSummary();
 }
 
 /**
  * Updates the state of the increment/decrement buttons based on the current
  * 'copies' value.
  */
 function updateCopiesButtonsState() {
   var copiesField = $('copies');
   if (!isNumberOfCopiesValid()) {
     copiesField.classList.add('invalid');
     $('increment').disabled = false;
     $('decrement').disabled = false;
-    showInvalidHint($('copies-hint'));
+    fadeInElement($('copies-hint'));
   }
   else {
     copiesField.classList.remove('invalid');
     $('increment').disabled = (getCopies() == copiesField.max) ? true : false;
     $('decrement').disabled = (getCopies() == copiesField.min) ? true : false;
-    hideInvalidHint($('copies-hint'));
+    fadeOutElement($('copies-hint'));
   }
 }
 
 /**
  * Updates the print summary based on the currently selected user options.
  *
  */
 function updatePrintSummary() {
   var copies = getCopies();
   var printSummary = $('print-summary');
+  var printToPDF = getSelectedPrinterName() == PRINT_TO_PDF;

[Lei Zhang, 2011/05/27 22:58:01]

move this to the top and set copies based on printToPDF.

[dpapad, 2011/05/27 23:16:17]

Done.

 
-  if (!isNumberOfCopiesValid()) {
+  if (printToPDF)
+    copies = 1;
+
+  if (!printToPDF && !isNumberOfCopiesValid()) {
     printSummary.innerHTML = localStrings.getString('invalidNumberOfCopies');
     return;
   }
 
   if (getSelectedPagesValidityLevel() != 1) {
     printSummary.innerHTML = '';
     return;
   }
 
   var pageList = getSelectedPagesSet();
   var numOfSheets = pageList.length;
   var sheetsLabel = localStrings.getString('printPreviewSheetsLabelSingular');
   var numOfPagesText = '';
   var pagesLabel = '';
 
-  if (isTwoSided())
+  if (!printToPDF && isTwoSided())
     numOfSheets = Math.ceil(numOfSheets / 2);
   numOfSheets *= copies;
 
   if (numOfSheets > 1)
     sheetsLabel = localStrings.getString('printPreviewSheetsLabelPlural');
 
   var html = '';
   if (pageList.length * copies != numOfSheets) {
     numOfPagesText = pageList.length * copies;
     pagesLabel = localStrings.getString('printPreviewPageLabelPlural');
@@ skipping 38 matching lines @@
     return;
 
   $('individual-pages').classList.remove('invalid');
   setDefaultValuesAndRegeneratePreview();
 }
 
 /**
  * Sets the default values and sends a request to regenerate preview data.
  */
 function setDefaultValuesAndRegeneratePreview() {
-  hideInvalidHint($('individual-pages-hint'));
+  fadeOutElement($('individual-pages-hint'));
   totalPageCount = -1;
   previouslySelectedPages.length = 0;
   requestPrintPreview();
 }
 
 /**
  * Returns a list of all pages in the specified ranges. The pages are listed in
  * the order they appear in the 'individual-pages' textbox and duplicates are
  * not eliminated. If the page ranges can't be parsed an empty list is
  * returned.
@@ skipping 210 matching lines @@
   this.isLandscape = '';
 }
 
 /**
  * Takes a snapshot of the print settings.
  */
 PrintSettings.prototype.save = function() {
   this.deviceName = getSelectedPrinterName();
   this.isLandscape = isLandscape();
 }