[chores] Added help icon

Author: nemesifier

openwisp_utils/admin_theme/menu.py

@@ -103,15 +103,9 @@ def __init__(self, config):
             raise ImproperlyConfigured(
                 f'"url" should be a type of "str". Error for the config- {config}'
             )
-        self.url = self.make_django_url(url)
+        self.url = url
         self.icon = config.get('icon')
 
-    def make_django_url(self, url):
-        # Add "/" at the end of link
-        if url[-1] != '/':
-            return url + '/'
-        return url
-
 
 class MenuGroup(BaseMenuItem):
     """

openwisp_utils/admin_theme/static/admin/css/openwisp.css

@@ -1470,6 +1470,10 @@ body::-webkit-scrollbar {
   mask-image: url(../../ui/openwisp/images/logout.svg);
   -webkit-mask-image: url(../../ui/openwisp/images/logout.svg);
 }
+.ow-help {
+  mask-image: url(../../ui/openwisp/images/help.svg);
+  -webkit-mask-image: url(../../ui/openwisp/images/help.svg);
+}
 
 /* USERS ICONS */
 

openwisp_utils/admin_theme/static/ui/openwisp/images/help.svg

@@ -204,7 +204,7 @@ def test_menu_link(self):
             menu_link = MenuLink(config=_config)
             context = menu_link.create_context()
             self.assertEqual(context.get('label'), _config['label'])
-            self.assertEqual(context.get('url'), _config['url'] + '/')
+            self.assertEqual(context.get('url'), _config['url'])
             self.assertEqual(context.get('icon'), _config['icon'])
 
         with self.subTest('Menu Link without icon'):
@@ -213,6 +213,13 @@ def test_menu_link(self):
             context = menu_link.create_context()
             self.assertEqual(context.get('icon'), None)
 
+        with self.subTest('Menu Link test mailto'):
+            mailto = 'mailto:[email protected]'
+            _config = self._get_menu_link_config(url=mailto)
+            menu_link = MenuLink(config=_config)
+            context = menu_link.create_context()
+            self.assertEqual(context.get('url'), mailto)
+
     def test_model_link(self):
 
         with self.subTest('Model Link without name'):