New issue
Advanced search Search tips

Issue 653689 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug

Blocked on:
issue 656807



Sign in to add a comment

MD Settings: CrOS, Migrate from paper-dropdown-menu-light to native select.

Project Member Reported by dpa...@chromium.org, Oct 6 2016

Issue description

ChromeOS UI still uses paper-dropdown-menu at 

device_page/display.html
printing_page/cups_add_printer_dialog.html
internet_page/network_nameservers.html
internet_page/network_apnlist.html
internet_page/network_proxy.html
internet_page/network_apnlist.js

Since I am focusing on release blockers for the Desktop part of Settings, I will be working on action menus next [1], so filing this separately so that we don't lose track. For example migrations see CLs attached to the original bug [2].

[1] https://bugs.chromium.org/p/chromium/issues/detail?id=639718
[2] https://bugs.chromium.org/p/chromium/issues/detail?id=651513
 
Cc: x...@chromium.org

Comment 2 by dbeam@chromium.org, Oct 12 2016

Cc: tbuck...@chromium.org
Labels: -Pri-2 M-55 Pri-1
Owner: steve...@chromium.org
Status: Assigned (was: Available)
Status: Started (was: Assigned)
@steven: I believe there are no more  paper-dropdown-menu-light instances in the MD Settings codebase. Still seeing a few, now obsolete, CSS rules at printing_page/cups_add_printer_dialog.html
settings_shared_css.html

Shall you have the honor of removing them?
I'm fine either way, but I won't be able to get to this until Wednesday at
the earliest.
Labels: -M-55 M-56
Blockedon: 656807
Labels: -M-56 M-57
Project Member

Comment 13 by bugdroid1@chromium.org, Dec 15 2016

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

commit 32ca522f776a4656163f9ff62c30c7d57d190364
Author: stevenjb <stevenjb@chromium.org>
Date: Thu Dec 15 22:16:21 2016

MD Settings: Eliminate remaining references to paper-dropdown-menu-light

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

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

[modify] https://crrev.com/32ca522f776a4656163f9ff62c30c7d57d190364/chrome/browser/resources/settings/settings_shared_css.html
[modify] https://crrev.com/32ca522f776a4656163f9ff62c30c7d57d190364/ui/webui/resources/polymer_resources.grdp

Status: Fixed (was: Started)
Status: Verified (was: Fixed)

Sign in to add a comment