Skip to content

Make changes for accessibility and add loading view #1494

Open
motm32 wants to merge 6 commits into
feat/CoRfrom
meganmott/webviewFixes
Open

Make changes for accessibility and add loading view #1494
motm32 wants to merge 6 commits into
feat/CoRfrom
meganmott/webviewFixes

Conversation

@motm32

@motm32 motm32 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

I made some accesibility changes so when zoomed in at 200% the view is able to be scrolled and the words don't go over or get hidden.

I also added a loading view so there is always a view open. Here is an example of how the loading view looks (the wording changes depending on which stage you are in):

image

Finally changed the input box so it is closed it now looks like this:
image

@motm32 motm32 requested a review from a team as a code owner June 15, 2026 17:04
void this.panel.webview.postMessage({ command: 'submitComplete' });

const relativePath = vscode.workspace.asRelativePath(this.sourceFileUri);
this.panel.dispose();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Any idea why this moved up?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Because we need to dispose of the requirements view panel before opening the loading view.

Megan Mott added 3 commits June 16, 2026 13:36
…esourcegroups into meganmott/webviewFixes

# Conflicts:
#	src/webviews/copilotOnRails/extension/controllers/LocalPlanViewController.ts
#	src/webviews/copilotOnRails/extension/controllers/RequirementsViewController.ts
#	src/webviews/copilotOnRails/extension/controllers/ScaffoldPlanViewController.ts
#	src/webviews/copilotOnRails/views/styles/uiPreviewCard.scss
@motm32

motm32 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Still need to remove the debugging commands

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.

2 participants