summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorSean McGivern <sean@gitlab.com>2018-06-27 11:11:08 +0100
committerSean McGivern <sean@gitlab.com>2018-06-27 12:04:25 +0100
commit52d578229a64cb8a6eedbfb35e77e8d5c6d68ac0 (patch)
treef60812d1b56f95a42c000e3419ba522e1e650117 /app
parent87f7597a4fb7852fc81f830158cdfd5fdec8fac4 (diff)
downloadgitlab-ce-52d578229a64cb8a6eedbfb35e77e8d5c6d68ac0.tar.gz
Start performance bar eagerly
On the issues page (for instance), we have an `Issue` class. The constructor for that class makes some calls to get things like related branches using Axios. However, those calls were happening before the performance bar app was mounted, and so the interceptor wasn't configured - meaning we missed out on the performance data for those calls. This changes the performance bar app to be loaded eagerly instead of lazily, which solves that problem - related branches calls will now show up in the performance bar. The downside is that now the performance bar app is always imported, even when it's not used. However, it appears to be part of the main app bundle anyway, so this shouldn't affect the size of our assets.
Diffstat (limited to 'app')
-rw-r--r--app/assets/javascripts/dispatcher.js6
1 files changed, 2 insertions, 4 deletions
diff --git a/app/assets/javascripts/dispatcher.js b/app/assets/javascripts/dispatcher.js
index b755458aa4b..a5af37e80b6 100644
--- a/app/assets/javascripts/dispatcher.js
+++ b/app/assets/javascripts/dispatcher.js
@@ -1,12 +1,12 @@
/* eslint-disable consistent-return, no-new */
import $ from 'jquery';
-import Flash from './flash';
import GfmAutoComplete from './gfm_auto_complete';
import { convertPermissionToBoolean } from './lib/utils/common_utils';
import GlFieldErrors from './gl_field_errors';
import Shortcuts from './shortcuts';
import SearchAutocomplete from './search_autocomplete';
+import performanceBar from './performance_bar';
function initSearch() {
// Only when search form is present
@@ -72,9 +72,7 @@ function initGFMInput() {
function initPerformanceBar() {
if (document.querySelector('#js-peek')) {
- import('./performance_bar')
- .then(m => new m.default({ container: '#js-peek' })) // eslint-disable-line new-cap
- .catch(() => Flash('Error loading performance bar module'));
+ performanceBar({ container: '#js-peek' });
}
}