Skip to content

[MAIN-IMP] Name and logo update on the sidebar#80

Merged
moda20 merged 1 commit intomasterfrom
Improvement/name-change-update
Mar 29, 2026
Merged

[MAIN-IMP] Name and logo update on the sidebar#80
moda20 merged 1 commit intomasterfrom
Improvement/name-change-update

Conversation

@moda20
Copy link
Copy Markdown
Owner

@moda20 moda20 commented Mar 28, 2026

Improvements :

  • Update the sidebar name with "TypeScheduler"
  • Update the logo to a simplified version

Summary by CodeRabbit

  • UI Updates
    • Updated the version switcher component with the TypeScheduler branding and logo. Replaced the icon with the TypeScheduler logo and adjusted layout spacing for improved visual presentation.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 28, 2026

📝 Walkthrough

Walkthrough

Updated the version switcher component to replace the built-in list icon with a simplified TypeScheduler logo image asset, adjust spacing with a gap-4 utility class, and change the display label from "Scheduler" to "TypeScheduler".

Changes

Cohort / File(s) Summary
Version Switcher UI Update
src/components/version-switcher.tsx
Added gap-4 spacing utility to SidebarMenuButton. Replaced ListBulletIcon element with an img-based element displaying TypeScheduler logo (TypeSchedulerLogoSimplified.png), increased size from size-8 to size-12, added border-primary border-2 and bg-white styling. Updated label text from "Scheduler" to "TypeScheduler".

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 The version switcher now wears a new face,
With TypeScheduler's logo in pride of place,
A bordered gem on white so bright,
The sidebar buttons spaced just right! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes in the PR: updating both the name from 'Scheduler' to 'TypeScheduler' and replacing the logo with a simplified version in the sidebar component.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch Improvement/name-change-update

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.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/components/version-switcher.tsx`:
- Around line 26-29: The <img> in the VersionSwitcher component is missing an
alt attribute which breaks accessibility; update the <img> element in
src/components/version-switcher.tsx (inside the VersionSwitcher component) to
include a meaningful alt text like "TypeScheduler logo" or, if the image is
purely decorative, an empty alt="" to mark it as decorative so screen readers
ignore it. Ensure the alt attribute is present on the <img> with
className="size-9".
- Around line 26-29: The img src uses a filesystem path that will break in
production; import the image file as a module at the top of the component (e.g.,
import TypeSchedulerLogo from "@/assets/images/TypeSchedulerLogoSimplified.png")
and then update the <img> in the VersionSwitcher component to use the imported
TypeSchedulerLogo variable as its src (and keep the existing className); remove
the hardcoded "src/assets/..." string so the bundler can resolve the asset.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 0565b19a-aac7-47db-9f6d-5e0b0a246a48

📥 Commits

Reviewing files that changed from the base of the PR and between 59409e5 and 701b778.

⛔ Files ignored due to path filters (2)
  • src/assets/images/TypeSchedulerLogo.png is excluded by !**/*.png
  • src/assets/images/TypeSchedulerLogoSimplified.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • src/components/version-switcher.tsx

@moda20 moda20 merged commit 74a5ab9 into master Mar 29, 2026
1 check passed
@moda20 moda20 deleted the Improvement/name-change-update branch March 29, 2026 12:25
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.

1 participant