Skip to content

Commit ea48d25

Browse files
krushnaroutclaude
andcommitted
Fix: make Daily Score toggle independent of Goal Tracker
Daily Score widget only uses GoalsProvider for the CTA button label (Add Goals vs New Task) — the score itself is purely task-based. Remove the hidden dependency so each of the three toggles works independently. Also pass null for goalsWidgetKey when Goal Tracker is off so the Add Goals button gracefully falls back to New Task. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent a758a4c commit ea48d25

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

app/lib/pages/conversations/conversations_page.dart

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@ class _ConversationsPageState extends State<ConversationsPage> with AutomaticKee
224224
if (convoProvider.showDailySummaries || isSearchActive || hasCalendarFilter) {
225225
return const SliverToBoxAdapter(child: SizedBox.shrink());
226226
}
227-
final showDailyScore = prefs.showGoalTrackerEnabled && prefs.showDailyScoreEnabled;
227+
final showDailyScore = prefs.showDailyScoreEnabled;
228228
final showTasks = prefs.showTasksEnabled;
229229
final showGoals = prefs.showGoalTrackerEnabled;
230230
if (!showDailyScore && !showTasks && !showGoals) {
@@ -236,7 +236,10 @@ class _ConversationsPageState extends State<ConversationsPage> with AutomaticKee
236236
if (showDailyScore)
237237
Padding(
238238
padding: const EdgeInsets.only(left: 16, right: 16, top: 16),
239-
child: DailyScoreWidget(key: _dailyScoreWidgetKey, goalsWidgetKey: _goalsWidgetKey),
239+
child: DailyScoreWidget(
240+
key: _dailyScoreWidgetKey,
241+
goalsWidgetKey: showGoals ? _goalsWidgetKey : null,
242+
),
240243
),
241244
if (showTasks) const TodayTasksWidget(),
242245
if (showGoals) GoalsWidget(key: _goalsWidgetKey, onRefresh: _refreshGoals),

0 commit comments

Comments
 (0)