New issue
Advanced search Search tips

Issue 735103 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Multi-line street address should be displayed on multiple lines in address editor

Project Member Reported by rouslan@chromium.org, Jun 20 2017

Issue description

Chrome Version       : 61.0.3136.0
OS Version: OS X 10.12.5
URLs (if applicable) : https://rsolomakhin.github.io/pr/single/

What steps will reproduce the problem?
(1) Add an address with multiple street address lines in chrome://settings/autofill
(2) Open https://rsolomakhin.github.io/pr/single/
(3) Click "Buy" button on the page.
(4) Edit one the address from step (1).

What is the expected result?
The street address should contain multiple lines.

What happens instead?
The street address is in a single line with "NL" character in the middle.

For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

 
Screen Shot 2017-06-16 at 2.03.49 PM.png
74.3 KB View Download
Components: -UI>Browser>Autofill>Payments UI>Browser>Payments
Labels: -M-61 M-62
Status: Started (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 3 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/af38db52e9cedeca6a7ad920339bf09b36c65255

commit af38db52e9cedeca6a7ad920339bf09b36c65255
Author: Anthony Vallee-Dubois <anthonyvd@chromium.org>
Date: Thu Aug 03 15:24:41 2017

[Web Payments] Display multiline addresses correctly in editor.

Bug:  735103 
Change-Id: I54ac93cf34a37144dbf257d5f3768af64e319151
Reviewed-on: https://chromium-review.googlesource.com/595027
Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Commit-Queue: Anthony Vallee-Dubois <anthonyvd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491744}
[modify] https://crrev.com/af38db52e9cedeca6a7ad920339bf09b36c65255/chrome/browser/ui/views/payments/shipping_address_editor_view_controller.cc

Status: Fixed (was: Started)
This is fixed now, although we should find a way to make the field accept multiline input.
Labels: TE-Verified-62.0.3178.0 TE-Verified-M62
Tested the issue on Latest Chrome Dev# 62.0.3178.0 (Windows, Mac & Linux) and Chrome Dev# 62.0.3176.0 (ChromeOS) and could not reproduce the issue.
No unwanted characters were seen in the text spaces. Adding screenshot for reference. Hence adding appropriate TE-Verified labels.

Note: Checked with 5 different addresses.
735103.png
48.9 KB View Download

Sign in to add a comment