From 9ed250056525756c459a3f234f8dd51fdec4527a Mon Sep 17 00:00:00 2001
From: lebaudantoine <lebaud.antoine131@gmail.com>
Date: Thu, 13 Nov 2025 20:24:43 +0100
Subject: [PATCH] =?UTF-8?q?=F0=9F=9A=B8(frontend)=20remove=20the=20?=
 =?UTF-8?q?=E2=80=9Cnone=E2=80=9D=20effect=20button?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

While this makes it slightly less explicit that clicking an already selected
option will disable the active effect, it improves accessibility by avoiding
automatic focus movement from the previously active option to a separate “none”
option. That focus shift could be misleading or hard to follow
for screen reader users.

Open to feedback on this decision.
---
 .../components/effects/EffectsConfiguration.tsx      | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/src/frontend/src/features/rooms/livekit/components/effects/EffectsConfiguration.tsx b/src/frontend/src/features/rooms/livekit/components/effects/EffectsConfiguration.tsx
index 33a57d33..7a5fcf76 100644
--- a/src/frontend/src/features/rooms/livekit/components/effects/EffectsConfiguration.tsx
+++ b/src/frontend/src/features/rooms/livekit/components/effects/EffectsConfiguration.tsx
@@ -15,7 +15,6 @@ import { BlurOnStrong } from '@/components/icons/BlurOnStrong'
 import { useTrackToggle } from '@livekit/components-react'
 import { Loader } from '@/primitives/Loader'
 import { useSyncAfterDelay } from '@/hooks/useSyncAfterDelay'
-import { RiProhibited2Line } from '@remixicon/react'
 import { FunnyEffects } from './FunnyEffects'
 import { useHasFunnyEffectsAccess } from '../../hooks/useHasFunnyEffectsAccess'
 
@@ -275,17 +274,6 @@ export const EffectsConfiguration = ({
                   gap: '1.25rem',
                 })}
               >
-                <ToggleButton
-                  variant="bigSquare"
-                  aria-label={t('clear')}
-                  onPress={async () => {
-                    await clearEffect()
-                  }}
-                  isSelected={!getProcessor()}
-                  isDisabled={processorPendingReveal || isDisabled}
-                >
-                  <RiProhibited2Line />
-                </ToggleButton>
                 <ToggleButton
                   variant="bigSquare"
                   aria-label={tooltipLabel(ProcessorType.BLUR, {