From bb09fd0febbac9bda69d38cd9ba6e1a0ba39a7e0 Mon Sep 17 00:00:00 2001
From: Sli <antoine@bartuccio.fr>
Date: Fri, 26 Sep 2025 14:33:17 +0200
Subject: [PATCH] Apply review comments

---
 core/templates/core/base/header.jinja | 2 +-
 trombi/views.py                       | 3 ---
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/core/templates/core/base/header.jinja b/core/templates/core/base/header.jinja
index 59fb6035..87453d22 100644
--- a/core/templates/core/base/header.jinja
+++ b/core/templates/core/base/header.jinja
@@ -89,7 +89,7 @@
               </span>
             {% endif %}
           </a>
-          <div id="header_notif" x-show="display" x-cloak x-transition>
+          <div id="header_notif" x-show="display" x-cloak x-transition @click.outside="display = false">
             <ul>
               {% if user.notifications.filter(viewed=False).count() > 0 %}
                 {% for n in user.notifications.filter(viewed=False).order_by('-date') %}
diff --git a/trombi/views.py b/trombi/views.py
index 56238c25..a1978ec3 100644
--- a/trombi/views.py
+++ b/trombi/views.py
@@ -405,9 +405,6 @@ class UserTrombiDeleteMembershipView(
     current_tab = "profile"
     success_message = _("User removed from trombi")
 
-    def get_success_url(self):
-        return super().get_success_url()
-
 
 # Used by admins when someone does not have every club in his list
 class UserTrombiAddMembershipView(TrombiTabsMixin, CreateView):