Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: popup.html

Issue 11161031: Show notifications on startup (Chrome) (Closed)
Patch Set: Address remaining issue Created July 23, 2013, 11:24 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« background.js ('K') | « notification.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: popup.html
===================================================================
--- a/popup.html
+++ b/popup.html
@@ -31,7 +31,34 @@
}
a { text-decoration: none }
-a:hover { color: #eeeeff }
+
+#notification
+{
+ min-width: 240px;
+ margin-bottom: 10px;
+ border-radius: 5px;
+ border: 2px solid;
+}
+
+#notification.information
+{
+ border-color: #09d;
+}
+
+#notification.critical
+{
+ border-color: #e40;
+}
+
+#notification>h1
+{
+ font-size: 16px;
+}
+
+#notification>*
+{
+ margin: 5px;
+}
#filtersList {
border: 1px dotted gray;
@@ -54,8 +81,14 @@
<script type="text/javascript" src="jquery-ui/js/jquery-1.7.1.min.js"></script>
<script type="text/javascript" src="i18n.js"></script>
<script type="text/javascript" src="popup.js"></script>
+<script type="text/javascript" src="notification.js"></script>
</head>
<body id="main">
+<div id="notification" style="display: none">
+<h1 id="title"></h1>
+<p id="message"></p>
+</div>
+
<div id="enabledCheckboxAndLabel" style="display:none"><input id="enabled" type="checkbox" checked><label for="enabled"><span class="i18n_enabled_for_site"></span></label></div>
<div id="clickHideInactiveStuff" style="display: none">
<div class="spacer"></div>
« background.js ('K') | « notification.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld