Merge pull request #70 from screwdriver-cd/alltheicons

Update all the icons

.eslintrc.js

@@ -16,6 +16,7 @@ module.exports = {
     indent: ['error', 2],
     'no-underscore-dangle': 'off',
     'prefer-const': 'off',
+    'import/extensions': 'off',
     'import/no-extraneous-dependencies': 'off',
     'import/no-unresolved': 'off',
     'import/no-mutable-exports': 'off',

app/authenticators/screwdriver-api.js

@@ -83,7 +83,7 @@ export default Base.extend({
    * @return {Promise}
    */
   invalidate() {
-    return new Ember.RSVP.Promise(resolve => {
+    return new Ember.RSVP.Promise((resolve) => {
       Ember.$.ajax({
         url: logoutUrl,
         method: 'POST',

app/build/route.js

@@ -6,7 +6,7 @@ export default Ember.Route.extend({
     this.transitionTo('pipeline.builds.build', model.pipeline.id, model.build.id);
   },
   model(params) {
-    return this.store.findRecord('build', params.build_id).then(build => {
+    return this.store.findRecord('build', params.build_id).then((build) => {
       // reload again in a little bit if queued
       const reloadQueuedBuild = () => {
         if (build.get('status') === 'QUEUED') {

app/components/app-header/styles.scss

@@ -28,7 +28,6 @@
 
 .logo {
   display: inline-block;
-  width: 156px;
   height: 40px;
 
   img {

app/components/build-log/component.js

@@ -78,7 +78,7 @@ export default Ember.Component.extend({
             Ember.run(() => {
               if (Array.isArray(data) && data.length) {
                 // Add lines to log container
-                data.forEach(line => {
+                data.forEach((line) => {
                   this.set('logContent',
                     `${this.get('logContent')}${ansi_up.ansi_to_html(line.m)}\n`
                   );

app/components/home-hero/styles.scss

@@ -102,9 +102,8 @@
 }
 
 footer {
-  background: url('/assets/main_bkgrd_home.png') transparent;
-  background-size: cover;
-  color: $ycolor-gray-3;
+  background-color: $ycolor-gray-4;
+  color: $ycolor-gray-7;
   padding: 10px 0;
   font-size: 11px;
   font-weight: 300;
@@ -119,7 +118,7 @@ footer {
   }
 
   a {
-    color: $ycolor-gray-3;
+    color: $ycolor-gray-7;
     text-decoration: none;
   }
   a:hover {

app/components/pipeline-builds-list/component.js

@@ -6,7 +6,7 @@ export default Ember.Component.extend(Ember.PromiseProxyMixin, {
       const workflow = [];
 
       // get a sub-workflow object to clone
-      this.get('jobs').forEach(j => {
+      this.get('jobs').forEach((j) => {
         workflow.push({
           id: j.get('id'),
           name: j.get('name')
@@ -29,7 +29,7 @@ export default Ember.Component.extend(Ember.PromiseProxyMixin, {
       const shas = {};
       let result = [];
 
-      combined.forEach(build => {
+      combined.forEach((build) => {
         const sha = build.get('sha');
 
         // create a workflow based on shas
@@ -38,7 +38,7 @@ export default Ember.Component.extend(Ember.PromiseProxyMixin, {
         }
 
         // add the build info to correct job in the workflow
-        shas[sha].some(j => {
+        shas[sha].some((j) => {
           if (j.id === build.get('jobId') && !j.build) {
             j.build = build;
 
@@ -50,7 +50,7 @@ export default Ember.Component.extend(Ember.PromiseProxyMixin, {
       });
 
       // put the results into an array
-      Object.keys(shas).forEach(key => {
+      Object.keys(shas).forEach((key) => {
         result.push(shas[key]);
       });
 

app/index.html

@@ -14,23 +14,15 @@
     <link rel="stylesheet" href="{{rootURL}}assets/vendor.css">
     <link rel="stylesheet" href="{{rootURL}}assets/screwdriver-ui.css">
 
-    <link rel="apple-touch-icon" sizes="57x57" href="{{rootURL}}assets/icons/apple-icon-57x57.png">
-    <link rel="apple-touch-icon" sizes="60x60" href="{{rootURL}}assets/icons/apple-icon-60x60.png">
-    <link rel="apple-touch-icon" sizes="72x72" href="{{rootURL}}assets/icons/apple-icon-72x72.png">
-    <link rel="apple-touch-icon" sizes="76x76" href="{{rootURL}}assets/icons/apple-icon-76x76.png">
-    <link rel="apple-touch-icon" sizes="114x114" href="{{rootURL}}assets/icons/apple-icon-114x114.png">
-    <link rel="apple-touch-icon" sizes="120x120" href="{{rootURL}}assets/icons/apple-icon-120x120.png">
-    <link rel="apple-touch-icon" sizes="144x144" href="{{rootURL}}assets/icons/apple-icon-144x144.png">
-    <link rel="apple-touch-icon" sizes="152x152" href="{{rootURL}}assets/icons/apple-icon-152x152.png">
-    <link rel="apple-touch-icon" sizes="180x180" href="{{rootURL}}assets/icons/apple-icon-180x180.png">
-    <link rel="icon" type="image/png" sizes="192x192"  href="{{rootURL}}assets/icons/android-icon-192x192.png">
-    <link rel="icon" type="image/png" sizes="32x32" href="{{rootURL}}assets/icons/favicon-32x32.png">
-    <link rel="icon" type="image/png" sizes="96x96" href="{{rootURL}}assets/icons/favicon-96x96.png">
-    <link rel="icon" type="image/png" sizes="16x16" href="{{rootURL}}assets/icons/favicon-16x16.png">
-    <link rel="manifest" href="{{rootURL}}assets/icons/manifest.json">
-    <meta name="msapplication-TileColor" content="#0034a3">
-    <meta name="msapplication-TileImage" content="{{rootURL}}assets/icons/ms-icon-144x144.png">
-    <meta name="theme-color" content="#0034a3">
+
+    <link rel="apple-touch-icon" sizes="180x180" href="{{rootURL}}assets/icons/apple-touch-icon.png?v=A0RbkpndO6">
+    <link rel="icon" type="image/png" href="{{rootURL}}assets/icons/favicon-32x32.png?v=A0RbkpndO6" sizes="32x32">
+    <link rel="icon" type="image/png" href="{{rootURL}}assets/icons/favicon-16x16.png?v=A0RbkpndO6" sizes="16x16">
+    <link rel="manifest" href="{{rootURL}}assets/icons/manifest.json?v=A0RbkpndO6">
+    <link rel="mask-icon" href="{{rootURL}}assets/icons/safari-pinned-tab.svg?v=A0RbkpndO6" color="#0034a3">
+    <link rel="shortcut icon" href="{{rootURL}}assets/icons/favicon.ico?v=A0RbkpndO6">
+    <meta name="msapplication-config" content="{{rootURL}}assets/icons/browserconfig.xml?v=A0RbkpndO6">
+    <meta name="theme-color" content="#b9bdc5">
 
     {{content-for "head-footer"}}
   </head>

app/pipeline/builds/build/route.js

@@ -6,7 +6,7 @@ export default Ember.Route.extend({
     return `${model.job.get('name')} > #${model.build.get('sha').substr(0, 6)}`;
   },
   model(params) {
-    return this.store.findRecord('build', params.build_id).then(build => {
+    return this.store.findRecord('build', params.build_id).then((build) => {
       // reload again in a little bit if queued
       const reloadQueuedBuild = () => {
         if (build.get('status') === 'QUEUED') {

app/pipeline/builds/index/route.js

@@ -9,16 +9,16 @@ import Ember from 'ember';
 function getBuilds(jobs) {
   const buildList = [];
 
-  jobs.forEach(j => {
+  jobs.forEach((j) => {
     buildList.push(j.get('builds'));
   });
 
   // fetch all the builds
-  return Ember.RSVP.all(buildList).then(builds => {
+  return Ember.RSVP.all(buildList).then((builds) => {
     let combined = [];
 
     // put all the builds in a single array
-    builds.forEach(b => {
+    builds.forEach((b) => {
       combined = combined.concat(b.slice());
     });
 
@@ -33,7 +33,7 @@ export default Ember.Route.extend({
   model() {
     return this.get('pipeline.jobs')
       // Split jobs from workflow
-      .then(jobs => {
+      .then((jobs) => {
         // filter out the PRs
         const workflowJobs = jobs.filter(j => !/^PR\-/.test(j.get('name')));
 

app/pipeline/secrets/controller.js

@@ -7,7 +7,7 @@ export default Ember.Controller.extend({
       const newSecret = this.store.createRecord('secret', { name, value, pipelineId, allowInPR });
 
       return newSecret.save()
-        .then(s => {
+        .then((s) => {
           this.set('errorMessage', '');
           this.get('model.secrets').reload();
 

app/secret/serializer.js

@@ -14,7 +14,7 @@ export default DS.RESTSerializer.extend({
 
     const dirty = snapshot.changedAttributes();
 
-    Object.keys(dirty).forEach(key => {
+    Object.keys(dirty).forEach((key) => {
       dirty[key] = dirty[key][1];
     });
 

app/styles/fuji-colors.scss

@@ -37,6 +37,7 @@ $ycolor-gray-2: #f1f1f5;
 $ycolor-gray-3: #e0e4e9;
 $ycolor-gray-4: #d8dade;
 $ycolor-gray-6: #979ba7;
+$ycolor-gray-7: #878c91;
 $ycolor-gray-11: #26282a;
 
 /* colors,starting at noon position and moving clockwise in 1/18 slices */

public/assets/icons/browserconfig.xml

@@ -1,2 +1,9 @@
-<?xml version="1.0" encoding="utf-8"?>
-<browserconfig><msapplication><tile><square70x70logo src="/assets/icons/ms-icon-70x70.png"/><square150x150logo src="/assets/icons/ms-icon-150x150.png"/><square310x310logo src="/assets/icons/ms-icon-310x310.png"/><TileColor>#ffffff</TileColor></tile></msapplication></browserconfig>
+<?xml version="1.0" encoding="utf-8"?>
+<browserconfig>
+  <msapplication>
+    <tile>
+      <square150x150logo src="/assets/icons/mstile-150x150.png?v=A0RbkpndO6"/>
+      <TileColor>#b9bdc5</TileColor>
+    </tile>
+  </msapplication>
+</browserconfig>

public/assets/icons/manifest.json

@@ -1,41 +1,6 @@
 {
- "name": "Screwdriver",
- "icons": [
-  {
-   "src": "\/assets\/icons\/android-icon-36x36.png",
-   "sizes": "36x36",
-   "type": "image\/png",
-   "density": "0.75"
-  },
-  {
-   "src": "\/assets\/icons\/android-icon-48x48.png",
-   "sizes": "48x48",
-   "type": "image\/png",
-   "density": "1.0"
-  },
-  {
-   "src": "\/assets\/icons\/android-icon-72x72.png",
-   "sizes": "72x72",
-   "type": "image\/png",
-   "density": "1.5"
-  },
-  {
-   "src": "\/assets\/icons\/android-icon-96x96.png",
-   "sizes": "96x96",
-   "type": "image\/png",
-   "density": "2.0"
-  },
-  {
-   "src": "\/assets\/icons\/android-icon-144x144.png",
-   "sizes": "144x144",
-   "type": "image\/png",
-   "density": "3.0"
-  },
-  {
-   "src": "\/assets\/icons\/android-icon-192x192.png",
-   "sizes": "192x192",
-   "type": "image\/png",
-   "density": "4.0"
-  }
- ]
+	"name": "screwdriver.cd",
+	"icons": [
+	],
+	"theme_color": "#b9bdc5"
 }

tests/acceptance/pipeline-builds-test.js

@@ -68,7 +68,7 @@ const shas = [
 const makeBuilds = (jobId) => {
   const builds = [];
 
-  shas.forEach(sha => {
+  shas.forEach((sha) => {
     const b = Ember.copy(BUILD, true);
     const config = {
       id: Math.floor(Math.random() * 99999999999),

tests/unit/application/adapter-test.js

@@ -34,7 +34,7 @@ test('it uses cors for ajax', function (assert) {
 
   let adapter = this.subject();
 
-  return adapter.ajax('fakeurl', 'GET').then(response => {
+  return adapter.ajax('fakeurl', 'GET').then((response) => {
     assert.deepEqual(response, { foo: 'bar' });
   });
 });