Skip to content

Fix resource manifest modal loading state#889

Open
anonymusk7 wants to merge 1 commit into
cyclops-ui:mainfrom
anonymusk7:codex/resource-manifest-modal-loading
Open

Fix resource manifest modal loading state#889
anonymusk7 wants to merge 1 commit into
cyclops-ui:mainfrom
anonymusk7:codex/resource-manifest-modal-loading

Conversation

@anonymusk7
Copy link
Copy Markdown

Summary

  • add a loading state for resource manifest fetches
  • render resource manifests with the same Ace-based layout/options used by the module/rendered manifest modals
  • remove the resource manifest modal OK footer so it closes via the modal close/cancel affordance

Closes #641.

Validation

  • npx --yes prettier@3.2.5 --check cyclops-ui/src/components/k8s-resources/ResourceList/ResourceList.tsx
  • git diff --check

Note

I attempted yarn --cwd cyclops-ui install --frozen-lockfile --ignore-scripts twice, including with --network-timeout 600000, but dependency fetching stalled in this environment before node_modules was created. I did not claim a full local build.

@anonymusk7 anonymusk7 requested a review from a team as a code owner May 12, 2026 23:07
@anonymusk7
Copy link
Copy Markdown
Author

@quest-bot loot #641

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.

Manifests modals to use the same theme

1 participant