-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Start workspace with workspace class override #15287
Comments
I think a "start with options" - URL combined with a prompting page that displays the available options is the most versatile option and communicates well with the user. |
FYI there's already a URL kind-of-parameter implementation to select JetBrains IDEs Line 79 in 96a7836
|
On it. Thanks for the ping, @svenefftinge! 🏓 Jotting down some notes:
|
@gtsiolis and me worked on this and would like to approach it as follows: We replace/extend the current open workspace modal to allow selecting:
(we can later add additional things, such as e.g. environment variables) The modal is triggerable and pre-fillable using a query param. If no arguments are provided the context URL must be chosen (using the search we have now). The other two args will be set to what the defaults are for the given context URL (i.e. if I pick a context URL on a project that is set to use large workspaces, large workspace class is pre-selected, if I have IntelliJ as my default IDE, it is preselected as well). The model will be embedded in We should allow using these options on restarts of workspaces as well, where the properties are preselected from the last workspace instance. George is going to provide some UI designs. |
Posting below some designs after discussing with @svenefftinge:
See design specs (internal). |
This is a followup to #15080
Gitpod users should be able to override the workspace class (determined globally or in project settings.) E.g. when opening a workspace on a repo which does not have a Gitpod project.
🛹 ideas
gitpod.io/#<<<REPO-URL>>>&gitpod-workspace-class=large
cc: @gtsiolis @atduarte
The text was updated successfully, but these errors were encountered: