Commit 5898111d authored by Michael Checo's avatar Michael Checo Committed by Commit Bot

Diagnostics: Base card styling and app layout

Description:
 * These styles are subject to change and are only for
   the base template.

 - Adds container styles to diagnostics app
 - Adds some styling to diagnostics-card
 - displays cards vertically

Screenshots:
 - Diagnostics Page (before) http://shortn/_2lCAX0PQSg
 - Diagnostics Page (after) http://shortn/_h8KubnXkcH

Bug: 01125150
Test: browser_tests --gtest_filter=DiagnosticsApp*
Change-Id: Idb17a82cbb1ccca99042872057c2979e0480ebba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425199Reviewed-by: default avatarZentaro Kavanagh <zentaro@chromium.org>
Reviewed-by: default avatarJimmy Gong <jimmyxgong@chromium.org>
Commit-Queue: Michael Checo <michaelcheco@google.com>
Cr-Commit-Position: refs/heads/master@{#810636}
parent e810de82
<style include="cr-shared-style diagnostics-shared diagnostics-fonts"> <style include="cr-shared-style diagnostics-shared diagnostics-fonts">
.diagonstics-cards-container {
display: flex;
flex-direction: column;
}
#diagnosticsContainer {
--cr-centered-card-max-width: 820px;
}
#header { #header {
@apply --diagnostics-header-font; @apply --diagnostics-header-font;
margin-bottom: 10px; margin-bottom: 10px;
...@@ -8,13 +17,15 @@ ...@@ -8,13 +17,15 @@
margin-bottom: 10px; margin-bottom: 10px;
} }
</style> </style>
<div class="main-container"> <div id="diagnosticsContainer"class="cr-centered-card-container">
<!-- TODO(zentaro) Localize once strings are confirmed --> <!-- TODO(zentaro) Localize once strings are confirmed -->
<div id="header">Diagnostics</div> <div id="header">Diagnostics</div>
<div class="overview-container"> <div class="overview-container">
<overview-card id="overviewCard"></overview-card> <overview-card id="overviewCard"></overview-card>
</div> </div>
<memory-card id="memoryCard"></memory-card> <div class="diagonstics-cards-container">
<cpu-card id="cpuCard"></cpu-card> <memory-card id="memoryCard"></memory-card>
<battery-status-card id="batteryStatusCard"></battery-status-card> <cpu-card id="cpuCard"></cpu-card>
<battery-status-card id="batteryStatusCard"></battery-status-card>
</div>
</div> </div>
<style include="diagnostics-shared diagnostics-fonts"> <style include="diagnostics-shared diagnostics-fonts">
#card-wrapper {
border: 1px solid var(--diagnostics-border-color);
border-radius: 14px;
height: 180px;
margin: 10px 0;
}
.title-container { .title-container {
@apply --diagnostics-default-font; @apply --diagnostics-default-font;
} }
.spacing {
margin-top: 18px;
padding-inline-end: 15px;
padding-inline-start: 15px;
}
</style> </style>
<div id="card-wrapper"> <div id="card-wrapper">
<div id="title" class="title-container"> <div id="title" class="title-container spacing">
<slot name="title"></slot> <slot name="title"></slot>
</div> </div>
<div id="body" class="body-container"> <div id="body" class="body-container spacing">
<slot name="body"></slot> <slot name="body"></slot>
</div> </div>
</div> </div>
...@@ -4,6 +4,11 @@ ...@@ -4,6 +4,11 @@
<link rel="stylesheet" href="chrome://resources/css/md_colors.css"> <link rel="stylesheet" href="chrome://resources/css/md_colors.css">
<template> <template>
<style include="cr-shared-style diagnostics-fonts"> <style include="cr-shared-style diagnostics-fonts">
:host {
/* TODO(michael) Replace temp hex code when colors are finalized. */
--diagnostics-border-color: #c5c7ca;
}
html { html {
background-color: var(--cros-bg-color); background-color: var(--cros-bg-color);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment