Conversation
WalkthroughReworked header construction and color resolution in the calendar widget: introduced BoxDecoration-based header, explicit default chevron icons, standardized padding fallbacks, and replaced payload.getColor with payload.evalColor across header, days-of-week, and calendar styling (including table border). Changes
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@lib/src/framework/widgets/calendar.dart`:
- Around line 170-192: The code currently assigns decoration = const
BoxDecoration() when shapeProps is null which discards any provided shape value;
update the branch where shapeProps == null to construct a BoxDecoration that
preserves the existing shape variable (and sets borderRadius null when shape ==
BoxShape.circle) so the 'shape' passed into the widget is retained; modify the
logic around decoration, shapeProps, and headerBorderRadius in the constructor
(references: decoration, shapeProps, shape, BoxDecoration, headerBorderRadius)
so the default path mirrors the non-null path's shape handling without
introducing runtime changes.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@lib/src/framework/widgets/calendar.dart`:
- Around line 139-141: The file fails Dart formatting; run the Dart formatter
and reformat this file so the CI check passes — specifically ensure the line
defining headerPadding (the call to
To.edgeInsets(headerStyle.get('headerPadding'), or: const
EdgeInsets.symmetric(vertical: 8.0))) is formatted according to dart format
style; run `dart format` (or your IDE formatter) on
lib/src/framework/widgets/calendar.dart and commit the updated formatting.
| EdgeInsets headerPadding = To.edgeInsets(headerStyle.get('headerPadding'), | ||
| or: const EdgeInsets.symmetric(vertical: 8.0)); | ||
|
|
There was a problem hiding this comment.
Run dart format to fix the failing CI check.
The pipeline reports a Dart format failure; please run the formatter (e.g., on this file) so the check passes.
🤖 Prompt for AI Agents
In `@lib/src/framework/widgets/calendar.dart` around lines 139 - 141, The file
fails Dart formatting; run the Dart formatter and reformat this file so the CI
check passes — specifically ensure the line defining headerPadding (the call to
To.edgeInsets(headerStyle.get('headerPadding'), or: const
EdgeInsets.symmetric(vertical: 8.0))) is formatted according to dart format
style; run `dart format` (or your IDE formatter) on
lib/src/framework/widgets/calendar.dart and commit the updated formatting.
No description provided.