fix: schedule ui gap

[amhsirak]

Summary by CodeRabbit

• Style
  • Refined spacing on the Schedule Settings page for a cleaner, more consistent layout.
  • Content is shifted slightly upward to make better use of screen space and reduce unnecessary scrolling.
  • Improved visual alignment and tighter spacing between sections for better readability.
  • No functional changes; this is a visual improvement only.

[coderabbitai]

Walkthrough

Updated ScheduleSettingsPage layout: replaced Box padding with a negative top margin to shift content upward and remove internal padding.

Changes

Cohort / File(s)	Summary of edits
UI layout tweak src/components/robot/pages/ScheduleSettingsPage.tsx	Changed Box style from padding: "20px" to marginTop: "-20px" to adjust vertical positioning and remove internal padding.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

Type: Bug, Scope: UI/UX

Poem

I nudged the page a whisker’s height,
A minus hop—now things feel light.
No padding clouds, just skies of space,
The schedule smiles in tidy place.
Boop! goes the UI—hop, hop, right! 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title clearly conveys the primary change of addressing a user interface gap in the schedule settings, matching the styling adjustment in the code where padding was replaced by a negative margin to shift content. It is concise, specific to the schedule UI, and indicates a bug fix without unnecessary detail.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch settings-ui

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (3)

src/components/robot/pages/ScheduleSettingsPage.tsx (3)

203-205: Remove redundant ternary operation.

Both branches of the ternary operator return the same translation key. This can be simplified to a direct assignment.

Apply this diff:

-                {["MONTHS", "WEEKS"].includes(settings.runEveryUnit)
-                  ? t("schedule_settings.start_from")
-                  : t("schedule_settings.start_from")}
+                {t("schedule_settings.start_from")}

---

267-269: Remove redundant ternary operation.

Similar to the display section above, both branches return the same translation key.

Apply this diff:

-                  {["MONTHS", "WEEKS"].includes(settings.runEveryUnit)
-                    ? t("schedule_settings.labels.start_from_label")
-                    : t("schedule_settings.labels.start_from_label")}
+                  {t("schedule_settings.labels.start_from_label")}

---

186-194: Override RobotConfigPage content margin instead of negative margin hack
The <Box> in ScheduleSettingsPage (lines 186–194) uses marginTop: "-20px" to cancel the mt:2 on RobotConfigPage’s content wrapper (RobotConfigPage.tsx:122–127). Instead of this workaround:

• extend RobotConfigPage with a contentSx prop for its content Box
• pass contentSx={{ mt: 0 }} in ScheduleSettingsPage
• remove the marginTop: "-20px" here

This removes the hack and keeps spacing predictable.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 95a4d3c and 9e06d14.

📒 Files selected for processing (1)

• src/components/robot/pages/ScheduleSettingsPage.tsx (1 hunks)