From f1ceb1a2a32e535f4006a163b96b7819d514777f Mon Sep 17 00:00:00 2001
From: Antonia Schwennesen <53372671+zophiana@users.noreply.github.com>
Date: Sat, 1 Feb 2025 08:54:59 +0100
Subject: [PATCH 1/2] fix(calendar-list): restrict calendar visibility toggle
 to checkbox only

Previously, clicking anywhere on the calendar name would toggle its visibility,
leading to accidental hiding of calendars. Now the visibility can only be
toggled by clicking the checkbox/bullet icon.

Fixes #3027

Signed-off-by: Antonia Schwennesen <53372671+zophiana@users.noreply.github.com>
---
 .../AppNavigation/CalendarList/CalendarListItem.vue        | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/components/AppNavigation/CalendarList/CalendarListItem.vue b/src/components/AppNavigation/CalendarList/CalendarListItem.vue
index fee0ee0c5d..7fc13ecd30 100644
--- a/src/components/AppNavigation/CalendarList/CalendarListItem.vue
+++ b/src/components/AppNavigation/CalendarList/CalendarListItem.vue
@@ -8,15 +8,16 @@
 		:aria-description="descriptionAppNavigationItem"
 		:name="calendarDisplayName || $t('calendar', 'Untitled calendar')"
 		:class="{deleted: isBeingDeleted, disabled: !calendar.enabled}"
-		@click.prevent.stop="toggleEnabled"
 		@update:menuOpen="actionsMenuOpen = $event">
 		<template #icon>
 			<CheckboxMarked v-if="calendar.enabled"
 				:size="20"
-				:fill-color="calendar.color" />
+				:fill-color="calendar.color"
+				@click.prevent.stop="toggleEnabled" />
 			<CheckboxBlank v-else
 				:size="20"
-				:fill-color="calendar.color" />
+				:fill-color="calendar.color"
+				@click.prevent.stop="toggleEnabled" />
 		</template>
 
 		<template #counter>

From 8fb5f9743f507f5d8679e77571fd3268e472945f Mon Sep 17 00:00:00 2001
From: Antonia Schwennesen <53372671+zophiana@users.noreply.github.com>
Date: Sat, 1 Feb 2025 09:03:45 +0100
Subject: [PATCH 2/2] fix(public-calendar): remove toggle functionality from
 public view

The enabled state cannot be persisted in public views, so the toggle
functionality should not be available. Remove click handler from bullet
icon to prevent misleading interaction.

Co-authored-by: Richard Steinmetz <richard@steinmetz.cloud>
Signed-off-by: Antonia Schwennesen <53372671+zophiana@users.noreply.github.com>
---
 .../AppNavigation/CalendarList/PublicCalendarListItem.vue  | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/components/AppNavigation/CalendarList/PublicCalendarListItem.vue b/src/components/AppNavigation/CalendarList/PublicCalendarListItem.vue
index dde60f6936..374eaaaa8e 100644
--- a/src/components/AppNavigation/CalendarList/PublicCalendarListItem.vue
+++ b/src/components/AppNavigation/CalendarList/PublicCalendarListItem.vue
@@ -6,12 +6,9 @@
 <template>
 	<AppNavigationItem :loading="calendar.loading"
 		:name="calendar.displayName || $t('calendar', 'Untitled calendar')"
-		:menu-open.sync="menuOpen"
-		@click.prevent.stop="toggleEnabled">
+		:menu-open.sync="menuOpen">
 		<template #icon>
-			<AppNavigationIconBullet v-if="calendar.enabled"
-				:color="calendar.color"
-				@click.prevent.stop="toggleEnabled" />
+			<AppNavigationIconBullet :color="calendar.color" />
 		</template>
 
 		<template #counter>