feat: lifestyle tabs ui match by tg14102005-a11y · Pull Request #95 · ColmanDevClubORG/Sagol360Management

tg14102005-a11y · 2026-06-24T20:27:27Z

Description

Match Lifestyle page UI to design mockups.

Changes:

Added SGLSwitch tabs (תזונה / פיזיותרפיה ואימונים)
Added SGLBrainHQ card above tabs
Added translation keys for all 4 languages
Created styles.ts for LifeStyle page

Related Issue(s)

Fixes # (issue number)

Checklist:

[ v] I have performed a self-review of my own code
[v ] My changes generate no new warnings

Screenshots (if appropriate):

Summary by CodeRabbit

New Features
- Added a tabbed lifestyle screen that lets users switch between diet/nutrition and physiotherapy/training.
- Displays diet content on the diet tab and physiotherapy/training content on the physio tab.
Localization
- Added new lifestyle tab labels for Arabic, English, Hebrew, and Russian (“Diet” and “Physiotherapy & Training”).
UI Improvements
- Updated the lifestyle page layout and styling to better organize the top controls and content area.

coderabbitai · 2026-06-24T20:27:40Z

Warning

Review limit reached

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

More reviews will be available in 45 minutes and 6 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.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5313d653-471e-4d26-bbef-6d45a0076b54

📥 Commits

Reviewing files that changed from the base of the PR and between e0cca6f and d8c7af6.

📒 Files selected for processing (4)

src/locales/ar/translation.json
src/locales/en/translation.json
src/locales/he/translation.json
src/locales/ru/translation.json

📝 Walkthrough

Walkthrough

Adds localized lifestyle tab labels in four languages and changes the LifeStyle page to switch between diet and physio/training content with a tab control.

Changes

Lifestyle tab navigation

Layer / File(s)	Summary
Lifestyle tab translations `src/locales/ar/translation.json`, `src/locales/en/translation.json`, `src/locales/he/translation.json`, `src/locales/ru/translation.json`	Adds `lifestyle.tab.diet` and `lifestyle.tab.physio` entries to the Arabic, English, Hebrew, and Russian locale files.
Tabbed lifestyle page `src/pages/lifeStyle/LifeStyle.tsx`, `src/pages/lifeStyle/constants.ts`, `src/pages/lifeStyle/styles.ts`	Turns `LifeStyle` into a tab-driven page with active tab state, translated switch options, conditional rendering for diet versus physio content, and new layout styles and tab constants.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

ColmanDevClubORG/Sagol360Management#43: Touches src/pages/lifeStyle/LifeStyle.tsx and related i18n usage, which is directly adjacent to the new translated tab labels and page wiring.
ColmanDevClubORG/Sagol360Management#46: Introduces SGLSwitch, which is used by the new tab-driven LifeStyle page.
ColmanDevClubORG/Sagol360Management#47: Adds BalanceTraining, which is conditionally rendered by the new physio tab flow.

Suggested reviewers

Tamir198
GilHeller

Poem

I hopped through tabs with joyful grace,
Four tongues of labels found their place.
Diet or physio, one switch to choose,
My bunny feet approve these views.
🐇✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: Lifestyle page tabs/UI updates.
Description check	✅ Passed	The template sections are present and the summary/screenshots are mostly complete, though the related issue and checklist entries are still placeholders.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch lifestyle-ui-match

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.}

Tamir198 · 2026-06-24T21:35:33Z

+import * as styles from './styles'
+
+const TAB_DIET = 'diet'
+const TAB_PHYSIO = 'physio'


I would save those inside constant file

Tamir198 · 2026-06-24T21:36:42Z

-      <RecomendationTraining />
-      <DietTabExample />
-    </>
+    <Box sx={styles.containerStyle}>


why do you need box from mui and not regular div? or even use the existing container component:

src/components/UI/Container/SGLContainer.tsx

Tamir198 · 2026-06-24T21:37:35Z

+      <Box sx={styles.switchWrapperStyle}>
+        <SGLSwitch options={options} value={activeTab} onChange={setActiveTab} />
+      </Box>
+      {activeTab === TAB_DIET && <DietTabExample />}


There are only 2 tabs, so you can do {activeTab === TAB_DIET ? : }

Tamir198 · 2026-06-24T21:37:44Z

@@ -0,0 +1,12 @@
+export const containerStyle = {
+  display: 'flex',
+  flexDirection: 'column' as const,


Why do you need as const in here? you can make this object satisfy CSSProperties instead

pnpm dev pnpm dev '

GilHeller

LGTM, but you have a conflict. Please fix it

tg14102005-a11y · 2026-06-25T18:39:01Z

Fixed!נשלח מה-iPhone שלי‫ב-25 ביוני 2026, בשעה 21:32, ‏‏GilHeller ***@***.***> כתב/ה:‬ @GilHeller commented on this pull request. LGTM, but you have a conflict. Please fix it —Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today! You are receiving this because you authored the thread.Message ID: ***@***.***>

feat: lifestyle tabs ui match

ce1e39a

tg14102005-a11y requested a review from Tamir198 June 24, 2026 20:27

tg14102005-a11y requested a review from GilHeller June 24, 2026 20:28

Tamir198 requested changes Jun 24, 2026

View reviewed changes

fix: apply PR review changes״

e0cca6f

pnpm dev pnpm dev '

tg14102005-a11y requested a review from Tamir198 June 25, 2026 18:25

GilHeller reviewed Jun 25, 2026

View reviewed changes

Merge branch 'main' into lifestyle-ui-match

d8c7af6

tg14102005-a11y requested a review from GilHeller June 25, 2026 18:38

GilHeller approved these changes Jun 25, 2026

View reviewed changes

GilHeller merged commit afcf52e into main Jun 25, 2026
2 checks passed

GilHeller deleted the lifestyle-ui-match branch June 25, 2026 19:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: lifestyle tabs ui match#95

feat: lifestyle tabs ui match#95
GilHeller merged 3 commits into
mainfrom
lifestyle-ui-match

tg14102005-a11y commented Jun 24, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Jun 24, 2026 •

edited

Loading

Review limit reached

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Uh oh!

Tamir198 Jun 24, 2026

Uh oh!

Tamir198 Jun 24, 2026

Uh oh!

Tamir198 Jun 24, 2026

Uh oh!

Tamir198 Jun 24, 2026

Uh oh!

GilHeller left a comment

Uh oh!

tg14102005-a11y commented Jun 25, 2026 via email

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

tg14102005-a11y commented Jun 24, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Related Issue(s)

Checklist:

Screenshots (if appropriate):

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Jun 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review limit reached

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Uh oh!

Tamir198 Jun 24, 2026

Choose a reason for hiding this comment

Uh oh!

Tamir198 Jun 24, 2026

Choose a reason for hiding this comment

Uh oh!

Tamir198 Jun 24, 2026

Choose a reason for hiding this comment

Uh oh!

Tamir198 Jun 24, 2026

Choose a reason for hiding this comment

Uh oh!

GilHeller left a comment

Choose a reason for hiding this comment

Uh oh!

tg14102005-a11y commented Jun 25, 2026 via email

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

tg14102005-a11y commented Jun 24, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jun 24, 2026 •

edited

Loading