Regression:Vertical separator line is not seen between print options and print preview page.
Reported by
shruti.j...@etouch.net,
Sep 18
|
|||
Issue descriptionChrome Version : 71.0.3554.0 (Official Build) 50b24262ba0c4e14c723112f9aad3686dc6462c1-refs/branch-heads/3554@{#1} 32/64 bit OS :Mac(10.12.6, 10.13.1, 10.14, 10.13.6) Windows (7, 8, 8.1 ,10) and Linux 14.04 LTS Steps to reproduce: 1.Launch chrome and give print command. 2.Click on '+' (Zoom in) and scroll page horizontally to RHS. 3.Observe vertical separator line between print options and print preview page. Actual Result:Vertical separator line is not seen between print options and print preview page. Expected Result: Vertical separator line should be seen between print options and print preview page This is a regression issue broken in ‘M-71’ and below is the 'per-revision' bisect information: Good Build : 71.0.3551.0 (Revision : 590851) Bad Build : 71.0.3552.2 (Revision : 591210) Change Log URL : https://chromium.googlesource.com/chromium/src/+log/974aa7b559b4361d7410479796becb0b933dc6a4..e73e92b7fbf4206ffdb20282ac73bcbed7cf910e Suspecting : https://chromium.googlesource.com/chromium/src/+/e73e92b7fbf4206ffdb20282ac73bcbed7cf910e rbpotter@ : Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Kindly refer the attached screen-cast. Thank you..!
,
Sep 20
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cabf4f1a31f2b03b308993e4d73e2d5d3f5c0abc commit cabf4f1a31f2b03b308993e4d73e2d5d3f5c0abc Author: rbpotter <rbpotter@chromium.org> Date: Thu Sep 20 00:05:19 2018 Print Preview refresh: Adjust font weight and add border Note that title is to be changed to weight 400 rather than 500 per offline discussion (it was previously 500, not 700 as stated on the bug). Bug: 884896 , 885109 Change-Id: I09aad405eaa07bcb7031b71e6bc91fb0c286b3e2 Reviewed-on: https://chromium-review.googlesource.com/1234995 Reviewed-by: Hector Carmona <hcarmona@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#592616} [modify] https://crrev.com/cabf4f1a31f2b03b308993e4d73e2d5d3f5c0abc/chrome/browser/resources/print_preview/new/app.html [modify] https://crrev.com/cabf4f1a31f2b03b308993e4d73e2d5d3f5c0abc/chrome/browser/resources/print_preview/new/header.html
,
Sep 20
Update: Rechecked the above issue using latest canary build#71.0.3557.0 on Mac(10.12.6, 10.13.1, 10.14, 10.13.6) Windows (7, 8, 8.1 ,10) and Linux 14.04 LTS and issue is fixed. Please find below attached screen cast Thank You...
,
Sep 20
|
|||
►
Sign in to add a comment |
|||
Comment 1 by namratakannan@chromium.org
, Sep 18