Commit 58f7b2b2 authored by Christos Froussios's avatar Christos Froussios Committed by Commit Bot

[Passwords] Focus on cancel button when edit dialog opens

Also some minor visual changes

Bug: 1066785
Change-Id: Ia6c58c2f3cc5b78fce2ab819f8d4fe1d45782e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128491Reviewed-by: default avatarJan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Christos Froussios <cfroussios@chromium.org>
Cr-Commit-Position: refs/heads/master@{#755362}
parent bd2c9e88
...@@ -7,6 +7,7 @@ ...@@ -7,6 +7,7 @@
<link rel="import" href="chrome://resources/cr_elements/cr_icons_css.html"> <link rel="import" href="chrome://resources/cr_elements/cr_icons_css.html">
<link rel="import" href="chrome://resources/cr_elements/shared_vars_css.html"> <link rel="import" href="chrome://resources/cr_elements/shared_vars_css.html">
<link rel="import" href="chrome://resources/html/assert.html"> <link rel="import" href="chrome://resources/html/assert.html">
<link rel="import" href="chrome://resources/html/cr/ui/focus_without_ink.html">
<link rel="import" href="chrome://resources/html/i18n_behavior.html"> <link rel="import" href="chrome://resources/html/i18n_behavior.html">
<link rel="import" href="../icons.html"> <link rel="import" href="../icons.html">
<link rel="import" href="../settings_shared_css.html"> <link rel="import" href="../settings_shared_css.html">
...@@ -26,9 +27,15 @@ ...@@ -26,9 +27,15 @@
} }
cr-icon-button { cr-icon-button {
--cr-icon-button-icon-size: 24px; --cr-icon-button-icon-size: 16px;
margin-inline-start: 2px; margin-inline-start: 2px;
} }
#footnote {
margin-inline-start: 2px;
margin-top: 16px;
}
</style> </style>
<cr-dialog id="dialog" close-text="$i18n{close}"> <cr-dialog id="dialog" close-text="$i18n{close}">
<div slot="title">$i18n{editCompromisedPasswordTitle}</div> <div slot="title">$i18n{editCompromisedPasswordTitle}</div>
...@@ -48,7 +55,7 @@ ...@@ -48,7 +55,7 @@
title="[[showPasswordTitle_(visible)]]"> title="[[showPasswordTitle_(visible)]]">
</cr-icon-button> </cr-icon-button>
</cr-input> </cr-input>
<div>[[getFootnote_(item.formattedOrigin)]]</div> <div id="footnote">[[getFootnote_(item.formattedOrigin)]]</div>
</div> </div>
<div slot="button-container"> <div slot="button-container">
<cr-button id="cancel" class="cancel-button" on-click="onCancel_"> <cr-button id="cancel" class="cancel-button" on-click="onCancel_">
......
...@@ -39,6 +39,7 @@ Polymer({ ...@@ -39,6 +39,7 @@ Polymer({
// Set the manager. These can be overridden by tests. // Set the manager. These can be overridden by tests.
this.passwordManager_ = PasswordManagerImpl.getInstance(); this.passwordManager_ = PasswordManagerImpl.getInstance();
this.$.dialog.showModal(); this.$.dialog.showModal();
cr.ui.focusWithoutInk(this.$.cancel);
}, },
/** Closes the dialog. */ /** Closes the dialog. */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment