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

Issue 689958 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression:Connected wifi name is seen chopped in WiFi section of chrome://md-settings when font size is Very large.

Project Member Reported by jbanavatu@chromium.org, Feb 8 2017

Issue description

Chrome Version: 58.0.3001.0/9261.0.0 dev-channel Glimmer, Zako, Panther, Peppy
OS: Chrome.

What steps will reproduce the problem?
(1)Navigate to chrome://md-settings/appearance >> Select font size as Very large
(2)Go to Internet connection section >> Click on WiFi for list to expand and Observe.

Expected: Connected WiFi name should be seen clearly.
Actual:Instead, connected WiFi name is seen chopped.

This is regression issue as it is working fine in 57.0.2987.30/9202.17.0 dev channel candy.

Attaching screenshots for reference.
 
Expected.png
30.6 KB View Download
Actual.png
27.0 KB View Download
Owner: steve...@chromium.org
Status: Assigned (was: Untriaged)
Labels: -Pri-1 Pri-2
P2, but will plan to address this for 58.

Status: Started (was: Assigned)
Fix: https://codereview.chromium.org/2684243003/

Note: There is an edge case where changing the font size while the list is open overlaps with the next entry until the list is closed/opened or the page is refreshed. This will not be a problem once the network list is un-inlined ( issue 673944 ) and does not seem worth fixing in the meanwhile.

Project Member

Comment 4 by bugdroid1@chromium.org, Feb 13 2017

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

commit a2665e010b198f117ab5bc9fcf60f7aa02b92f67
Author: stevenjb <stevenjb@chromium.org>
Date: Mon Feb 13 21:28:23 2017

Settings: Change the cr-network-list-item height to min-height

This fixes a lyaout bug for very large fonts

BUG= 689958 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/a2665e010b198f117ab5bc9fcf60f7aa02b92f67/ui/webui/resources/cr_elements/network/cr_network_list_item.html

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
58.0.3015.0/9304.0.0 

Sign in to add a comment