Improve assets loading: remove http call to load task colors and async load of app.js

app/Controller/App.php

@@ -93,16 +93,6 @@ class App extends Base
         $this->response->html($this->helper->text->markdown($payload['text']));
     }
 
-    /**
-     * Colors stylesheet
-     *
-     * @access public
-     */
-    public function colors()
-    {
-        $this->response->css($this->color->getCss());
-    }
-
     /**
      * Task autocompletion (Ajax)
      *

app/Helper/Asset.php

@@ -16,9 +16,9 @@ class Asset extends \Core\Base
      * @param  string   $filename   Filename
      * @return string
      */
-    public function js($filename)
+    public function js($filename, $async = false)
     {
-        return '<script type="text/javascript" src="'.$filename.'?'.filemtime($filename).'"></script>';
+        return '<script '.($async ? 'async' : '').' type="text/javascript" src="'.$filename.'?'.filemtime($filename).'"></script>';
     }
 
     /**
@@ -48,4 +48,15 @@ class Asset extends \Core\Base
 
         return '';
     }
+
+    /**
+     * Get CSS for task colors
+     *
+     * @access public
+     * @return string
+     */
+    public function colorCss()
+    {
+        return '<style>'.$this->color->getCss().'</style>';
+    }
 }

app/Model/Acl.php

@@ -21,7 +21,6 @@ class Acl extends Base
         'user' => array('google', 'github'),
         'task' => array('readonly'),
         'board' => array('readonly'),
-        'app' => array('colors'),
         'webhook' => '*',
         'ical' => '*',
         'feed' => '*',

app/Template/analytic/layout.php

@@ -1,6 +1,3 @@
-<?= $this->asset->js('assets/js/vendor/d3.v3.min.js') ?>
-<?= $this->asset->js('assets/js/vendor/c3.min.js') ?>
-
 <section id="main">
     <div class="page-header">
         <ul>
@@ -32,4 +29,7 @@
             <?= $content_for_sublayout ?>
         </div>
     </section>
-</section>
+</section>
+
+<?= $this->asset->js('assets/js/vendor/d3.v3.min.js') ?>
+<?= $this->asset->js('assets/js/vendor/c3.min.js') ?>

app/Template/budget/index.php

@@ -1,6 +1,3 @@
-<?= $this->asset->js('assets/js/vendor/d3.v3.min.js') ?>
-<?= $this->asset->js('assets/js/vendor/c3.min.js') ?>
-
 <div class="page-header">
     <h2><?= t('Budget overview') ?></h2>
 </div>
@@ -32,3 +29,6 @@
 <?php else: ?>
     <p class="alert"><?= t('There is not enough data to show something.') ?></p>
 <?php endif ?>
+
+<?= $this->asset->js('assets/js/vendor/d3.v3.min.js') ?>
+<?= $this->asset->js('assets/js/vendor/c3.min.js') ?>

app/Template/layout.php

@@ -12,12 +12,12 @@
         <?php endif ?>
 
         <?php if (! isset($not_editable)): ?>
-            <?= $this->asset->js('assets/js/app.js') ?>
+            <?= $this->asset->js('assets/js/app.js', true) ?>
         <?php endif ?>
 
-        <?= $this->asset->css($this->url->href('app', 'colors'), false, 'all') ?>
         <?= $this->asset->css('assets/css/app.css') ?>
         <?= $this->asset->css('assets/css/print.css', true, 'print') ?>
+        <?= $this->asset->colorCss() ?>
         <?= $this->asset->customCss() ?>
 
         <link rel="icon" type="image/png" href="assets/img/favicon.png">

tests/units/AclTest.php

@@ -37,7 +37,6 @@ class AclTest extends Base
         $acl = new Acl($this->container);
         $this->assertTrue($acl->isPublicAction('board', 'readonly'));
         $this->assertFalse($acl->isPublicAction('board', 'show'));
-        $this->assertTrue($acl->isPublicAction('app', 'colors'));
         $this->assertTrue($acl->isPublicAction('feed', 'project'));
         $this->assertTrue($acl->isPublicAction('feed', 'user'));
     }