Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Welcome plug-in shouldn't open welcome page each page load #95

Closed
mmorhun opened this issue Mar 6, 2019 · 15 comments
Closed

Welcome plug-in shouldn't open welcome page each page load #95

mmorhun opened this issue Mar 6, 2019 · 15 comments
Assignees
Labels
kind/bug Something isn't working status/code-review

Comments

@mmorhun
Copy link
Contributor

mmorhun commented Mar 6, 2019

When @eclipse-che/welcome-plugin is included it opens welcome page every page load. It happens each page reload even if plenty of editors are already opened. Also focus goes to that new tab, so user have to manually close it. The same problem when a user switches to dashboard and goes back to Che Theia.

@mmorhun mmorhun added the kind/bug Something isn't working label Mar 6, 2019
@sunix
Copy link
Contributor

sunix commented Mar 7, 2019

Should open welcome page only once/the first time you run the workspace

@vitaliy-guliy
Copy link
Contributor

Or just when there is no opened files

@benoitf
Copy link
Contributor

benoitf commented Mar 14, 2019

as each person seems to have its own view, I don't know what should be the expected behavior

But we could have an option to turn it off (like VSCode)

@vitaliy-guliy
Copy link
Contributor

Let's start from having this option

@benoitf
Copy link
Contributor

benoitf commented Mar 14, 2019

turn it off per workspace or per user ? or user has the two choices ?

@vitaliy-guliy
Copy link
Contributor

I prefer per workspace option

@vitaliy-guliy
Copy link
Contributor

Disable by pressing the button or a check box on the Welcome Page

@AndrienkoAleksandr
Copy link
Contributor

AndrienkoAleksandr commented Apr 5, 2019

+1 per workspace option

@AndrienkoAleksandr AndrienkoAleksandr self-assigned this Apr 8, 2019
@AndrienkoAleksandr AndrienkoAleksandr added the status/in-progress Indicates the issue is taken in progress by assignee label Apr 8, 2019
@AndrienkoAleksandr AndrienkoAleksandr added status/code-review and removed status/in-progress Indicates the issue is taken in progress by assignee labels Apr 9, 2019
@makandre
Copy link

For products that want to build on top of Che Theia and maybe supply their own welcome page, can this option be set somewhere (in the stack definition perhaps)?

@AndrienkoAleksandr
Copy link
Contributor

Hello @makandre . This changes introduced to welcome-plugin. In case if you build own IDE on top of che-theia you could apply own list predefined plugins. If you don't need welcome plugin - you could postpone this plugin, or apply yours analog. che-theia has folder in the image and in this folder stored predefined plugins. CHE Theia process knows where is this folder located with help environment variables:
THEIA_PLUGINS
f.e: THEIA_PLUGINS=local-dir:///plugins
So, as you can see, plugins are not compiled in the theia binaries, they stored in the separated folder or even more - in separated container(in case remote che-theia plugin).

@makandre
Copy link

@AndrienkoAleksandr Thanks for your reply. But could you explain what you meant by "you could postpone this plugin" or how to apply my own list of plugins? Sorry I'm new to Che Theia

@AndrienkoAleksandr
Copy link
Contributor

@makandre we have che-theia image: https://github.com/eclipse/che-theia/tree/master/dockerfiles/theia/
For ci build image, we used file where is defined list of Theia extensions and plugins https://github.com/eclipse/che-theia/blob/master/extensions/extensions.yml#L22 .
It used in logic here: https://github.com/eclipse/che-theia/blob/master/dockerfiles/theia/Dockerfile#L68
As you can see we have che:theia tools and this tool applies plugins and images to the theia from yaml file.

@sunix
Copy link
Contributor

sunix commented Apr 15, 2019

Welcome page is provided through a Theia plugin (a .theia file), you can include it or exclude in your product or replace it by your own. You can have a look in how it is done: https://github.com/eclipse/che-theia/tree/master/plugins/welcome-plugin

@makandre
Copy link

Thanks for the info! Just to make sure I understand, the suggested methods involve re-building/re-packaging Che Theia right? Are there any ways to use Che Theia as-is with some runtime configuration change to include/exclude other plugins? Thanks.

@benoitf
Copy link
Contributor

benoitf commented Apr 15, 2019

@makandre you could set THEIA_DEFAULT_PLUGINS to an empty variable or override it with another location (like mounting a volume where you've all your *.theia files) or mount all your customized theia files to /default-theia-plugins location

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug Something isn't working status/code-review
Projects
None yet
Development

No branches or pull requests

6 participants