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

Unified Diff: popup.html

Issue 11627039: Added ad counting functionality (Closed)
Patch Set: Changed strings & using appLocale Created Sept. 18, 2013, 9:45 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
Index: popup.html
===================================================================
--- a/popup.html
+++ b/popup.html
@@ -76,14 +76,50 @@
width: 5px;
display: block;
}
+
+#statsStuff
Wladimir Palant 2013/09/18 13:22:17 #statsContainer? :)
+{
+ margin: 10px 5px;
+ padding: 5px;
+ border: 1px solid rgb(150, 150, 200);
+ border-radius: 5px;
+ white-space: nowrap;
+ background: -webkit-linear-gradient(top, rgb(250, 250, 255), rgb(200, 200, 255) 50px);
+ background: linear-gradient(top, rgb(250, 250, 255), rgb(200, 200, 255) 50px);
Wladimir Palant 2013/09/18 13:22:17 This isn't valid linear-gradient syntax, see https
+}
+
+#statsPage > strong,
+#statsTotal > strong
+{
+ display: inline-block;
+ min-width: 30px;
Wladimir Palant 2013/09/18 13:22:17 Hm, we'll easily see six-digit numbers here... May
+ color: rgb(255, 50, 50);
+ text-align: right;
+}
+
+#share
+{
+ display: block;
+ color: rgb(75, 75, 200);
+ text-align: right;
+ text-decoration: underline;
+}
+
+#shareBox > a
+{
+ display: block;
+ margin-top: 5px;
+}
</style>
<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>
+<script type="text/javascript" src="stats.js"></script>
</head>
<body id="main">
+
<div id="notification" style="display: none">
<h1 id="title"></h1>
<p id="message"></p>
@@ -101,6 +137,22 @@
<button id="cancelButton" class="ui-button ui-widget ui-state-default ui-corner-all ui-button-text-only"><span class="i18n_cancel"></span></button>
</div>
+<div id="statsStuff" hidden>
+ <strong class="i18n_stats_title"></strong>
+ <div id="stats">
+ <div id="statsPage" class="i18n_stats_label_page label"></div>
+ <div id="statsTotal" class="i18n_stats_label_total label"></div>
Wladimir Palant 2013/09/18 13:22:17 Please remove the i18n classes here, just fill the
+ </div>
+ <div>
+ <a id="share" class="i18n_stats_share_title" href="#"></a>
+ <div id="shareBox" hidden>
+ <a class="i18n_stats_share_label_fb ui-button ui-widget ui-state-default ui-corner-all" data-social="facebook"></a>
+ <a id="shareTwitter" class="i18n_stats_share_label_twitter ui-button ui-widget ui-state-default ui-corner-all" data-social="twitter"></a>
+ <a id="shareGplus" class="i18n_stats_share_label_gplus ui-button ui-widget ui-state-default ui-corner-all" data-social="gplus"></a>
+ </div>
+ </div>
+</div>
+
</div>
</body>
</html>

Powered by Google App Engine
This is Rietveld