Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5024)

Unified Diff: chrome/browser/resources/settings/languages_page/add_languages_dialog.html

Issue 2519853005: MD Settings: Enable to search "Add languages" list. (Closed)
Patch Set: Resolve minor conflicts Created 3 years, 9 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 | « AUTHORS ('k') | chrome/browser/resources/settings/languages_page/add_languages_dialog.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/languages_page/add_languages_dialog.html
diff --git a/chrome/browser/resources/settings/languages_page/add_languages_dialog.html b/chrome/browser/resources/settings/languages_page/add_languages_dialog.html
index 332323927da2e95e116c10c370082e63a1459759..76c2a27773ce20449179d6c04d8bfda0154ef278 100644
--- a/chrome/browser/resources/settings/languages_page/add_languages_dialog.html
+++ b/chrome/browser/resources/settings/languages_page/add_languages_dialog.html
@@ -1,10 +1,14 @@
<link rel="import" href="chrome://resources/cr_elements/cr_dialog/cr_dialog.html">
+<link rel="import" href="chrome://resources/cr_elements/icons.html">
<link rel="import" href="chrome://resources/html/polymer.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-input/iron-input.html">
<link rel="import" href="chrome://resources/polymer/v1_0/iron-list/iron-list.html">
-<link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.html">
<link rel="import" href="chrome://resources/polymer/v1_0/paper-checkbox/paper-checkbox.html">
<link rel="import" href="languages.html">
<link rel="import" href="../settings_shared_css.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-input/paper-input-container.html">
<dom-module id="settings-add-languages-dialog">
<template>
@@ -15,12 +19,18 @@
}
.body {
+ -webkit-padding-end: 0 !important;
display: flex;
flex-direction: column;
max-height: 250px;
overflow: auto;
}
+ paper-input-container {
+ -webkit-padding-end: 24px;
+ padding-top: 4px;
+ }
+
.ripple-padding {
/* Create a little extra space for checkbox ink ripple to flow into. */
-webkit-padding-start: 20px;
@@ -29,10 +39,14 @@
<dialog is="cr-dialog" id="dialog" close-text="$i18n{close}">
<div class="title">$i18n{addLanguagesDialogTitle}</div>
<div class="body">
- <!-- TODO(michaelpg): Dynamic language search/filtering. -->
- <iron-list class="ripple-padding" scroll-target="[[$$('.body')]]"
+ <paper-input-container no-label-float>
+ <iron-icon id="searchIcon" icon="cr:search" prefix></iron-icon>
+ <input is="iron-input" bind-value="{{filterValue_}}"></input>
+ </paper-input-container>
+ <iron-list class="ripple-padding"
+ scroll-target="[[$$('.ripple-padding')]]"
items="[[getAvailableLanguages_(
- languages.supported, languages.enabled.*)]]">
+ languages.supported, languages.enabled.*, filterValue_)]]">
<template>
<paper-checkbox class="list-item" checked="[[willAdd_(item.code)]]"
title$="[[item.nativeDisplayName]]"
« no previous file with comments | « AUTHORS ('k') | chrome/browser/resources/settings/languages_page/add_languages_dialog.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698