New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 696913 link

Starred by 1 user
Status: Fixed
Owner:
sammiequon@chromium.org
Closed: Mar 2017
Cc:
nyerramilli@chromium.org
kkaluri@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression:After hovering mouse over screenshot misalignment of text is observed in report an issue overlay.

Reported by vku...@etouch.net, Feb 28 2017 
Chrome Version:58.0.3026.0 (Official Build) fe586ab75aca1b8ab839db23bceac5f621389fed-refs/heads/master@{#453454} (32/64 Bit) 
OS:Windows(7,8,8.1,10)

What steps will reproduce the problem?
(1)Launch chrome, navigate to chrome://md-settings/manageLanguages page and select 'French' from 'All languages' list.
(2)Click on iron icon of french and select 'disply google chrome..' option, relauch the browser.
(3)Click on Menu button > About chrome > Report an issue,hover mouse on screenshot and observe the "include this screenshot" text.

Actual: After hovering mouse over screenshot misalignment of text is observed in report an issue overlay.

Expected: After hovering mouse over screenshot text should be properly displayed.

This is a regression issue broken in 'M57' and below is the manual regression range:
Good Build: 57.0.2970.0 
Bad Build: 57.0.2971.0 

Note: Issue not seen on Mac & Linux OS.
Actual_Report.mp4
228 KB View Download
Expected_Report.mp4
492 KB View Download

Comment 1 by vku...@etouch.net, Feb 28 2017

Labels: hasbisect
Owner: sammiequon@chromium.org
Status: Assigned (was: Unconfirmed)
CL:
https://chromium.googlesource.com/chromium/src/+log/57.0.2970.0..57.0.2971.0?pretty=fuller&n=10000
(Uanble to narrow down the range since 'report an issue' option doesn't appear in chromium builds)

Suspecting: 441263 ?
Kindly help to reassign if this is not related your change.

Comment 2 by msrchandra@chromium.org, Feb 28 2017 

Provided manual bisect as per-revision bisect script was providing all good builds.
Thank You.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 10 2017 

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

commit 67435cd8a4ae34c8005f848cb0a2e6b1ebabbafd
Author: sammiequon <sammiequon@chromium.org>
Date: Fri Mar 10 20:19:01 2017

Feedback Dialog: Fixes text wrapping issue for French.

It is said to be a M57 regression, but no one has touchedthe feedback dialog for a while. Regardless, this CL makes the dialog look like the expected_report.mp4 in the bug. Tested for RTL as well.

TEST=manual
BUG= 696913 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2723613005
Cr-Commit-Position: refs/heads/master@{#456147}

[modify] https://crrev.com/67435cd8a4ae34c8005f848cb0a2e6b1ebabbafd/chrome/browser/resources/feedback/css/feedback.css

Comment 4 by sammiequon@chromium.org, Mar 10 2017

Status: Fixed (was: Assigned)

Comment 5 by kkaluri@chromium.org, Mar 17 2017

Cc: kkaluri@chromium.org
Labels: TE-Verified-59.0.3043.0 TE-Verified-M59
Verified this issue on Windows 10 with chrome #59.0.3043.0
Followed the steps as mentioned in comment #0 and observed after hovering mouse over screenshot, the text is displayed properly without any misalignment.

Attaching the screen-cast for reference.

Hence adding TE-Verified labels.
Issue 696913.mp4
2.0 MB View Download
►

Sign in to add a comment