From f2eb9de4425db1d7bd7044b6f6b041a3b4b8d49b Mon Sep 17 00:00:00 2001
From: Ryan Wyllie <ryan@moodle.com>
Date: Tue, 14 May 2019 11:56:33 +0800
Subject: [PATCH] MDL-65577 forum: move forum focus outline onto post div
---
mod/forum/styles.css | 9 +++++++++
mod/forum/templates/forum_discussion_post.mustache | 4 ++--
2 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/mod/forum/styles.css b/mod/forum/styles.css
index d1e59bcdd5c..c7f31dd9a45 100644
--- a/mod/forum/styles.css
+++ b/mod/forum/styles.css
@@ -342,10 +342,19 @@ span.unread {
font-size: 80%;
}
+.forum-post-container:focus {
+ outline: 0;
+}
+
/* This style is copied directly from the Bootstrap reboot file which adds button outline */
/* stylelint-disable declaration-block-no-duplicate-properties */
.post-actions .btn:focus {
outline: 1px dotted;
outline: 5px auto -webkit-focus-ring-color;
}
+
+.forum-post-container:focus > .focus-target {
+ outline: 1px dotted;
+ outline: 5px auto -webkit-focus-ring-color;
+}
/* stylelint-enable declaration-block-no-duplicate-properties */
diff --git a/mod/forum/templates/forum_discussion_post.mustache b/mod/forum/templates/forum_discussion_post.mustache
index 4fe9a4b4e25..e5a8e1d1c48 100644
--- a/mod/forum/templates/forum_discussion_post.mustache
+++ b/mod/forum/templates/forum_discussion_post.mustache
@@ -31,7 +31,7 @@
}}
<article
id="p{{id}}"
- class="relativelink mb-2"
+ class="forum-post-container mb-2"
data-post-id="{{id}}"
data-region="post"
data-target="{{id}}-target"
@@ -41,7 +41,7 @@
>
<!-- The firstpost and starter classes below aren't used for anything other than to identify the first post in behat -->
<div
- class="d-flex border p-2 mb-2 forumpost {{#unread}}unread{{/unread}} {{#firstpost}}firstpost starter{{/firstpost}}"
+ class="d-flex border p-2 mb-2 forumpost focus-target {{#unread}}unread{{/unread}} {{#firstpost}}firstpost starter{{/firstpost}}"
aria-label='{{#str}} postbyuser, mod_forum, {"post": "{{subject}}", "user": "{{author.fullname}}"} {{/str}}'
data-post-id="{{id}}" data-content="forum-post"
>