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

Unified Diff: chrome/browser/resources/shared/css/menu.css

Issue 10389016: Use hidden attribute rather the 'display: none' for hiding menus. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Prevent FoUC with menus, using Evan's suggestion. Created 8 years, 7 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: chrome/browser/resources/shared/css/menu.css
diff --git a/chrome/browser/resources/shared/css/menu.css b/chrome/browser/resources/shared/css/menu.css
index 94bc6faf36d3fbff0b70643cfe81b6028ea1730c..56de9fb6f7a2650611540f23eda38e71236e51e2 100644
--- a/chrome/browser/resources/shared/css/menu.css
+++ b/chrome/browser/resources/shared/css/menu.css
@@ -8,7 +8,6 @@ menu {
background: white;
color: black;
cursor: default;
- display: none;
left: 0;
margin: 0;
outline: 1px solid rgba(0, 0, 0, 0.2);
@@ -18,6 +17,10 @@ menu {
z-index: 3;
}
+menu:not(.decorated) {
+ display: none;
+}
+
menu > * {
box-sizing: border-box;
display: block;
« no previous file with comments | « chrome/browser/resources/ntp4/other_sessions.js ('k') | chrome/browser/resources/shared/js/cr/ui/context_menu_handler.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698