chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.html - Issue 2222583002: Lock screen pin keyboard ui upgrades.

Unified Diff: chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.html

Issue 2222583002: Lock screen pin keyboard ui upgrades. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkgr

Patch Set: Created 4 years, 4 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« no previous file with comments | « chrome/app/generated_resources.grd ('k') | chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.js » ('j') | chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

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

diff --git a/chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.html b/chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.html

index 4de7620dff9b27cab69aec7536646b80f1f1baf1..ccc1608b171e8ccd353d163c196d8778d6f5dfa2 100644

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

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

@@ -27,68 +27,60 @@

.digit-button {

align-items: center;

background: none;

- border-radius: 20px;

+ border-radius: 100px;

box-sizing: border-box;

- color: var(--paper-blue-grey-700);

+ color: #000;

display: flex;

flex-direction: column;

- font-size: 32px;

- height: 70px;

+ font-size: 18px;

+ height: 48px;

justify-content: center;

- margin: 0;

- min-height: 70px;

- min-width: 70px;

- padding: 15px 20px;

- width: 70px;

+ margin: 0 6px;

+ min-height: 48px;

+ min-width: 48px;

+ opacity: 0.87;

+ padding: 15px;

+ width: 48px;

}

- .digit-button.clear-button {

- display: block;

+ .digit-button.bottom-row-button {

+ margin-bottom: 6px;

}

- .digit-button.center-button {

- margin: 0 30px;

+ .digit-button.top-row-button {

+ margin-top: 6px;

}

- .digit-button.submit-button {

- left: 20px;

- position: relative;

+ .digit-button.backspace-button {

+ opacity: var(--dark-primary-opacity);

}

- .digit-button.submit-button.submit-button-hidden {

- display: none;

+ .digit-button.backspace-button[disabled] {

+ color: #000;

+ opacity: 0.26;

+ padding: 14px;

}

- .digit-button .icon-container {

- background: var(--paper-grey-400);

- border-radius: 100px;

- height: 24px;

- margin: auto;

- width: 24px;

- }

- .digit-button .icon-container.ready-background {

- background: var(--paper-blue-800);

+ .digit-button.submit-button {

+ color: var(--google-blue-500);

+ height: 43px;

+ left: 137px;

+ margin: 0;

+ min-height: 43px;

+ min-width: 43px;

+ opacity: var(--light-primary-opacity);

+ padding: 11.5px 10px;

+ position: absolute;

+ width: 43px;

}

- .digit-button .icon-container .icon {

- color: white;

- display: block;

- height: 16px;

- left: 4px;

- top: 4px;

- width: 16px;

+ .digit-button.submit-button[disabled] {

+ color: #000;

+ opacity: 0.26;

}

- .digit-button .icon-subheading {

- color: var(--paper-grey-400);

- font-size: 10px;

- left: 0;

- position: absolute;

- text-align: center;

- text-transform: capitalize;

- top: 44px;

- width: 100%;

+ .digit-button.submit-button.submit-button-hidden {

+ display: none;

}

.digit-button inner-text {

@@ -98,31 +90,40 @@

height: 52px;

}

- .first-row {

- height: 64px;

- min-height: 64px;

- padding: 28px 15px 12px 15px;

- }

#pin-input {

-webkit-text-security: disc;

background-color: white;

border: 0;

box-sizing: border-box;

- font-size: 18px;

- height: 64px;

- left: 20px;

+ font-face: Roboto-Regular;

+ font-size: 13px;

+ height: 43px;

+ left: 10px;

+ opacity: var(--dark-secondary-opacity);

outline: 0;

- padding: 28px 3px 12px 3px;

position: relative;

- width: 180px;

+ text-color: #000;

+ width: 127px;

+ }

+ #pin-input.ready-background {

+ opacity: var(--dark-primary-opacity);

}

#pin-input.submit-button-hidden {

- border-bottom: 2px;

- border-bottom-style: solid;

+ left: 0;

text-align: center;

- width: 230px;

+ width: 180px;

+ }

+ #pin-seperator {

jdufault 2016/08/05 21:58:51 nit: sp separator

sammiequon 2016/08/05 23:34:50 Done.

+ border: 0;

+ border-bottom: 1px solid #000;

+ left: 10px;

+ margin: 0;

+ opacity: 0.14;

+ position: relative;

+ width: 160px;

}

#pin-input[type=number]::-webkit-inner-spin-button,

@@ -140,31 +141,33 @@

- <div class="row horizontal-center">

+ <div class="row">

<input id="pin-input" type="[[getInputType_(enablePassword)]]"

class$="[[getInputClass_(value)]]

- [[getSubmitHiddenClass_(enableSubmitButton)]]"

+ [[getSubmitClass_(enableSubmitButton)]]

+ [[getPasswordReadyClass_(value)]]"

placeholder="[[getInputPlaceholder_(enablePassword)]]"

value="{{value::input}}"

on-keydown="onInputKeyDown_"></input>

- <paper-button class$="digit-button first-row submit-button

- [[getSubmitHiddenClass_(enableSubmitButton)]]"

- on-tap="firePinSubmitEvent_">

- <div class$="icon-container [[getSubmitClass_(value)]]">

- <iron-icon class="icon" icon="icons:forward"><iron-icon>

- </div>

- </paper-button>

+ <paper-icon-button icon="icons:arrow-forward"

+ class$="digit-button submit-button

+ [[getSubmitClass_(enableSubmitButton)]]"

+ disabled="[[getButtonsDisabled_(value)]]"

+ on-tap="firePinSubmitEvent_">

+ </paper-icon-button>

</div>

+ <hr id="pin-seperator">

jdufault 2016/08/05 21:58:51 Apply the css to all <hr> elements instead of maki

sammiequon 2016/08/05 23:34:50 Done.

jdufault 2016/08/05 21:58:51 Can the top-row-button css class be applied to thi

sammiequon 2016/08/05 23:34:50 Done.

- <paper-button class="digit-button" on-tap="onNumberTap_" value="1">

+ <paper-button class="digit-button top-row-button"

+ on-tap="onNumberTap_" value="1">

<inner-text>$i18n{pinKeyboard1}</inner-text>

</paper-button>

- <paper-button class="digit-button center-button" on-tap="onNumberTap_"

- value="2">

+ <paper-button class="digit-button top-row-button"

+ on-tap="onNumberTap_" value="2">

<inner-text>$i18n{pinKeyboard2}</inner-text>

</paper-button>

- <paper-button class="digit-button" on-tap="onNumberTap_" value="3">

+ <paper-button class="digit-button top-row-button"

+ on-tap="onNumberTap_" value="3">

<inner-text>$i18n{pinKeyboard3}</inner-text>

</paper-button>

</div>

@@ -172,8 +175,7 @@

<paper-button class="digit-button" on-tap="onNumberTap_" value="4">

<inner-text>$i18n{pinKeyboard4}</inner-text>

</paper-button>

- <paper-button class="digit-button center-button" on-tap="onNumberTap_"

- value="5">

+ <paper-button class="digit-button" on-tap="onNumberTap_" value="5">

<inner-text>$i18n{pinKeyboard5}</inner-text>

</paper-button>

<paper-button class="digit-button" on-tap="onNumberTap_" value="6">

@@ -184,8 +186,7 @@

<paper-button class="digit-button" on-tap="onNumberTap_" value="7">

<inner-text>$i18n{pinKeyboard7}</inner-text>

</paper-button>

- <paper-button class="digit-button center-button" on-tap="onNumberTap_"

- value="8">

+ <paper-button class="digit-button" on-tap="onNumberTap_" value="8">

<inner-text>$i18n{pinKeyboard8}</inner-text>

</paper-button>

<paper-button class="digit-button" on-tap="onNumberTap_" value="9">

@@ -193,18 +194,15 @@

</paper-button>

</div>

- <div class="digit-button"></div>

- <paper-button class="digit-button center-button" on-tap="onNumberTap_"

- value="0">

+ <div class="digit-button bottom-row-button"></div>

+ <paper-button class="digit-button bottom-row-button"

+ on-tap="onNumberTap_" value="0">

<inner-text>$i18n{pinKeyboard0}</inner-text>

</paper-button>

- <paper-button class="digit-button clear-button" on-tap="onPinClear_">

- <div class="icon-container">

- <iron-icon class="icon" icon="icons:clear"><iron-icon>

- </div>

- <inner-text class="icon-subheading">

- $i18n{pinKeyboardClear}

- </inner-text>

+ <paper-icon-button class="digit-button backspace-button

+ bottom-row-button"

+ disabled="[[getButtonsDisabled_(value)]]"

+ on-tap="onPinClear_" icon="icons:backspace">

jdufault 2016/08/05 21:58:51 Align with the line above

sammiequon 2016/08/05 23:34:50 Done.

</paper-button>

</div>