config_service: improve landing page

appengine/config_service/ui/src/config-ui/front-page.html

+<!--
+  Copyright 2017 The LUCI Authors. All rights reserved.
+  Use of this source code is governed under the Apache License, Version 2.0
+  that can be found in the LICENSE file.
+-->
+
+<link rel="import" href="../../bower_components/polymer/polymer.html">
+<link rel="import" href="../../bower_components/app-layout/app-layout.html">
+<link rel="import" href="../../bower_components/paper-search/paper-search-bar.html">
+<link rel="import" href="../../bower_components/paper-button/paper-button.html">
+<link rel="import" href="../../bower_components/iron-ajax/iron-ajax.html">
+<link rel="import" href="config-card.html">
+
+<dom-module id="front-page">
+  <template>
+    <style>
+      app-toolbar {
+        background-color: #efefef;
+        color: #232323;
+      }
+
+      app-header {
+        @apply --layout-fixed-top;
+      }
+
+      .logo {
+        height: 100%;
+      }
+
+      .title {
+        padding-left: 1%;
+        font-size: 150%;
+        font-family: sans-serif;
+      }
+
+      .search-bar {
+        padding-top: 7%;
+        padding-bottom: 2%;
+      }
+
+      paper-search-bar {
+        border-style: solid;
+        border-width: 2px;
+        width: 40%;
+        height: 100%;
+        margin: auto;
+      }
+    </style>
+
+    <app-header reveals>
+      <app-toolbar>
+        <image class="logo" src="../../images/chromium.png"/>
+        <div class="title" main-title>Configuration Service (not fully implemented)</div>
+      </app-toolbar>
+    </app-header>
+
+    <iron-ajax
+      auto
+      id="requestConfigs"
+      url="{{url}}"
+      handle-as="json"
+      on-response="_getConfigs">
+    </iron-ajax>
+
+    <div class="search-bar">
+      <paper-search-bar query="{{search}}"></paper-search-bar>
+    </div>
+    
+    <div class="config-list">
+      <template is="dom-if" if="{{_not(isLoading)}}">
+        <template is="dom-if" if="{{_not(isEmpty)}}">
+          <template is="dom-repeat" items="[[searchList]]" as="config">
+            <config-card name="[[config.config_set]]"></config-card>
+          </template>
+        </template>
+        <template is="dom-if" if="{{isEmpty}}">
+          <config-card name="No config sets found."></config-card>
+        </template>
+      </template>
+      <template is="dom-if" if="{{isLoading}}">
+        <config-card name="Fetching configs sets..."></config-card>
+      </template>
+    </div>
+  </template>
+
+  <script>
+    Polymer({
+      is: 'front-page',
+
+      properties: {
+        configList: {
+          type: Array,
+          value: []
+        },
+
+        isEmpty: {
+          type: Boolean,
+          value: true
+        },
+
+        isLoading: {
+          type: Boolean,
+          value: true
+        },
+
+        search: {
+          type: String,
+          observer: '_searched'
+        },
+
+        searchList: {
+          type: Array,
+          value: []
+        },
+
+        url: {
+          type: String,
+          // TODO(cwpayton): Fix URL to reflect specific instances of the luci
+          // config service (i.e., just "/_ah/api/..." instead of "https://luci...").
+          // Currently, this link is being left in on purpose in order to grab data
+          // for local instances, but this should change when deploying final product.
+          value: "https://luci-config.appspot.com/_ah/api/config/v1/config-sets"

[Sergey Berezin (google), 2017/06/12 22:19:49]

I'd say, let's remove the domain part of the URL from the code before landing
this. It's easy to put it back locally for testing, and eventually we should
come up with a way to serve test data locally.

[cwpayton, 2017/06/12 23:40:05]

Done.

+        }
+      },
+
+      _getConfigs: function(event) {
+        this.configList = event.detail.response.config_sets;
+        this.searchList = this._searched();

[Sergey Berezin (google), 2017/06/12 22:19:49]

No need to assign to this.searchList; _searched() already updates it.

[cwpayton, 2017/06/12 23:40:05]

This needs to stay in because searchList has no initial value. If this line is
removed, then when a user first lands on a page, they will get the "no config
sets found" message until they actually start typing a search query.

+        this.isLoading = false;
+        this.isEmpty = this.searchList.length === 0;
+      },
+
+      _not: function(bool) {
+        return (!(bool));
+      },
+
+      _searched: function() {
+        this.searchList = [];
+        this.configList.forEach((elem) => {
+          if (elem.config_set.includes(this.search)) {
+            this.searchList.push(elem);
+          }
+        });
+        this.isEmpty = this.searchList.length === 0;
+        return this.searchList;

[Sergey Berezin (google), 2017/06/12 22:19:49]

No need to return the result, since it modifies this.seachList.

[cwpayton, 2017/06/12 23:40:05]

The return statement is in so that searchList will be returned when _searched is
called in line 128

+      }
+    });
+  </script>
+</dom-module>