Commit 5dab43b7 authored by Esmael El-Moslimany's avatar Esmael El-Moslimany Committed by Commit Bot

Bookmarks: set fixed height of dialog depending on number of inputs

Bug: 846291
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I16f71655271eab77480c3744250618a2077481fe
Reviewed-on: https://chromium-review.googlesource.com/1072041Reviewed-by: default avatarScott Chen <scottchen@chromium.org>
Reviewed-by: default avatarDemetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Esmael El-Moslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#561652}
parent 8eab247b
...@@ -15,7 +15,7 @@ ...@@ -15,7 +15,7 @@
<div slot="title"> <div slot="title">
[[getDialogTitle_(isFolder_, isEdit_)]] [[getDialogTitle_(isFolder_, isEdit_)]]
</div> </div>
<div slot="body"> <div slot="body" style="height: [[getHeightOfBody_(isFolder_)]]px">
<paper-input always-float-label id="name" <paper-input always-float-label id="name"
label="$i18n{editDialogNameInput}" label="$i18n{editDialogNameInput}"
value="{{titleValue_}}" value="{{titleValue_}}"
......
...@@ -138,4 +138,11 @@ Polymer({ ...@@ -138,4 +138,11 @@ Polymer({
onCancelButtonTap_: function() { onCancelButtonTap_: function() {
this.$.dialog.cancel(); this.$.dialog.cancel();
}, },
/** @private */
getHeightOfBody_: function() {
const heightTwoInputs = 136;
const heightOneInput = 62;
return this.isFolder_ ? heightOneInput : heightTwoInputs;
},
}); });
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