Skip to content

refactor: move summary section to the top#165

Merged
wfelliss merged 2 commits into
mainfrom
130-refactor-summary-section-refactor
May 16, 2026
Merged

refactor: move summary section to the top#165
wfelliss merged 2 commits into
mainfrom
130-refactor-summary-section-refactor

Conversation

@wfelliss

Copy link
Copy Markdown
Owner

feat: add recommendations section to inform the user of the tooltips

feat: add recommendations section to inform the user of the tooltips
@wfelliss wfelliss linked an issue May 16, 2026 that may be closed by this pull request

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a recommendation system for suspension settings, providing suggestions for fork and shock pressure and volume spacers based on sag and travel metrics. The UI layout in MainContent was also reorganized to prioritize the summary section. Review feedback suggests addressing a React key anti-pattern and inconsistent styling in the recommendation list, as well as optimizing performance by lifting redundant metric calculations to a higher-level component to avoid duplicate processing.

Comment thread apps/frontend/app/components/runs/summary-section.tsx Outdated
Comment thread apps/frontend/app/components/runs/summary-section.tsx Outdated
@wfelliss
wfelliss merged commit c33b8f5 into main May 16, 2026
2 checks passed
@wfelliss
wfelliss deleted the 130-refactor-summary-section-refactor branch May 16, 2026 09:38
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.

[REFACTOR] Summary section refactor

1 participant