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

Issue 629421 link

Starred by 3 users

Issue metadata

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



Sign in to add a comment

Regression: Learn more link in Smart lock section of md-settings is not active in CrOS

Project Member Reported by sc00335...@techmahindra.com, Jul 19 2016

Issue description

Version: 54.0.2798.0/8605.0.0 (Official Build) dev-channel daisy,peppy
OS: Chrome os

What steps will reproduce the problem?
(1) Sign in to user and go to chrome://md-settings >> Observe Learn more link in Smart lock(beta) section

Expected: Learn more link should be in enabled state and should be clickable.
Actual: Instead link is not clickable.

This is a regression issue as it is working fine in 51.0.2704.106/8172.62.0 (Official Build) stable-channel daisy
 
Expected_link.png
90.4 KB View Download
Actual_link.png
88.2 KB View Download
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on chrome(Daisy) Version: 54.0.2798.0/8605.0.0 dev channel. Observed Learn more link is not active under chrome://md-settings.
Blocking: 614589
Labels: -Pri-1 -M-54 Hotlist-MD-Settings-PeopleCrOS Pri-2
Owner: tommycli@chromium.org
Status: Assigned (was: Untriaged)
Blocking: -614589
Removing as dev-blocker since this does not prevent actual configuration of Settings.
Cc: tommycli@chromium.org
Owner: mahmadi@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 9 2016

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

commit 89e14cb01bb848e376c6ffe6178b16d3d653f514
Author: mahmadi <mahmadi@chromium.org>
Date: Wed Nov 09 19:27:17 2016

[MD Settings][People] UI polish for the easy unlock row

BUG= 608945 ,  629421 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/89e14cb01bb848e376c6ffe6178b16d3d653f514/chrome/app/settings_strings.grdp
[modify] https://crrev.com/89e14cb01bb848e376c6ffe6178b16d3d653f514/chrome/browser/resources/settings/people_page/people_page.html

Project Member

Comment 8 by bugdroid1@chromium.org, Nov 9 2016

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

commit 89e14cb01bb848e376c6ffe6178b16d3d653f514
Author: mahmadi <mahmadi@chromium.org>
Date: Wed Nov 09 19:27:17 2016

[MD Settings][People] UI polish for the easy unlock row

BUG= 608945 ,  629421 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/89e14cb01bb848e376c6ffe6178b16d3d653f514/chrome/app/settings_strings.grdp
[modify] https://crrev.com/89e14cb01bb848e376c6ffe6178b16d3d653f514/chrome/browser/resources/settings/people_page/people_page.html

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified on ChromeOS  8980.0.0, 56.0.2915.0

Sign in to add a comment