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

Issue 746923 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Unwanted horizontal scroll bar is seen for 'Destination' dialogue box of print preview in RTL

Reported by sans...@etouch.net, Jul 20 2017

Issue description

Chrome Version:61.0.3162.0(Official Build) 46d636d25e054da0b0d8270d525c84f39146ce30-refs/heads/master@{#488073} 32/64 bit
OS: Windows (7,8,10), Mac(10.11.6, 10.12.3), Linux(14.04 LTS)

Pre-condition: Enable 'Force UI direction' from chrome://flags

Steps:
1. Launch Chrome and give print command on NTP
2. Click on 'Change' button under Destination and observe

Actual: Unwanted scroll bar is seen for 'Destination' dialogue box which shows spinner after scrolling horizontally towards L.H.S

Expected: No such scroll bar is seen for 'Destination' dialogue box 

This is non-regression issue seen from M-55, since this is new feature introduced in 55.0.2868.0
 
Actual_video.mp4
505 KB View Download
Status: Untriaged (was: Unconfirmed)
As this being a Non-Regression issue, changing the status to Untriaged so that the issue would get addressed.

Thank You!

Comment 2 by dpa...@chromium.org, Jul 26 2017

Owner: rbpotter@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 27 2017

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

commit 3d8fde94e7c0ec8ed6cf67db52521ef35e2232dc
Author: rbpotter <rbpotter@chromium.org>
Date: Thu Jul 27 23:46:42 2017

Print Preview: Fix RTL spinner bug

Spinners for destination search overlay were always displayed on the
far left, leading to a horizontal scrollbar in RTL. Fix spinners to
display correctly for RTL.

Bug:  746923 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I02ab43e81a2fc422737f83d865111d9bc214a763
Reviewed-on: https://chromium-review.googlesource.com/588108
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#490128}
[modify] https://crrev.com/3d8fde94e7c0ec8ed6cf67db52521ef35e2232dc/chrome/browser/resources/print_preview/search/destination_list.css

Comment 4 by sans...@etouch.net, Jul 28 2017

Labels: TE-Verified-62.0.3169.0 TE-Verified-M62
Note: Above issue is fixed on latest Canary version: 62.0.3169.0 on Win,Mac & Linux OS
Please refer the attached video.

Thank you.
Fixed_video.mp4
601 KB View Download
Status: Fixed (was: Started)
Thanks for verifying.

Sign in to add a comment