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

Redesign of the MiniStatus window #126

Closed
gbraad opened this issue Feb 1, 2022 · 5 comments · Fixed by #137
Closed

Redesign of the MiniStatus window #126

gbraad opened this issue Feb 1, 2022 · 5 comments · Fixed by #137
Assignees
Labels
enhancement New feature or request

Comments

@gbraad
Copy link
Collaborator

gbraad commented Feb 1, 2022

A possible design could be
image

where the Pause/Play button toggles between the states that the instance is in.

Related to: crc-org/crc-react-components#29

@gbraad gbraad added the enhancement New feature or request label Feb 1, 2022
@gbraad gbraad self-assigned this Feb 1, 2022
@deboer-tim
Copy link
Contributor

Having play & pause on the same button is confusing to me, e.g. which is it going to do? One option to fix this would be to alternate the icon based on the state, but then it has to be really clear what the current state is. It's clear when YouTube is playing because there is movement and sound and it immediately toggles between states, but with a VM the fan noise is a bit delayed and there are delays and unknown states. :) IMHO we're going to need a clear and separate current state (e.g. "Starting" label) beside the action.

IMHO Delete is also potentially confusing for new users, especially if you hadn't made the connection to a VM yet. You don't know if this is deleting the install, whether it is recoverable, how would I create a new one, etc. In this case I think there just needs to be a really clear confirmation dialog.

@gbraad
Copy link
Collaborator Author

gbraad commented Feb 5, 2022

Having play & pause on the same button is confusing to me

where the Pause/Play button toggles between the states that the instance is in (this means the icon changes according to the state of the VM).

image


  • The instance state is stopped or unknown:
    image

  • the instance state is starting:
    image

  • the instances state is running:
    image

@gbraad
Copy link
Collaborator Author

gbraad commented Feb 5, 2022

I think there just needs to be a really clear confirmation dialog.

I also suggest you to look at the doc that is shared between me, Kevin, and Beau of UX, as this have been addressed and discussed there.

@gbraad
Copy link
Collaborator Author

gbraad commented Feb 7, 2022

Would like to display memory, cpu and disk usage in 'real-time'.

For which

import {
    OutlinedHddIcon,
    MemoryIcon,
    MicrochipIcon
} from '@patternfly/react-icons';

can be used. Originally discussed as part of crc-org/crc-react-components#5

@gbraad
Copy link
Collaborator Author

gbraad commented Feb 9, 2022

  • when stopped/stopping
    image

  • when starting
    image

  • when running
    image

gbraad added a commit to gbraad-redhat/crc-tray-electron that referenced this issue Feb 9, 2022
@gbraad gbraad linked a pull request Feb 9, 2022 that will close this issue
gbraad added a commit to gbraad-redhat/crc-tray-electron that referenced this issue Feb 9, 2022
gbraad added a commit to gbraad-redhat/crc-tray-electron that referenced this issue Feb 9, 2022
@gbraad gbraad moved this from In Progress to Ready for review in Project planning Feb 9, 2022
gbraad added a commit to gbraad-redhat/crc-tray-electron that referenced this issue Feb 9, 2022
gbraad added a commit to gbraad-redhat/crc-tray-electron that referenced this issue Feb 9, 2022
gbraad added a commit that referenced this issue Feb 10, 2022
Repository owner moved this from Ready for review to Done in Project planning Feb 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants