Skip to content

Conversation

@adity1raut
Copy link
Contributor

Description

This feature update modifies the Staged and Deployed Workloads UI component so that it expands to occupy the full right-hand side of the user interface. The goal is to improve visibility, enhance layout usability, and provide a cleaner presentation of workload details.

Related Issue

Fixes #2231

Changes Made

  • Updated ...
  • Refactored ...
  • Fixed ...
  • Added tests for ...

Checklist

Please ensure the following before submitting your PR:

  • I have reviewed the project's contribution guidelines.
  • I have written unit tests for the changes (if applicable).
  • I have updated the documentation (if applicable).
  • I have tested the changes locally and ensured they work as expected.
  • My code follows the project's coding standards.

Screenshots or Logs (if applicable)

Screencast.From.2025-11-19.16-08-00.webm

Additional Notes

Signed-off-by: adity1raut <[email protected]>
@kubestellar-prow
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign btwshivam for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubestellar-prow kubestellar-prow bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Nov 19, 2025
@kubestellar-prow
Copy link

Hi @adity1raut. Thanks for your PR.

I'm waiting for a kubestellar member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@adity1raut
Copy link
Contributor Author

/cc @clubanderson

@clubanderson
Copy link
Contributor

Can you provide a video so I can see what has changed in both tree components?

@khushiiagrawal
Copy link
Contributor

Hi @adity1raut,
I wanted to let you know that I am already working on the control bar placement on the right side (issue #2248).
Could you please take a quick look? I’m planning to raise a PR soon, so I want to make sure there’s no overlap.
@clubanderson please take a look.
thank you

@adity1raut
Copy link
Contributor Author

Hi @adity1raut, I wanted to let you know that I am already working on the control bar placement on the right side (issue #2248). Could you please take a quick look? I’m planning to raise a PR soon, so I want to make sure there’s no overlap. @clubanderson please take a look. thank you

@khushiiagrawal You Can Check #2231

@adity1raut
Copy link
Contributor Author

Befoure*

Screencast.From.2025-11-19.19-52-26.webm

After

Screencast.From.2025-11-19.16-08-00.webm

@adity1raut
Copy link
Contributor Author

PTAL @clubanderson

@clubanderson
Copy link
Contributor

@adity1raut I think @khushiiagrawal has the right solution for the UI. The control panel and the workload creation and view items on on the right side and the tree view has the entire right-side for view and control.

@clubanderson
Copy link
Contributor

@adity1raut can you test this PR - that would help us.

@khushiiagrawal looks good to me - lets get @adity1raut and @btwshivam to test.

@clubanderson clubanderson requested review from btwshivam and removed request for clubanderson November 19, 2025 14:25
@clubanderson
Copy link
Contributor

looks good to me... need others to review

@khushiiagrawal
Copy link
Contributor

Hi @adity1raut, I wanted to let you know that I am already working on the control bar placement on the right side (issue #2248). Could you please take a quick look? I’m planning to raise a PR soon, so I want to make sure there’s no overlap. @clubanderson please take a look. thank you

@khushiiagrawal You Can Check #2231

@adity1raut , the issue was regarding the staged and deployed workloads, the issue in which i am working is regarding the control panel
i guess @clubanderson also asked you to fix the control panel that's why you did it but there is a seperate issue opened for that so i'll raising PR to fix the control panel
@adity1raut you can work on staged and deployed workloads, i'll be working on the control panel part, works ?

@clubanderson
Copy link
Contributor

thats a good way to split the work - please proceed!

@khushiiagrawal
Copy link
Contributor

thats a good way to split the work - please proceed!

Thanks! Proceeding with the PR.

@adity1raut
Copy link
Contributor Author

That issue has already been solved, @clubanderson . What is the need to open another PR for this?
Why do you want another PR, @khushiiagrawal ?

@clubanderson
Copy link
Contributor

@adity1raut and @khushiiagrawal please work together to determine how to split or combine this work.

@adity1raut
Copy link
Contributor Author

Screencast.From.2025-11-21.14-24-59.webm

PTAL @clubanderson Everything is okk

@adity1raut
Copy link
Contributor Author

/cc @MAVRICK-1 @kunal-511

@clubanderson
Copy link
Contributor

@MAVRICK-1 @kunal-511 is this ready?

@Arpit529Srivastava
Copy link
Member

/ok-to-test

@clubanderson
Copy link
Contributor

@btwshivam please advise

@btwshivam btwshivam closed this Dec 6, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in KubeStellar UI Project Dec 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[FEATURE]: make the staged and deployed workloads ui component the entire right side of the ui

5 participants