[1432] Review corrections; removed underline from icon

linssen814 · linssen814 · commit bae071ae6c7f · 2025-02-25T10:41:53.000+01:00
diff --git a/bin/check_celery_worker_liveness.py b/bin/check_celery_worker_liveness.py
@@ -59,4 +59,4 @@
     sys.exit(1)
 
 print("Celery worker heartbeat found: OK.")
-sys.exit(0)
+sys.exit(0)
diff --git a/bootstrap.py b/bootstrap.py
@@ -13,26 +13,36 @@
 Set up my development environment for me!
 """
 
-project_name = 'open_inwoner'
+project_name = "open_inwoner"
 
 parser = argparse.ArgumentParser(description=description)
-parser.add_argument('target', choices=['production', 'staging', 'test', 'jenkins', 'dev'],
-                    help='production/staging/test/jenkins/dev')
-parser.add_argument('--project', default=project_name,
-                    help='Name of the project in your src directory, "%s" by default' % project_name)
-parser.add_argument('--init', action='store_true',
-                    help='Initialize a fresh "startproject" by pinning the requirements using pip-tools compile. '
-                         'Automatically done if requirements/base.txt does not yet exist.')
-parser.add_argument('--env', default='env',
-                    help='Directory name for virtualenv, "env" by default')
+parser.add_argument(
+    "target",
+    choices=["production", "staging", "test", "jenkins", "dev"],
+    help="production/staging/test/jenkins/dev",
+)
+parser.add_argument(
+    "--project",
+    default=project_name,
+    help='Name of the project in your src directory, "%s" by default' % project_name,
+)
+parser.add_argument(
+    "--init",
+    action="store_true",
+    help='Initialize a fresh "startproject" by pinning the requirements using pip-tools compile. '
+    "Automatically done if requirements/base.txt does not yet exist.",
+)
+parser.add_argument(
+    "--env", default="env", help='Directory name for virtualenv, "env" by default'
+)
 
 args = parser.parse_args()
 
 
 def replace_or_append(file_path, search_val, replace_val):
     file_handle, abs_path = mkstemp()
-    new_file = open(abs_path, 'w')
-    old_file = open(file_path, 'r')
+    new_file = open(abs_path, "w")
+    old_file = open(file_path, "r")
     found = False
     for line in old_file:
         if line.startswith(search_val):
@@ -51,89 +61,109 @@ def replace_or_append(file_path, search_val, replace_val):
 
 
 def replace_wsgi_settings(target):
-    path = os.path.join('src', project_name, 'wsgi.py')
+    path = os.path.join("src", project_name, "wsgi.py")
     replace_or_append(
-        path, 'os.environ.setdefault',
-        'os.environ.setdefault("DJANGO_SETTINGS_MODULE", "%s.conf.%s")\n' % (project_name, target)
+        path,
+        "os.environ.setdefault",
+        'os.environ.setdefault("DJANGO_SETTINGS_MODULE", "%s.conf.%s")\n'
+        % (project_name, target),
     )
 
 
 def replace_manage_settings(target):
-    path = os.path.join('src', project_name, 'manage.py')
+    path = os.path.join("src", project_name, "manage.py")
     replace_or_append(
-        path, '    os.environ.setdefault',
-        '    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "%s.conf.%s")\n' % (project_name, target)
+        path,
+        "    os.environ.setdefault",
+        '    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "%s.conf.%s")\n'
+        % (project_name, target),
     )
 
 
 def append_settings_activate(project, target, env):
-    if os.name == 'posix':
-        path = '%s/bin/activate' % env
-        replace_or_append(path, 'export DJANGO_SETTINGS_MODULE=',
-                          'export DJANGO_SETTINGS_MODULE=\'%s.conf.%s\'\n' %
-                          (project, target))
-    elif os.name == 'nt':
-        path = '%s\\Scripts\\activate.bat' % env
-        replace_or_append(path, 'set DJANGO_SETTINGS_MODULE=',
-                          'set DJANGO_SETTINGS_MODULE=%s.conf.%s\n' %
-                          (project, target))
-        path = '%s\\Scripts\\deactivate.bat' % env
-        replace_or_append(path, 'set DJANGO_SETTINGS_MODULE=',
-                          'set DJANGO_SETTINGS_MODULE=\n')
+    if os.name == "posix":
+        path = "%s/bin/activate" % env
+        replace_or_append(
+            path,
+            "export DJANGO_SETTINGS_MODULE=",
+            "export DJANGO_SETTINGS_MODULE='%s.conf.%s'\n" % (project, target),
+        )
+    elif os.name == "nt":
+        path = "%s\\Scripts\\activate.bat" % env
+        replace_or_append(
+            path,
+            "set DJANGO_SETTINGS_MODULE=",
+            "set DJANGO_SETTINGS_MODULE=%s.conf.%s\n" % (project, target),
+        )
+        path = "%s\\Scripts\\deactivate.bat" % env
+        replace_or_append(
+            path, "set DJANGO_SETTINGS_MODULE=", "set DJANGO_SETTINGS_MODULE=\n"
+        )
 
 
 def pip_compile_pin_requirements(virtualenv):
-    print('\n== Compiling base requirements ==\n')
-    if os.name == 'posix':
-        pip_path = os.path.join(virtualenv, 'bin', 'pip')
-    elif os.name == 'nt':
-        pip_path = os.path.join(virtualenv, 'Scripts', 'pip')
-    cmd_tpl = '{pip} install pip-tools'.format(pip=pip_path)
+    print("\n== Compiling base requirements ==\n")
+    if os.name == "posix":
+        pip_path = os.path.join(virtualenv, "bin", "pip")
+    elif os.name == "nt":
+        pip_path = os.path.join(virtualenv, "Scripts", "pip")
+    cmd_tpl = "{pip} install pip-tools".format(pip=pip_path)
     call(cmd_tpl, shell=True)
-    print('Error: Run `. env/bin/activate && ./bin/compile_dependencies.sh` to ensure you have requirements/base.txt and requirements/dev.txt')
-    print('After that rerun bootstrap.py')
+    print(
+        "Error: Run `. env/bin/activate && ./bin/compile_dependencies.sh` to ensure you have requirements/base.txt and requirements/dev.txt"
+    )
+    print("After that rerun bootstrap.py")
     sys.exit(1)
 
+
 def main():
     virtualenv = args.env
-    if not hasattr(sys, 'real_prefix'):
-        print('\n== Creating virtual environment ==\n')
-        call('virtualenv {0} --python=python3 --prompt="({1}-{2}) "'.format(
-            virtualenv, args.project, args.target
-        ), shell=True)
-    print('\n== Set "%s.conf.%s" as default settings ==\n' % (args.project, args.target))
+    if not hasattr(sys, "real_prefix"):
+        print("\n== Creating virtual environment ==\n")
+        call(
+            'virtualenv {0} --python=python3 --prompt="({1}-{2}) "'.format(
+                virtualenv, args.project, args.target
+            ),
+            shell=True,
+        )
+    print(
+        '\n== Set "%s.conf.%s" as default settings ==\n' % (args.project, args.target)
+    )
     append_settings_activate(args.project, args.target, args.env)
 
-    if os.name == 'posix':
+    if os.name == "posix":
         # Make manage.py executable
-        st = os.stat('src/manage.py')
-        os.chmod('src/manage.py', st.st_mode | stat.S_IEXEC)
-        django_admin_symlink = os.path.join(virtualenv, 'bin', 'django')
+        st = os.stat("src/manage.py")
+        os.chmod("src/manage.py", st.st_mode | stat.S_IEXEC)
+        django_admin_symlink = os.path.join(virtualenv, "bin", "django")
         if not os.path.exists(django_admin_symlink):
-            os.symlink('../../src/manage.py', django_admin_symlink)
-
-    print('\n== Upgrading Pip ==\n')
-    if os.name == 'posix':
-        pip_path = os.path.join(virtualenv, 'bin', 'pip')
-    elif os.name == 'nt':
-        pip_path = os.path.join(virtualenv, 'Scripts', 'pip')
-    cmd_tpl = '{pip} install --upgrade pip'.format(pip=pip_path)
+            os.symlink("../../src/manage.py", django_admin_symlink)
+
+    print("\n== Upgrading Pip ==\n")
+    if os.name == "posix":
+        pip_path = os.path.join(virtualenv, "bin", "pip")
+    elif os.name == "nt":
+        pip_path = os.path.join(virtualenv, "Scripts", "pip")
+    cmd_tpl = "{pip} install --upgrade pip".format(pip=pip_path)
     call(cmd_tpl, shell=True)
 
-    if args.init \
-       or not os.path.exists(os.path.join('requirements', 'base.txt')) \
-       or not os.path.exists(os.path.join('requirements', 'dev.txt')):
+    if (
+        args.init
+        or not os.path.exists(os.path.join("requirements", "base.txt"))
+        or not os.path.exists(os.path.join("requirements", "dev.txt"))
+    ):
         pip_compile_pin_requirements(virtualenv)
 
-    print('\n== Installing %s requirements ==\n' % args.target)
-    if os.name == 'posix':
-        os.environ['TMPDIR'] = '/var/tmp/'
-        pip_path = os.path.join(virtualenv, 'bin', 'pip')
-        cmd_tpl = '{pip} install -r requirements/{target}.txt'
-    elif os.name == 'nt':
-        pip_path = os.path.join(virtualenv, 'Scripts', 'pip')
-        cmd_tpl = '{pip} install -r requirements\\{target}.txt'
+    print("\n== Installing %s requirements ==\n" % args.target)
+    if os.name == "posix":
+        os.environ["TMPDIR"] = "/var/tmp/"
+        pip_path = os.path.join(virtualenv, "bin", "pip")
+        cmd_tpl = "{pip} install -r requirements/{target}.txt"
+    elif os.name == "nt":
+        pip_path = os.path.join(virtualenv, "Scripts", "pip")
+        cmd_tpl = "{pip} install -r requirements\\{target}.txt"
     return call(cmd_tpl.format(pip=pip_path, target=args.target), shell=True)
 
-if __name__ == '__main__':
+
+if __name__ == "__main__":
     sys.exit(main())
diff --git a/src/open_inwoner/utils/ckeditor.py b/src/open_inwoner/utils/ckeditor.py
@@ -55,7 +55,12 @@ def get_rendered_content(content: str) -> str:
             if element.name == "a" and element.attrs.get("href", "").startswith("http"):
                 # icon & screenreader support
                 icon = soup.new_tag("span")
-                icon.attrs.update({"aria-hidden": "true", "class": "material-icons"})
+                icon.attrs.update(
+                    {
+                        "aria-hidden": "true",
+                        "class": "material-icons text-decoration-none",
+                    }
+                )
                 icon.append("open_in_new")
 
                 screen_reader_only_text = soup.new_tag("span")
