Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4191)

Unified Diff: chrome/browser/resources/md_history/shared_style.html

Issue 2583353003: MD History: Replace last usage of cr-shared-menu with cr-action-menu. (Closed)
Patch Set: Fix more Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/md_history/shared_style.html
diff --git a/chrome/browser/resources/md_history/shared_style.html b/chrome/browser/resources/md_history/shared_style.html
index 5d58cc00b063f8f6539adce6469dd6974ebc9fd9..4cca685129b280be9d0e454931ba9d50fdfcf2de 100644
--- a/chrome/browser/resources/md_history/shared_style.html
+++ b/chrome/browser/resources/md_history/shared_style.html
@@ -37,11 +37,20 @@
justify-content: center;
}
- .menu-item {
- -webkit-user-select: none;
+ /* TODO(dpapad): Copied from MD Settings, find a way to re-use style. */
tsergeant 2016/12/21 00:39:36 Can it be moved into cr_elements/shared_style_css.
dpapad 2016/12/21 01:04:48 cr_elements/shared_style_css.html sounds better th
tsergeant 2016/12/21 02:37:34 Hmm, yeah, if there's nothing in settings that use
Dan Beam 2016/12/21 03:11:19 yes, i'd move the default dropdown-item (and hr) s
dpapad 2016/12/21 03:50:18 Done for dropdown-item. Did not move hr styling be
+ .dropdown-item {
+ align-items: center;
+ background: none;
+ border: none;
+ box-sizing: border-box;
+ color: var(--paper-grey-800);
cursor: pointer;
+ display: flex;
font: inherit;
- white-space: nowrap;
+ min-height: 32px;
+ padding: 0 24px;
+ text-align: start;
+ width: 100%;
}
.website-icon {

Powered by Google App Engine
This is Rietveld 408576698