-
Notifications
You must be signed in to change notification settings - Fork 715
fix: added loader for search buttons in disable state #9558
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
Conversation
|
Failed to generate code suggestions for PR |
Greptile OverviewGreptile SummaryAdded loading spinner visual feedback to run query and refresh buttons across logs, metrics, traces, and dashboards by binding the
All changes reuse existing loading state variables that were already controlling the Confidence Score: 5/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant User
participant Button as Run Query Button
participant State as Loading State
participant API as Backend API
User->>Button: Click Run Query
Button->>State: Set loading = true
Note over Button: Button shows spinner<br/>and becomes disabled
State->>API: Execute query
API-->>State: Return results
State->>Button: Set loading = false
Note over Button: Spinner disappears<br/>button becomes enabled
Button-->>User: Display results
|
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.
3 files reviewed, no comments
66d52fa to
a9f9ade
Compare
a9f9ade to
1ce2162
Compare
No description provided.