Skip to content

feat(ui): consolidate History, Shot Analyzer, and Statistics under a unified Analysis hub#770

Open
Serokz wants to merge 4 commits into
jniebuhr:masterfrom
Serokz:pr-analysis-hub
Open

feat(ui): consolidate History, Shot Analyzer, and Statistics under a unified Analysis hub#770
Serokz wants to merge 4 commits into
jniebuhr:masterfrom
Serokz:pr-analysis-hub

Conversation

@Serokz

@Serokz Serokz commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR groups Shot History, Shot Analyzer, and Statistics under a single, unified Analysis page with lazy-loaded tabs. This consolidation simplifies the sidebar navigation and introduces client-side caching to keep history data persistent between tab switches.

As with settings, the old standalone pages and routes redirect to the new tabs to ensure backward compatibility during transitions.

What's Changed

  • Unified Analysis Container (web/src/pages/Analysis/): Created a tabbed container that loads History, Analyzer, and Statistics.
  • Interactive Destructive Controls: Added a vibrating, pointer-based HoldToConfirmButton to replace standard confirm dialogs for critical actions.
  • History Cache Service (ShotHistoryCache.js): Implemented in-memory history caching so switching tabs does not trigger redundant network requests.
  • UX Improvements: Improved list layouts with collapsible accordions, overflow menus, and skeletal loading states.

This is part of a series of independent, story-driven PRs split from the larger frontend modernization effort.

@cla-bot cla-bot Bot added the cla-signed label Jun 14, 2026
@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@Serokz, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 6 minutes and 18 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dd56f546-19d6-472d-80d0-6c6b80491e30

📥 Commits

Reviewing files that changed from the base of the PR and between 591abe7 and 81fd362.

📒 Files selected for processing (33)
  • web/src/components/GmLogoIcon.jsx
  • web/src/components/HoldToConfirmButton.jsx
  • web/src/components/Navigation.jsx
  • web/src/components/PageHeader.jsx
  • web/src/components/PageLayout.jsx
  • web/src/components/ProgressiveContent.jsx
  • web/src/components/Skeletons.jsx
  • web/src/components/TabBar.jsx
  • web/src/components/skeletons/AnalysisSkeleton.jsx
  • web/src/hooks/useIntersectionObserver.js
  • web/src/index.jsx
  • web/src/pages/Analysis/index.jsx
  • web/src/pages/ShotAnalyzer/components/AnalysisTable.jsx
  • web/src/pages/ShotAnalyzer/components/ColumnControls.jsx
  • web/src/pages/ShotAnalyzer/components/EmptyState.jsx
  • web/src/pages/ShotAnalyzer/components/LibraryPanel.jsx
  • web/src/pages/ShotAnalyzer/components/LibraryRow.jsx
  • web/src/pages/ShotAnalyzer/components/LibrarySection.jsx
  • web/src/pages/ShotAnalyzer/components/NotesBar.jsx
  • web/src/pages/ShotAnalyzer/components/SourceMarker.jsx
  • web/src/pages/ShotAnalyzer/components/StatusBar.jsx
  • web/src/pages/ShotAnalyzer/components/shotChart/ShotChartControls.jsx
  • web/src/pages/ShotAnalyzer/index.jsx
  • web/src/pages/ShotHistory/HistoryCard.jsx
  • web/src/pages/ShotHistory/index.jsx
  • web/src/pages/Statistics/components/MetricsTable.jsx
  • web/src/pages/Statistics/components/StatisticsShotCompareSection.jsx
  • web/src/pages/Statistics/components/StatisticsToolbar.jsx
  • web/src/pages/Statistics/components/StatisticsView.jsx
  • web/src/pages/Statistics/components/SummaryCards.jsx
  • web/src/pages/Statistics/components/statisticsDropdownSurface.js
  • web/src/pages/Statistics/index.jsx
  • web/src/services/ShotHistoryCache.js
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
9.3% Duplication on New Code (required ≤ 4%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant