maxmustermann/kibana
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Clear core loading indicator just before UI is rendered (#55242)

Browse source 
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
This commit is contained in:
Josh Dover 2020-01-21 09:19:08 -07:00 committed by GitHub
parent 2bf111c50f
commit b0af1bf95c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/core/public/core_system.test.ts
View file

@ -429,15 +429,14 @@ describe('Notifications targetDomElement', () => {
rootDomElement,
});
let targetDomElementParentInStart: HTMLElement | null;
let targetDomElementInStart: HTMLElement | null;
MockNotificationsService.start.mockImplementation(({ targetDomElement }): any => {
expect(targetDomElement.parentElement).not.toBeNull();
targetDomElementParentInStart = targetDomElement.parentElement;
targetDomElementInStart = targetDomElement;
});
// Starting the core system should pass the targetDomElement as a child of the rootDomElement
await core.setup();
await core.start();
expect(targetDomElementParentInStart!).toBe(rootDomElement);
expect(targetDomElementInStart!.parentElement).toBe(rootDomElement);
});
});

15
src/core/public/core_system.ts
View file

@ -222,13 +222,6 @@ export class CoreSystem {
const notificationsTargetDomElement = document.createElement('div');
const overlayTargetDomElement = document.createElement('div');
// ensure the rootDomElement is empty
this.rootDomElement.textContent = '';
this.rootDomElement.classList.add('coreSystemRootDomElement');
this.rootDomElement.appendChild(coreUiTargetDomElement);
this.rootDomElement.appendChild(notificationsTargetDomElement);
this.rootDomElement.appendChild(overlayTargetDomElement);
const overlays = this.overlay.start({
i18n,
targetDomElement: overlayTargetDomElement,
@ -276,6 +269,14 @@ export class CoreSystem {
};
const plugins = await this.plugins.start(core);
// ensure the rootDomElement is empty
this.rootDomElement.textContent = '';
this.rootDomElement.classList.add('coreSystemRootDomElement');
this.rootDomElement.appendChild(coreUiTargetDomElement);
this.rootDomElement.appendChild(notificationsTargetDomElement);
this.rootDomElement.appendChild(overlayTargetDomElement);
const rendering = this.rendering.start({
application,
chrome,