Changed look of pin keyboard.

chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.html

 <!-- TODO(crbug.com/603217): Use i18n instead of string literals. Figure out
                              what i18n to use for keypad, ie, does 1 ABC make
                              sense in every scenario? -->
 
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-styles/paper-styles.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icons/iron-icons.html">
 
 <dom-module id="pin-keyboard">
   <template>
     <style>
       #root {
         display: flex;
       }
 
-      /* TODO(jdufault): Remove this CSS once we inline the PIN element with the
-       * user-pod. */
-      #container-constrained-width {
-        -webkit-tap-highlight-color: transparent;
-        background: white;
-        border-radius: 2px;
-        box-shadow: 0 4px 23px 5px rgba(0, 0, 0, 0.2),
-                    0 2px 6px rgba(0, 0, 0, 0.15),
-                    0 3px 0 rgba(0, 0, 0, 0.08);
-        cursor: pointer;
-        flex-direction: column;
-        outline: none;
-      }
-
       .row {
         display: flex;
       }
 
       .row.horizontal-center {
         justify-content: center;
       }
 
       .digit-button {
         align-items: center;
         background: none;
         border-radius: 50px;
+        box-sizing: border-box;
         display: flex;
+        flex-direction: column;
         font-size: 30px;
+        height: 70px;
         justify-content: center;
-        margin: 5px;
-        min-width: 94px;
-        width: 94px;
+        margin: 0;
+        min-height: 70px;
+        min-width: 85px;
+        padding: 15px 20px;
+        width: 85px;
+      }
+
+      .digit-button.clear-button {
+        display: block;
+      }
+
+      .digit-button .icon-div {
+        background: var(--paper-grey-500);
+        border-radius: 100px;
+        height: 24px;
+        margin: auto;

[jdufault, 2016/06/22 23:18:00]

Is this needed?

[sammiequon, 2016/06/22 23:43:28]

Yes.

+        width: 24px;
+      }
+
+      .digit-button .icon-div .icon {
+        color: white;
+        display: block;

[jdufault, 2016/06/22 23:18:00]

Is display: block needed?

[sammiequon, 2016/06/22 23:43:28]

Yes.

+        height: 16px;
+        left: 4px;
+        top: 4px;
+        width: 16px;
+      }
+
+      .digit-button .icon-text {

[jdufault, 2016/06/22 23:18:01]

Can you please verify that all of the CSS here contains the minimum that needs
to be specified?

[sammiequon, 2016/06/22 23:43:28]

Yes.

+        color: var(--paper-grey-500);
+        font-size: 10px;
+        padding: 10px;
+        text-transform: capitalize;
       }
 
       .digit-button inner-text {
         display: flex;
         flex-direction: column;
         height: 52px;
       }
 
-      .digit-button inner-text subhead {
-        color: var(--paper-grey-500);
-        font-size: 16px;
-      }
-
-      .digit-button .submit-button {
-        background: var(--paper-blue-a400);
-        color: white;
-        font-size: 20px;
-        height: 60px;
-        min-width: 0;
-        width: 60px;
+      .first-row {
+        height: 54px;
+        min-height: 54px;
       }
 
       #pin-input {
         -webkit-text-security: disc;
-        border-bottom-color: lightgray;
-        border-left: none;
-        border-right: none;
-        border-top: none;
+        border: 0;
         font-size: 20px;
-        height: 30px;
-        margin-top: 10px;
+        height: 54px;
         text-align: center;
-        width: 95%;
+        width: 75%;

[jdufault, 2016/06/22 23:18:00]

Can you eliminate the % and use a margin/offset instead?

[sammiequon, 2016/06/22 23:43:28]

Done.

       }
 
       #pin-input[type=number]::-webkit-inner-spin-button,
       #pin-input[type=number]::-webkit-outer-spin-button {
         -webkit-appearance: none;
         margin: 0;
       }
 
       /* Ensure that all children of paper-button do not consume events. This
        * simplifies the event handler code. */
       paper-button * {
         pointer-events: none;
       }
     </style>
 
     <div id="root">
       <div id="container-constrained-width">
         <div class="row horizontal-center">
           <input id="pin-input" type="number" placeholder="Enter PIN"
                  on-keydown="onInputKeyDown_"></input>
+          <div class="digit-button first-row">
+            <paper-button class="digit-button submit-button"
+                          on-tap="onPinSubmit_">
+              <div class="icon-div">
+                  <iron-icon class="icon" icon="icons:forward"><iron-icon>
+              </div>
+            </paper-button>
+          </div>
         </div>
 
         <div class="row keyboard">
           <paper-button class="digit-button" on-tap="onNumberTap_" value="1">
             <inner-text>1</inner-text>
           </paper-button>
           <paper-button class="digit-button" on-tap="onNumberTap_" value="2">
-            <inner-text>2<subhead>ABC</subhead></inner-text>
+            <inner-text>2</inner-text>
           </paper-button>
           <paper-button class="digit-button" on-tap="onNumberTap_" value="3">
-            <inner-text>3<subhead>DEF</subhead></inner-text>
+            <inner-text>3</inner-text>
           </paper-button>
         </div>
         <div class="row keyboard">
           <paper-button class="digit-button" on-tap="onNumberTap_" value="4">
-            <inner-text>4<subhead>GHI</subhead></inner-text>
+            <inner-text>4</inner-text>
           </paper-button>
           <paper-button class="digit-button" on-tap="onNumberTap_" value="5">
-            <inner-text>5<subhead>JKL</subhead></inner-text>
+            <inner-text>5</inner-text>
           </paper-button>
           <paper-button class="digit-button" on-tap="onNumberTap_" value="6">
-            <inner-text>6<subhead>MNO</subhead></inner-text>
+            <inner-text>6</inner-text>
           </paper-button>
         </div>
         <div class="row keyboard">
           <paper-button class="digit-button" on-tap="onNumberTap_" value="7">
-            <inner-text>7<subhead>PQRS</subhead></inner-text>
+            <inner-text>7</inner-text>
           </paper-button>
           <paper-button class="digit-button" on-tap="onNumberTap_" value="8">
-            <inner-text>8<subhead>TUV</subhead></inner-text>
+            <inner-text>8</inner-text>
           </paper-button>
           <paper-button class="digit-button" on-tap="onNumberTap_" value="9">
-            <inner-text>9<subhead>WXYZ</subhead></inner-text>
+            <inner-text>9</inner-text>
           </paper-button>
         </div>
         <div class="row keyboard">
           <div class="digit-button"></div>
           <paper-button class="digit-button" on-tap="onNumberTap_" value="0">
             <inner-text>0</inner-text>
           </paper-button>
-          <div class="digit-button">
-            <paper-icon-button class="digit-button submit-button"
-                               icon="icons:check" on-tap="onPinSubmit_">
-            </paper-icon-button>
-          </div>
+          <paper-button class="digit-button clear-button" on-tap="onPinClear_">
+            <div class="icon-div">

[jdufault, 2016/06/22 23:18:00]

Is there a better class name than icon-div and icon-text? icon-container?
icon-subheading?

[sammiequon, 2016/06/22 23:43:28]

Done.

+              <iron-icon class="icon" icon="icons:clear"><iron-icon>
+            </div>
+            <inner-text class="icon-text">Clear</inner-text>
+          </paper-button>
         </div>
       </div>
     </div>
   </template>
   <script src="pin_keyboard.js"></script>
 </dom-module>