Skip to content

Conversation

adietish
Copy link
Collaborator

@adietish adietish commented Sep 18, 2025

depends on #171

@codecov-commenter
Copy link

codecov-commenter commented Sep 18, 2025

Codecov Report

❌ Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (71098f6) to head (55a372c).
⚠️ Report is 217 commits behind head on main.

Files with missing lines Patch % Lines
.../gateway/view/steps/DevSpacesWorkspacesStepView.kt 0.00% 9 Missing ⚠️
...tlin/com/redhat/devtools/gateway/DevSpacesIcons.kt 0.00% 7 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##            main    #178    +/-   ##
======================================
  Coverage   0.00%   0.00%            
======================================
  Files          4      29    +25     
  Lines         26     952   +926     
  Branches       0     151   +151     
======================================
- Misses        26     952   +926     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@adietish
Copy link
Collaborator Author

adietish commented Sep 18, 2025

this PR is replacing phase text [Running] or [Stopped] by icons when listing the workspaces
image
Here's how things are displayed without this patch:
image

The icons match the web UI:
image

@adietish adietish force-pushed the workspace-phase-icons branch from 683d147 to 5b4e1e6 Compare September 18, 2025 17:04
@adietish adietish moved this to Unplanned Tasks in Eclipse Che Team C Backlog Sep 18, 2025
@adietish adietish force-pushed the workspace-phase-icons branch 3 times, most recently from 8ef8ce7 to a1ef372 Compare September 19, 2025 21:15
@adietish adietish marked this pull request as ready for review September 19, 2025 21:16
Copy link
Member

@azatsarynnyy azatsarynnyy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice UI improvements!

@adietish adietish force-pushed the workspace-phase-icons branch from a1ef372 to 55a372c Compare September 22, 2025 14:19
@adietish adietish merged commit 7a3d610 into redhat-developer:main Sep 22, 2025
5 checks passed
@adietish adietish deleted the workspace-phase-icons branch September 22, 2025 14:31
@adietish adietish moved this from Unplanned Tasks to ✅ Done in Eclipse Che Team C Backlog Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants