[Autofill] Text wrapping removes ability to remove addresses |
||||
Issue descriptionVersion 57.0.2935.0 canary (64-bit) What steps will reproduce the problem? (1) Add autofill address (2) Add a bunch of dummy text in the street address field. Click Save (3) The overflow menu for the corresponding address missing What is the expected result? See screenshot. Add an "..." to indicate overflow text and ensure menu is accessible. Marking P1 because UI is not usable.
,
Dec 6 2016
,
Dec 9 2016
,
Dec 13 2016
Screenshot for fix, CL at https://codereview.chromium.org/2569313002
,
Dec 13 2016
lgtm
,
Dec 20 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3283e4b6ebd5009d8a6d6efffdc11489f8a0fa46 commit 3283e4b6ebd5009d8a6d6efffdc11489f8a0fa46 Author: hcarmona <hcarmona@chromium.org> Date: Tue Dec 20 23:51:03 2016 [MD-Settings] Add Ellipsis for the Address List Pieces. BUG= 669117 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2569313002 Cr-Commit-Position: refs/heads/master@{#439934} [modify] https://crrev.com/3283e4b6ebd5009d8a6d6efffdc11489f8a0fa46/chrome/browser/resources/settings/passwords_and_forms_page/autofill_section.html [modify] https://crrev.com/3283e4b6ebd5009d8a6d6efffdc11489f8a0fa46/chrome/browser/resources/settings/passwords_and_forms_page/autofill_section.js [modify] https://crrev.com/3283e4b6ebd5009d8a6d6efffdc11489f8a0fa46/chrome/test/data/webui/settings/passwords_and_autofill_fake_data.js [modify] https://crrev.com/3283e4b6ebd5009d8a6d6efffdc11489f8a0fa46/chrome/test/data/webui/settings/settings_autofill_section_browsertest.js
,
Jan 6 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by bettes@chromium.org
, Nov 28 2016