Skip to content

Correct mismatched example code in Culture sample of BitCalendar demo (#12274)#12275

Merged
msynk merged 1 commit into
bitfoundation:developfrom
zahra-yousefi:12274-example-code-mismatch-calendar-demo
Apr 25, 2026
Merged

Correct mismatched example code in Culture sample of BitCalendar demo (#12274)#12275
msynk merged 1 commit into
bitfoundation:developfrom
zahra-yousefi:12274-example-code-mismatch-calendar-demo

Conversation

@zahra-yousefi
Copy link
Copy Markdown
Member

@zahra-yousefi zahra-yousefi commented Apr 25, 2026

closes #12274

Summary by CodeRabbit

  • Documentation
    • Updated Calendar component demo samples to reflect the correct parameter usage for the "go to today" feature across culture-specific examples.

@msynk msynk requested a review from Copilot April 25, 2026 18:23
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 278f004f-26dc-45e4-9bef-4c36040436eb

📥 Commits

Reviewing files that changed from the base of the PR and between 7212f65 and 93f7ddd.

📒 Files selected for processing (1)
  • src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Inputs/Calendar/BitCalendarDemo.razor.samples.cs

Walkthrough

Updated demo code samples in the BitCalendar component to use the GoToTodayTitle parameter instead of GoToToday in the culture section example. The change corrects a mismatch between documented example code and the actual implementation.

Changes

Cohort / File(s) Summary
Calendar Demo Sample Code
src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Inputs/Calendar/BitCalendarDemo.razor.samples.cs
Updated example4RazorCode to replace GoToToday parameter with GoToTodayTitle in both culture-specific demo examples, aligning sample markup with current API.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through demo code with glee,
Fixing parameter names, one, two, three!
From GoToToday to GoToTodayTitle true,
The culture examples now shine anew! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Correct mismatched example code in Culture sample of BitCalendar demo' clearly describes the main change—updating example code to match the implementation.
Linked Issues check ✅ Passed The pull request directly addresses issue #12274 by updating the GoToTodayTitle parameter in the example code to match the current implementation.
Out of Scope Changes check ✅ Passed All changes are scoped to the BitCalendar demo sample file, updating only the example code strings to match the implementation as required by issue #12274.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

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

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the BitCalendar demo’s Culture sample to align the displayed example code with the actual component API, addressing the mismatch reported in #12274.

Changes:

  • Replaces the outdated GoToToday attribute in example #4’s code sample with the correct GoToTodayTitle attribute.

@msynk msynk merged commit 4b1a75e into bitfoundation:develop Apr 25, 2026
7 checks passed
@zahra-yousefi zahra-yousefi deleted the 12274-example-code-mismatch-calendar-demo branch April 25, 2026 18:39
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.

Example code mismatch in the Culture section of the BitCalendar demo page

3 participants