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

Issue 732761 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 3
Type: Bug-Regression

Blocked on:
issue 725401

Blocking:
issue 741083



Sign in to add a comment

Regression: Cancel Button appears chopped on changing browser language to French on chrome://settings/clearBrowserData

Reported by aiman.an...@etouch.net, Jun 13 2017

Issue description

Chrome Version: 61.0.3128.0 (Official Build) 4138c6d4e9f036df01c2a6092a242a3aa39b6afc-refs/heads/master@{#478527}(32/64 Bit).

OS:  Windows(7,8,8.1,10).

Steps to reproduce:
1.Launch Chrome, go to chrome://settings, change browser language to 'French' from chrome://settings/languages.
2.Relaunch chrome, go to chrome://settings/appearacne and change Font Size to 'Very Large' .
3.Go to chrome://settings/clearBrowserData, observe Cancel Button.

Actual Result: Cancel Button is aligned to left and appears chopped.
Expected Result: Cancel Button should not appear chopped.

This is regression issue broken in 'M 60' and will soon update the bisect info:

Manual Bisect Info: 
Good Build 60.0.3089.0
Bad Build 60.0.3090.0

Note: Issue is not reproducible on MAC and LINUX OS.
 
Actual Result.mp4
1.6 MB View Download
Expected Result.mp4
1.1 MB View Download
Labels: hasbisect
Owner: dbeam@chromium.org
Status: Assigned (was: Unconfirmed)
Narrow Bisect:
https://chromium.googlesource.com/chromium/src/+log/6abba8e52ba13f660d2350b08f3d1829ce282edd..58d8871f8d95a4743ee02f752922855e48dad4ea?pretty=fuller&n=100

Suspecting: r469494?

@dbeam: Kindly help to re-assign if your change is not the cause for this issue.

Comment 2 by dbeam@chromium.org, Jun 13 2017

Labels: -Pri-2 Pri-3

Comment 3 by dbeam@chromium.org, Jun 13 2017

Owner: ----
Status: Available (was: Assigned)

Comment 4 by dpa...@chromium.org, Jun 13 2017

Labels: Proj-MaterialDesign-WebUI Hotlist-MD-Settings-Privacy-CBD

Comment 5 by kolos@chromium.org, Jun 19 2017

Components: Privacy

Comment 6 by battre@chromium.org, Jun 19 2017

Cc: dullweber@chromium.org msramek@chromium.org

Comment 7 by kmaoy@google.com, Jul 3 2017

Owner: kmaoy@google.com
Status: Assigned (was: Available)
Thanks for your feedback. We're internally triaging the bug and it's currently in progress. We'll update this thread once it's fixed.

Comment 8 by dpa...@chromium.org, Jul 11 2017

Blocking: 741083
Labels: -Proj-MaterialDesign-WebUI -hasbisect -Hotlist-MD-Settings-Privacy-CBD -M-61 OS-iOS
Owner: ----
Status: WontFix (was: Assigned)
Hi All,

Please see the below response of the language POC:

¨Thank you for your feedback.
I am afraid there is nothing we can do on the linguists side since we cannot make the button shorter.¨

Regards!
Labels: Proj-MaterialDesign-WebUI Hotlist-MD-Settings-Privacy-CBD
Status: Available (was: WontFix)
The buttons should be wrapping to a new line, show that they are both fully visible, see screenshot.
wrap_around.png
122 KB View Download
Labels: -OS-iOS OS-Linux
Owner: mencinares@google.com
Thanks! I´ll report it back to the language POC.

Regards!
Blockedon: 725401
Cc: mencinares@google.com
Owner: dullweber@chromium.org
Status: Started (was: Available)
The label of this button is going to change to "Clear data" with the new CBD ui. That should solve this issue.
Project Member

Comment 14 by bugdroid1@chromium.org, Sep 25 2017

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

commit db054f2e008d4bbb983d5f13ad983acba43df555
Author: Christian Dullweber <dullweber@chromium.org>
Date: Mon Sep 25 09:56:09 2017

Margin, color and text tweaks for CBD

This CL changes the button label from "Clear browsing data" to
"Clear data", the tab text color to blue and a different grey and
makes some improvements to the margins around "Time range".

Before:
https://screenshot.googleplex.com/HOySYiU6gtx.png
After:
https://screenshot.googleplex.com/6Jk2Y1jok5C.png

Bug:  725401 ,  732761 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I6f52f248513d3cfaeecc807d1d6a17b7863cb4bc
Reviewed-on: https://chromium-review.googlesource.com/678508
Commit-Queue: Christian Dullweber <dullweber@chromium.org>
Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#504015}
[modify] https://crrev.com/db054f2e008d4bbb983d5f13ad983acba43df555/chrome/app/settings_strings.grdp
[modify] https://crrev.com/db054f2e008d4bbb983d5f13ad983acba43df555/chrome/browser/resources/settings/clear_browsing_data_dialog/clear_browsing_data_dialog_tabs.html
[modify] https://crrev.com/db054f2e008d4bbb983d5f13ad983acba43df555/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc

Note:
Retested the above issue on Windows(7,8,10) on latest Chrome #63.0.3223.8 and issue is still reproducible.

Current_Result.png
100 KB View Download
The new dialog is still behind the "tabs-in-cbd" flag but I'm about to enable it by default and launch the dialog in M63.

Owner: mencinares@google.com
Status: Fixed (was: Started)
Hi All,

This has now been fixed by the linguists and it is now on the live UI. Please see the attached screenshot.

Regards!
wrap.jpg
128 KB View Download
Labels: TE-Verified-63.0.3225.0 TE-Verified
Note:
Retested the above issue on Win(7,8,10) for latest Canary #63.0.3225.0 and fix is working as intended.

Current_Result.png
105 KB View Download
Labels: -TE-Verified TE-Verified-M63
Status: Verified (was: Fixed)
Thanks for the verification.

Regards!

Sign in to add a comment