feat: Add loading spinners to refresh buttons and improve Logger UX #27
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add spinner animation to refresh buttons in BaseCard and all admin views - Swap button order in BaseCard footer (Details left, Refresh right) - Use isFetching in AutomaticGroupsAdmin for better loading state detection - Consistent spinner styling across all components Co-authored-by: Ona <no-reply@ona.com>
- Add refresh button with loading spinner to LoggerSummaryAdminBulk - Add spinner to LoggerSummaryAdmin for consistency - Fix button styling to match other admin panels - Use isFetching for better loading state detection in bulk cache - Change default days filter from 3 to 1 in bulk view Co-authored-by: Ona <no-reply@ona.com>
- Remove LoggerSummaryAdmin.vue (legacy version, not used in production) - Remove unused import from App.vue - LoggerSummaryAdminBulk is the active version with better caching - Keep useLoggerSummary.ts as it exports types used by useLoggerBulkCache Co-authored-by: Ona <no-reply@ona.com>
- Rename LoggerSummaryAdminBulk.vue to LoggerSummaryAdmin.vue - Update imports in App.vue - Remove '(Bulk Cache)' from component title - Now that legacy version is removed, bulk version becomes the standard Co-authored-by: Ona <no-reply@ona.com>
- Fix table title to show correct filtered count instead of paginated count - Add dynamic title showing active filters (days, category) - Remove custom pagination in favor of AdminTable's built-in features - Enable AdminTable search to work across all filtered logs - Change both Card and Admin to use 1 day as default - Increase Card log limit from 1000 to 1500 entries - Show '1500+' in Card when log limit is reached - Remove duplicate search field issue - Fix goToPage undefined error Co-authored-by: Ona <no-reply@ona.com>
- Document all phases of implementation - Record technical decisions and rationale - Capture lessons learned for future reference - Include package.json version bump Co-authored-by: Ona <no-reply@ona.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds loading spinners to all refresh buttons across the dashboard and significantly improves the Logger admin panel user experience.
Changes
Refresh Button Spinners
isFetchingfor manual refreshesLogger Component Cleanup
LoggerSummaryAdmin.vue(806 lines, unused)LoggerSummaryAdminBulk.vuetoLoggerSummaryAdmin.vueLogger Admin Panel UX Improvements
Technical Details
Loading State Management
isLoadingandisFetchingfrom TanStack QueryPagination Simplification
Consistency Improvements
Testing
npm run lintDocumentation
docs/DEVELOPMENT_SESSION_2025-11-19_Refresh_Button_Spinner.mdCommits
Co-authored-by: Ona no-reply@ona.com