From 92366ebed796bcbccdfe40c8f89332f6871fade4 Mon Sep 17 00:00:00 2001
From: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri, 28 Jan 2022 19:20:52 +0100
Subject: [PATCH] manager: use .clickable-rows class (#59991)

And deprecate .plaintable which is now useless.
---
 .../static/authentic2/manager/css/style.css      |  4 ----
 src/authentic2/manager/tables.py                 | 16 ++++++++--------
 2 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/src/authentic2/manager/static/authentic2/manager/css/style.css b/src/authentic2/manager/static/authentic2/manager/css/style.css
index 6793ab00..0935a635 100644
--- a/src/authentic2/manager/static/authentic2/manager/css/style.css
+++ b/src/authentic2/manager/static/authentic2/manager/css/style.css
@@ -270,10 +270,6 @@ span.activity {
 	padding-right: 30px;
 }
 
-table.main.plaintable tr:hover td {
-	cursor: default;
-}
-
 a.role-inheritance-view-all {
     font-style: italic;
 }
diff --git a/src/authentic2/manager/tables.py b/src/authentic2/manager/tables.py
index 0e919623..ecc3aea2 100644
--- a/src/authentic2/manager/tables.py
+++ b/src/authentic2/manager/tables.py
@@ -77,7 +77,7 @@ class UserTable(Table):
 
     class Meta(Table.Meta):
         model = User
-        attrs = {'class': 'main', 'id': 'user-table'}
+        attrs = {'class': 'main clickable-rows', 'id': 'user-table'}
         fields = ('username', 'email', 'first_name', 'last_name', 'ou')
         sequence = ('link', '...')
         empty_text = _('None')
@@ -155,7 +155,7 @@ class OUTable(Table):
 
     class Meta(Table.Meta):
         model = OrganizationalUnit
-        attrs = {'class': 'main', 'id': 'ou-table'}
+        attrs = {'class': 'main clickable-rows', 'id': 'ou-table'}
         fields = ('name', 'slug', 'default')
         empty_text = _('None')
 
@@ -190,7 +190,7 @@ class OuUserRolesTable(Table):
 
     class Meta(Table.Meta):
         model = Role
-        attrs = {'class': 'main plaintable', 'id': 'role-table'}
+        attrs = {'class': 'main', 'id': 'role-table'}
         fields = ('name', 'ou')
         empty_text = _('None')
         order_by = ('name',)
@@ -216,7 +216,7 @@ class UserRolesTable(Table):
 
     class Meta(Table.Meta):
         model = Role
-        attrs = {'class': 'main', 'id': 'role-table'}
+        attrs = {'class': 'main clickable-rows', 'id': 'role-table'}
         fields = ('name', 'ou')
         empty_text = _('None')
         order_by = ('name', 'ou')
@@ -229,7 +229,7 @@ class ServiceTable(Table):
 
     class Meta(Table.Meta):
         model = Service
-        attrs = {'class': 'main', 'id': 'service-table'}
+        attrs = {'class': 'main clickable-rows', 'id': 'service-table'}
         fields = ('ou', 'name', 'slug')
         empty_text = _('None')
         order_by = ('ou', 'name', 'slug')
@@ -240,7 +240,7 @@ class ServiceRolesTable(Table):
 
     class Meta(Table.Meta):
         model = Role
-        attrs = {'class': 'main', 'id': 'service-role-table'}
+        attrs = {'class': 'main clickable-rows', 'id': 'service-role-table'}
         fields = ('name',)
         empty_text = _('No access restriction. All users are allowed to connect to this service.')
 
@@ -252,7 +252,7 @@ class UserAuthorizationsTable(Table):
 
     class Meta(Table.Meta):
         model = OIDCAuthorization
-        attrs = {'class': 'main plaintable', 'id': 'user-authorizations-table'}
+        attrs = {'class': 'main', 'id': 'user-authorizations-table'}
         fields = ('client', 'created', 'expired')
         empty_text = _('This user has not granted profile data access to any service yet.')
 
@@ -274,6 +274,6 @@ class InheritanceRolesTable(Table):
 
     class Meta(Table.Meta):
         model = Role
-        attrs = {'class': 'main plaintable', 'id': 'inheritance-role-table'}
+        attrs = {'class': 'main', 'id': 'inheritance-role-table'}
         fields = ('name', 'ou')
         empty_text = _('None')
-- 
2.34.1