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

Unified Diff: web/apps/logdog-view/elements/logdog-view/logdog-view.html

Issue 2570963003: Revert of Rewrite LogDog log viewer app. (Closed)
Patch Set: Created 4 years 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: web/apps/logdog-view/elements/logdog-view/logdog-view.html
diff --git a/web/apps/logdog-view/elements/logdog-view/logdog-view.html b/web/apps/logdog-view/elements/logdog-view/logdog-view.html
index c2a2328ab0507f34def82e1e6c2e84ef90537730..e7e181ab5b271c744f2e5538d835df93eb6be414 100644
--- a/web/apps/logdog-view/elements/logdog-view/logdog-view.html
+++ b/web/apps/logdog-view/elements/logdog-view/logdog-view.html
@@ -32,9 +32,7 @@
#banner {
background-color: #FAFAFA;
overflow: hidden;
- border-color: lightcoral;
- border-width: 0px 0px 2px 0px;
- border-style: solid;
+ vertical-align: center;
}
.banner-content {
@@ -52,7 +50,12 @@
#auth-bubble {
float: right;
display: inline-block;
- border: none;
+ border: solid #AAAAAA;
+ border-radius: 5px;
+ border-width: 1px;
+ padding: 2px;
+ margin-left: auto;
+ margin-right: 0;
}
</style>
@@ -76,14 +79,12 @@
</template>
</div>
- <logdog-stream-view
- id="streamView"
- host="[[host]]"
- mobile="[[mobile]]"
- metadata="[[metadata]]"
- toolbar-anchor="{{banner}}"
- toolbarAnchor="{{banner}}"
- streams="[[streams]]"></logdog-stream-view>
+ <template is="dom-if" if="{{_hasStreams}}">
+ <logdog-stream-view
+ id="view"
+ host="[[host]]"
+ streams="[[streams]]"></logdog-stream-view>
+ </template>
</div>
</template>
@@ -96,29 +97,20 @@
host: {
type: String,
notify: true,
- value: null,
},
-
clientId: {
type: String,
- value: null,
+ nofity: true,
},
-
+ accessToken: {
+ type: String,
+ notify: true,
+ readOnly: true,
+ },
streams: {
type: Array,
- value: [],
+ notify: true,
readOnly: true,
- },
-
- metadata: {
- type: String,
- value: false,
- readOnly: true,
- },
-
- mobile: {
- type: Boolean,
- value: false,
},
_hasStreams: {
@@ -126,22 +118,12 @@
},
},
- ready: function() {
- // Test if we're mobile.
- this.mobile = false;
- if ( window.matchMedia ) {
- this.mobile = window.matchMedia(
- "only screen and (max-width: 760px)").maches;
- }
-
+ attached: function() {
// Parse the stream names from the "s" query parameter.
var queryString = window.location.search.replace(/^\?/, "");
var streams = logdog.getQueryValues(queryString, "s").
map(logdog.correctStreamPath);
this._setStreams(streams);
-
- // Parse the metadata option.
- this._setMetadata(!!logdog.getQueryValue(queryString, "m", false));
},
_onSignin: function(e) {
« no previous file with comments | « web/apps/logdog-view/elements/logdog-view/logdog-scripts-main.html ('k') | web/apps/logdog-view/gulpfile.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698