Skip to content
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

'Open Terminal in specific container' usability #14987

Closed
RomanNikitenko opened this issue Oct 24, 2019 · 4 comments
Closed

'Open Terminal in specific container' usability #14987

RomanNikitenko opened this issue Oct 24, 2019 · 4 comments
Assignees
Labels
area/editor/theia Issues related to the che-theia IDE of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.
Milestone

Comments

@RomanNikitenko
Copy link
Member

Summary

Open Terminal in specific container action displays empty list of containers for case when user
has only tool containers.

containers_list

Please see https://youtu.be/khkxUxTVgH0

Is it expected behavior?
May be for this case list of tool containers should be displayed without empty dropdown list and additional actions from user's side.

Relevant information

Example of Devfile
apiVersion: 1.0.0
metadata:
  name: test-terminals
projects:
  - name: che-theia
    source:
      type: git
      location: 'https://github.com/eclipse/che-theia.git'
components:
  - reference: >-
      https://raw.githubusercontent.com/eclipse/che-plugin-registry/master/v3/plugins/eclipse/che-theia/next/meta.yaml
    type: cheEditor
    alias: theia-editor
commands:
  - name: test task
    actions:
      - workdir: /projects
        type: exec
        command: sleep 30 && echo "hello world"
        component: theia-editor

@RomanNikitenko RomanNikitenko added the kind/question Questions that haven't been identified as being feature requests or bugs. label Oct 24, 2019
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Oct 24, 2019
@azatsarynnyy
Copy link
Member

I don't think that hiding some part of the whole list is a good UX. I'd agree if it was a huge list of containers. But usually, it isn't. There're just several items.

Occasionally I see the questions, on Che MM channel, that there's no sidecar container in the list. Maybe it isn't obvious that one should hit the hotkey twice. I think it's an unnecessary complication today.

As far as I remember it was applied when we were not able to use a separator in Theia.

I propose not to hide the tool containers but use a separator between the two groups. E.g. Runtime containers and Tool containers.
Like we're already using it for the tasks list:

image

I believe it would be much simpler both from the UX point of view and impl.

@azatsarynnyy
Copy link
Member

azatsarynnyy commented Oct 25, 2019

@slemeur WDYT about dividing the list to two groups with a separator instead of hiding a part of the list? Wouldn't it be simpler and clearer for the user?

@tsmaeder
Copy link
Contributor

Maybe it isn't obvious that one should hit the hotkey twice. I think it's an unnecessary complication today.

It most certainly is not! However, the terminals can be opened from the "My Workspace" view. P2

@tsmaeder tsmaeder added kind/bug Outline of a bug - must adhere to the bug report template. team/ide area/editor/theia Issues related to the che-theia IDE of Che severity/P2 Has a minor but important impact to the usage or development of the system. and removed kind/question Questions that haven't been identified as being feature requests or bugs. status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Oct 28, 2019
@vparfonov vparfonov added this to the Backlog - IDE 1 milestone Oct 29, 2019
@vparfonov vparfonov modified the milestones: Backlog - IDE 1, 7.4.0, 7.5.0 Nov 11, 2019
@vzhukovs
Copy link
Contributor

Fixed with #13390

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/editor/theia Issues related to the che-theia IDE of Che kind/bug Outline of a bug - must adhere to the bug report template. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Development

No branches or pull requests

6 participants