-
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
Provide curated list of containers to run tasks #13390
Comments
@slemeur, we can't assign hotkey to switch display machine list, due to following flow. Hotkey assigns to specific command, command then executes code. With new terminal item this is easy to implement. There is a boolean flag which just switch displaying of elements. But with machine list for the exec commands its a bit tricky. Tricky in that way, that we first call run task menu item, task list may contains of not only che task type, but shell and exec. Then, when user pick che task, we show machine list. We can assign hotkey to show the list of machines and switch display, but we need to update the whole mechanism of displaying task list. Because theoretical, user can call hotkey from any place and machine list will be showing. I think it will be not obvious for the end user. So I propose a bit different approach. At first step show to user non-tooling containers and display the option to show all containers. |
@l0rd @vzhukovskii @vparfonov this is not part of the endgame anymore right ? |
@sunix this issue is not mentioned in endgame plan |
@vzhukovskii : How the issue is related to hotkeys? I think you are mis-interpreting the description. Also, I'd like to understand why this issue has been added to the sprint, while it's not part of the endgame and we have a busy list of other issues to address... |
and use a similar mechanism than for opening a terminal In opening terminal when user opens machine picker he can see only user runtimes and after pressing the hotkey, he can observe all containers. So I've tried to align the same mechanism and for the task execution part.
It's not a problem to update the behavior. Should we display the only user's runtimes and not display plugin containers? |
@slemeur Please take a look |
Good for me! |
Description
When running a tasks, the user is asked on which container to run the tasks while he has not idea of what are all the containers (especially those for plugins).
We should currate the list of containers where to execute the tasks to only the user's runtime ones and use a similar mechanism than for opening a terminal.
Update: behavior should be the same when select container dialog appears.
The text was updated successfully, but these errors were encountered: