-
Notifications
You must be signed in to change notification settings - Fork 393
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(frontend): Add missing loading state for non-runnable components … #3797
Conversation
…+ fix select and aggrid styling
Deploying windmill with Cloudflare Pages
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❌ Changes requested. Reviewed everything up to e859e68 in 1 minute and 39 seconds
More details
- Looked at
207
lines of code in7
files - Skipped
0
files when reviewing. - Skipped posting
0
drafted comments based on config settings.
Workflow ID: wflow_Agxbk1dbPoCejyin
Want Ellipsis to fix these issues? Tag @ellipsis-dev
in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
<div class={'bg-blue-100 dark:bg-blue-400 transition-all p-1 rounded-component'}> | ||
<LoaderIcon size={14} class="animate-spin text-blue-800 dark:text-white" /> | ||
</div> | ||
<svelte:fragment slot="text"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The slot for text in the Popover component should be named 'content' to match the expected API of the Popover component. This ensures that the text ('Refreshing...' or 'Refresh') is displayed correctly.
<svelte:fragment slot="text"> | |
<svelte:fragment slot="content"> |
…+ fix select and aggrid styling
Summary:
This PR enhances UI responsiveness and styling across various frontend components, particularly focusing on loading states and visual consistency in dark mode.
Key points:
NonRunnableComponent.svelte
.RefreshIndicator.svelte
for visual loading indication.AppSelect.svelte
to handle dark mode border color.windmill-theme.css
.rx.ts
to include unsubscribe functionality.Generated with ❤️ by ellipsis.dev