-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
are the changes for core packages proposed upstream ? |
* Adds 'Development Host' status bar element and appends the same prefix to window title. | ||
*/ | ||
@injectable() | ||
export class HostedPluginInformer { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this class should implement FrontendApplicationContribution
and registered through
bind(FrontendApplicationContribution).to(HostedPluginInformer).inSingletonScope();
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix line ending please.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just added new empty line to the end
@@ -87,6 +80,7 @@ export class StatusBarImpl extends VirtualWidget implements StatusBar { | |||
|
|||
async setElement(id: string, entry: StatusBarEntry): Promise<void> { | |||
await this.ready; | |||
(entry as any).elementId = id; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we apply elementId to the StatusBarEntry interface to prevent such casting: (entry as any)
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has been already reworked
PR to upstream containing proposals to core package merged! |
import { StatusBarAlignment, StatusBarEntry, FrontendApplicationContribution } from '@theia/core/lib/browser'; | ||
import { HostedPluginServer } from '../../common/plugin-protocol'; | ||
import { HostedPluginManagerClient, HostedPluginState, HostedPluginCommands } from './plugin-manager-client'; | ||
import { CommandRegistry } from "@phosphor/commands"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's use " or ' - common for all
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, thanks!
if (!pluginMetadata) { | ||
this.frontendApplicationStateService.reachedState('ready').then(() => { | ||
this.hostedPluginManagerClient.onStateChanged(e => { | ||
if (e === 'starting') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can be moved to constants or enum
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, thanks!
| 'running' | ||
| 'stopping' | ||
| 'failed'; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will be better to use :
export enum HostedPluginState {
Stopped = 'stopped',
Starting = 'starting',
Running = 'running',
Stopping= 'stopping',
Failed = 'failed'
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
* UI for development mode * Code improvements
* UI for development mode * Code improvements
* UI for development mode * Code improvements
* UI for development mode * Code improvements
* UI for development mode * Code improvements Signed-off-by: Vitaliy Gulyy <vguliy@codenvy.com>
Adds a status bar element displaying the state of Theia instance with hosted plugin.
Adds a status bar element notifying that Theia is running in dev mode with hosted plugin.
Issues to be resolved
Add info about Dev mode instance to the main Theia one eclipse-che/che#9402
Add UI to differ developer mode in Theia eclipse-che/che#9214