From a73573c165e15ef09e2d344f5f8bd1ffdafe23e6 Mon Sep 17 00:00:00 2001 From: Pea Tyczynska Date: Thu, 10 Sep 2020 16:31:35 +0100 Subject: [PATCH] Turn autofocus on for copy template Accessibility audit uncovered issues screen-reader users would have with autofocus on Notify pages. Autofocus can lead to confusion for those types of users and make them miss parts of the page. Hence, we decided that autofocus will be turned off by default for live-search component. We decided to keep the autofocus on for live-search text box when: 1. the page is a task page - as opposed to browsing pages, where user wonders freely, task pages have more established flow, so page focusing on textbox can actually be helpful. 2. page does not have actionable elements above the autofocus. --- app/templates/views/templates/copy.html | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/app/templates/views/templates/copy.html b/app/templates/views/templates/copy.html index 7aac461ec..6bd5f74ba 100644 --- a/app/templates/views/templates/copy.html +++ b/app/templates/views/templates/copy.html @@ -2,15 +2,16 @@ {% from "components/live-search.html" import live_search %} {% extends "withnav_template.html" %} +{% set page_title = "Copy an existing template" %} {% block service_page_title %} - Copy an existing template + {{ page_title }} {% endblock %} {% block maincolumn_content %}
-

Copy an existing template

+

{{ page_title }}

{{ copy_folder_path(template_folder_path, current_service.id, from_service, current_user) }}
{% if not services_templates_and_folders.templates_to_show %} @@ -18,7 +19,12 @@ This folder is empty

{% else %} - {{ live_search(target_selector='#template-list .template-list-item', show=True, form=search_form) }} + {{ live_search( + target_selector='#template-list .template-list-item', + show=True, + form=search_form, + autofocus=True + ) }}