MD Settings: settings-subpage-search, add text ellipsis for placeholder text.

MD Settings: settings-subpage-search, add text ellipsis for placeholder text.

BUG=733095
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2940883005
Cr-Commit-Position: refs/heads/master@{#479456}
Committed: https://chromium.googlesource.com/chromium/src/+/0a9fd735b58bf6fe1a7eb2b9d743dc65650afc02

[dpapad, 2017-06-14 17:06:47]

Description was changed from

==========
MD Settings: settings-subpage-search, add text ellipsis for placeholder text.

BUG=733095
==========

to

==========
MD Settings: settings-subpage-search, add text ellipsis for placeholder text.

BUG=733095
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2017-06-14 17:07:08]

The CQ bit was checked by dpapad@chromium.org to run a CQ dry run

[dpapad, 2017-06-14 17:07:28]

dpapad@chromium.org changed reviewers:
+ scottchen@chromium.org

[dpapad, 2017-06-14 17:07:29]

Before/after screenshots at http://imgur.com/a/uRWZK.

[commit-bot: I haz the power, 2017-06-14 17:07:35]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[scottchen, 2017-06-14 17:50:14]

On 2017/06/14 17:07:29, dpapad wrote:
> Before/after screenshots at http://imgur.com/a/uRWZK.

LGTM, but I did notice something interesting - when you click on the input, it
no longer ellipsis. 
Personally not too concerned, just wanted to share my finding.
screenshots: http://imgur.com/a/nmYVU

[dpapad, 2017-06-14 17:52:38]

On 2017/06/14 at 17:50:14, scottchen wrote:
> On 2017/06/14 17:07:29, dpapad wrote:
> > Before/after screenshots at http://imgur.com/a/uRWZK.
> 
> LGTM, but I did notice something interesting - when you click on the input, it
no longer ellipsis. 
> Personally not too concerned, just wanted to share my finding.
> screenshots: http://imgur.com/a/nmYVU

Yes, I noticed that too. That is the native behavior on Chrome (did not try
other browsers). I suppose this is an effort to show as much text as possible
when the input is focused.

[dpapad, 2017-06-14 18:05:11]

The CQ bit was unchecked by dpapad@chromium.org

[dpapad, 2017-06-14 18:05:14]

The CQ bit was checked by dpapad@chromium.org

[commit-bot: I haz the power, 2017-06-14 18:05:21]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-14 18:51:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497463514777830, "parent_rev":
"ae7702ad350761014f8daf0df3deea65f61ef54b", "commit_rev":
"0a9fd735b58bf6fe1a7eb2b9d743dc65650afc02"}

[commit-bot: I haz the power, 2017-06-14 18:51:19]

Description was changed from

==========
MD Settings: settings-subpage-search, add text ellipsis for placeholder text.

BUG=733095
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: settings-subpage-search, add text ellipsis for placeholder text.

BUG=733095
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2940883005
Cr-Commit-Position: refs/heads/master@{#479456}
Committed:
https://chromium.googlesource.com/chromium/src/+/0a9fd735b58bf6fe1a7eb2b9d743...
==========

[commit-bot: I haz the power, 2017-06-14 18:51:20]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/0a9fd735b58bf6fe1a7eb2b9d743...