Skip to content

Add dynamic loading view when using CopilotUserInput#1027

Merged
motm32 merged 9 commits intomainfrom
meganmott/dynamicLoadingView
Apr 20, 2026
Merged

Add dynamic loading view when using CopilotUserInput#1027
motm32 merged 9 commits intomainfrom
meganmott/dynamicLoadingView

Conversation

@motm32
Copy link
Copy Markdown
Contributor

@motm32 motm32 commented Feb 10, 2026

Here is a video on how this currently works:

dynamicLoadingView.mp4

To do:

  • Add in subscription, container app environment and container app to the view (needs to be tested with the re run work)

@motm32 motm32 requested a review from a team as a code owner February 10, 2026 21:47
@alexweininger
Copy link
Copy Markdown
Member

Great work, my nitpick UI feedback is that the checklist should also be centered underneath the Generating Copilot Response heading.

And then a potential bigger improvement would be to have the items be more descriptive, rather than a single word like "Registry"

@MicroFish91
Copy link
Copy Markdown
Contributor

MicroFish91 commented Feb 11, 2026

This looks really slick - my only nit is maybe move the "Generating..." message and step progress list vertically up a little bit more

@motm32
Copy link
Copy Markdown
Contributor Author

motm32 commented Feb 19, 2026

I made some changes to the view so it now has a placeholder for when no items have been selected which looks like this:
image
After copilot has picked items they will populate below which looks like this:
image
Let me know if there is anything else we shoudl change about the UI.

@motm32
Copy link
Copy Markdown
Contributor Author

motm32 commented Feb 24, 2026

I need to release this tools PR for the tests to pass btw.

Copy link
Copy Markdown
Member

@alexweininger alexweininger left a comment

Choose a reason for hiding this comment

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

Two small comments to apply to a bunch of the code

Comment thread src/commands/createContainerApp/ContainerAppListStep.ts Outdated
Comment thread src/commands/createContainerApp/ContainerAppListStep.ts Outdated
Comment thread src/webviews/LoadingView.tsx
Comment thread src/webviews/LoadingView.tsx Outdated
Comment thread src/webviews/LoadingView.tsx Outdated
Comment thread src/webviews/LoadingView.tsx Outdated
Comment thread src/webviews/LoadingView.tsx
@motm32 motm32 merged commit 0252afc into main Apr 20, 2026
2 checks passed
@motm32 motm32 deleted the meganmott/dynamicLoadingView branch April 20, 2026 20:31
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.

4 participants