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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 1932413003: [MD History] Refactor history.html into a history-app element. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address nit Created 4 years, 8 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
« no previous file with comments | « chrome/browser/resources/md_history/elements.html ('k') | chrome/browser/resources/md_history/history.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/md_history/history.html
diff --git a/chrome/browser/resources/md_history/history.html b/chrome/browser/resources/md_history/history.html
index 82ac05b0f3a62979a9b288cdf15c140bdb35fd76..009049ab7fe815776d2511e4cbf1388d352e030d 100644
--- a/chrome/browser/resources/md_history/history.html
+++ b/chrome/browser/resources/md_history/history.html
@@ -18,31 +18,18 @@
background: var(--md-background-color);
display: flex;
flex-direction: column;
- min-width: 800px;
- }
-
- #toolbar {
- flex: 0 0 56px;
- }
-
- #history-list,
- #history-synced-device-manager {
- flex: 1 0 0;
- }
-
- #main-container {
- display: flex;
- flex: 1 0 0;
- overflow: hidden;
}
/* Minimal styling required to display app shim. */
- .loading #main-container,
- .loading history-toolbar,
+ .loading history-app,
.app-shim {
display: none;
}
+ history-app {
+ flex: 1;
+ }
+
.loading .app-shim {
display: flex;
font-size: 123%;
@@ -57,7 +44,6 @@
height: 56px;
}
- history-toolbar,
.loading #loading-toolbar {
background: var(--md-toolbar-color);
}
@@ -73,15 +59,7 @@
</head>
<body class="loading">
- <history-toolbar class="paper-header" id="toolbar"></history-toolbar>
-
- <div id="main-container">
- <history-side-bar id="history-side-bar"></history-side-bar>
- <history-list id="history-list"></history-list>
- <history-synced-device-manager id="history-synced-device-manager" hidden>
- </history-synced-device-manager>
- </div>
-
+ <history-app id="history-app"></history-app>
<span class="app-shim" id="loading-toolbar">$i18n{title}</span>
<span class="app-shim" id="loading-message">$i18n{loading}</span>
@@ -92,7 +70,7 @@
<script src="strings.js"></script>
<script src="history.js"></script>
- <link rel="import" href="chrome://history/elements.html" async id="bundle">
+ <link rel="import" href="chrome://history/app.html" async id="bundle">
</body>
</html>
« no previous file with comments | « chrome/browser/resources/md_history/elements.html ('k') | chrome/browser/resources/md_history/history.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698