Merge pull request #4417 from timvandermeij/examples-style

Making all examples adhere to the style guide

examples/acroforms/forms.js

@@ -13,16 +13,18 @@ function setupForm(div, content, viewport) {
   function bindInputItem(input, item) {
     if (input.name in formFields) {
       var value = formFields[input.name];
-      if (input.type == 'checkbox')
+      if (input.type == 'checkbox') {
         input.checked = value;
-      else if (!input.type || input.type == 'text')
+      } else if (!input.type || input.type == 'text') {
         input.value = value;
       }
+    }
     input.onchange = function pageViewSetupInputOnBlur() {
-      if (input.type == 'checkbox')
+      if (input.type == 'checkbox') {
         formFields[input.name] = input.checked;
-      else if (!input.type || input.type == 'text')
+      } else if (!input.type || input.type == 'text') {
         formFields[input.name] = input.value;
+      }
     };
   }
   function createElementWithStyle(tagName, item) {
@@ -61,8 +63,9 @@ function setupForm(div, content, viewport) {
       switch (item.subtype) {
         case 'Widget':
           if (item.fieldType != 'Tx' && item.fieldType != 'Btn' &&
-              item.fieldType != 'Ch')
+              item.fieldType != 'Ch') {
             break;
+          }
           var inputDiv = createElementWithStyle('div', item);
           inputDiv.className = 'inputHint';
           div.appendChild(inputDiv);
@@ -119,7 +122,6 @@ function renderPage(div, pdf, pageNumber, callback) {
     canvas.height = pageDisplayHeight;
     pageDivHolder.appendChild(canvas);
 
-
     // Render PDF page into canvas context
     var renderContext = {
       canvasContext: context,
@@ -141,8 +143,9 @@ PDFJS.getDocument(pdfWithFormsPath).then(function getPdfForm(pdf) {
   var viewer = document.getElementById('viewer');
   var pageNumber = 1;
   renderPage(viewer, pdf, pageNumber++, function pageRenderingComplete() {
-    if (pageNumber > pdf.numPages)
+    if (pageNumber > pdf.numPages) {
       return; // All pages rendered
+    }
     // Continue rendering of the next page
     renderPage(viewer, pdf, pageNumber++, pageRenderingComplete);
   });

examples/text-selection/js/minimal.js

@@ -1,21 +1,19 @@
-//Minimal PDF rendering and text-selection example using pdf.js by Vivin Suresh Paliath (http://vivin.net)
-//This example uses a built version of pdf.js that contains all modules that it requires.
+// Minimal PDF rendering and text-selection example using PDF.js by Vivin Suresh Paliath (http://vivin.net)
+// This example uses a built version of PDF.js that contains all modules that it requires.
 //
+// The problem with understanding text selection was that the text selection code has heavily intertwined
+// with viewer.html and viewer.js. I have extracted the parts I need out of viewer.js into a separate file
+// which contains the bare minimum required to implement text selection. The key component is TextLayerBuilder,
+// which is the object that handles the creation of text-selection divs. I have added this code as an external
+// resource.
 //
-//The problem with understanding text selection was that the text selection code has heavily intertwined
-//with viewer.html and viewer.js. I have extracted the parts I need out of viewer.js into a separate file
-//which contains the bare minimum required to implement text selection. The key component is TextLayerBuilder,
-//which is the object that handles the creation of text-selection divs. I have added this code as an external
-//resource.
+// This demo uses a PDF that only has one page. You can render other pages if you wish, but the focus here is
+// just to show you how you can render a PDF with text selection. Hence the code only loads up one page.
 //
-//This demo uses a PDF that only has one page. You can render other pages if you wish, but the focus here is
-//just to show you how you can render a PDF with text selection. Hence the code only loads up one page.
-//
-//The CSS used here is also very important since it sets up the CSS for the text layer divs overlays that
-//you actually end up selecting.
+// The CSS used here is also very important since it sets up the CSS for the text layer divs overlays that
+// you actually end up selecting.
 
 window.onload = function () {
-
   var scale = 1.5; //Set this to whatever you want. This is basically the "zoom" factor for the PDF.
   PDFJS.workerSrc = '../../src/worker_loader.js';
 
@@ -32,13 +30,13 @@ window.onload = function () {
     var viewport = page.getViewport(scale);
     var $canvas = jQuery("<canvas></canvas>");
 
-        //Set the canvas height and width to the height and width of the viewport
+    // Set the canvas height and width to the height and width of the viewport
     var canvas = $canvas.get(0);
     var context = canvas.getContext("2d");
     canvas.height = viewport.height;
     canvas.width = viewport.width;
 
-        //Append the canvas to the pdf container div
+    // Append the canvas to the pdf container div
     var $pdfContainer = jQuery("#pdfContainer");
     $pdfContainer.css("height", canvas.height + "px").css("width", canvas.width + "px");
     $pdfContainer.append($canvas);
@@ -53,7 +51,7 @@ window.onload = function () {
         left: canvasOffset.left
       });
 
-        //The following few lines of code set up scaling on the context if we are on a HiDPI display
+    // The following few lines of code set up scaling on the context if we are on a HiDPI display
     var outputScale = getOutputScale(context);
     if (outputScale.scaled) {
       var cssScale = 'scale(' + (1 / outputScale.sx) + ', ' +
@@ -76,12 +74,10 @@ window.onload = function () {
     $pdfContainer.append($textLayerDiv);
 
     page.getTextContent().then(function (textContent) {
-
       var textLayer = new TextLayerBuilder({
         textLayerDiv: $textLayerDiv.get(0),
         pageIndex: 0
       });
-
       textLayer.setTextContent(textContent);
 
       var renderContext = {