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

Unified Diff: ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar.html

Issue 2280513002: MD History: promote menu button to show clear browsing data in narrow mode (Closed)
Patch Set: damn it Created 4 years, 3 months 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: ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar.html
diff --git a/ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar.html b/ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar.html
index 28200326d9545b53e8d4504805e5f8b19c79a8cb..35ac7c844821bf554896758f51cfce4e161e0aef 100644
--- a/ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar.html
+++ b/ui/webui/resources/cr_elements/cr_toolbar/cr_toolbar.html
@@ -8,9 +8,10 @@
<style>
:host {
--cr-toolbar-field-width: 580px;
+ --cr-toolbar-height: 56px;
color: #fff;
display: flex;
- height: 56px;
+ height: var(--cr-toolbar-height);
}
h1 {
@@ -71,12 +72,70 @@
:host([narrow_][showing-search_]) #leftContent {
opacity: 0;
}
+
+ #menuPromo {
+ -webkit-padding-end: 12px;
+ -webkit-padding-start: 8px;
+ align-items: center;
+ background: #616161;
+ border-radius: 2px;
+ color: white;
+ display: flex;
+ font-size: 92.3%;
+ font-weight: 500;
+ opacity: 0;
+ padding-bottom: 6px;
+ padding-top: 6px;
+ position: absolute;
+ top: var(--cr-toolbar-height);
+ white-space: nowrap;
+ z-index: 2;
+ }
+
+ #menuPromo::before {
+ background: inherit;
+ /* Up arrow. 105% in Y coordinates fixes glitch at 110/125% zoom. */
+ clip-path: polygon(0 105%, 100% 105%, 50% 0);
+ content: '';
+ display: block;
+ left: 10px;
+ height: 6px;
+ position: absolute;
+ top: -6px;
+ width: 12px;
+ }
+
+ :host-context([dir=rtl]) #menuPromo::before {
+ left: auto;
+ right: 10px;
+ }
+
+ #closePromo {
+ -webkit-appearance: none;
+ -webkit-margin-start: 12px;
+ background: none;
+ border: none;
+ color: inherit;
+ font-size: 20px; /* Shouldn't change if default font-size changes. */
+ line-height: 20px;
+ padding: 0;
+ width: 20px;
+ }
</style>
<div id="leftContent">
- <template is="dom-if" if="[[showMenu]]">
+ <!-- Note: showing #menuPromo relies on this dom-if being [restamp]. -->
+ <template is="dom-if" if="[[showMenu]]" restamp>
<paper-icon-button id="menuButton" icon="cr:menu" on-tap="onMenuTap_"
- title="[[menuLabel]]">
+ title="[[titleIfNotShowMenuPromo_(menuLabel, showMenuPromo)]]"
+ aria-label$="[[menuLabel]]">
</paper-icon-button>
+ <template is="dom-if" if="[[showMenuPromo]]">
+ <div id="menuPromo" role="tooltip">
+ [[menuPromo]]
+ <button id="closePromo" on-tap="onClosePromoTap_"
+ aria-label$="[[closeMenuPromo]]">&#x2715;</button>
+ </paper-tooltip>
+ </template>
</template>
<h1>[[pageName]]</h1>
</div>

Powered by Google App Engine
This is Rietveld 408576698