Link "Topic Area" dropdown to the visibility message via aria-describedby.

This commit is contained in:
Sven Marnach
2017-09-25 17:52:57 +02:00
committed by Alex Dusenbery
parent 49a13216a4
commit 4837b6eb65
3 changed files with 3 additions and 8 deletions

View File

@@ -151,7 +151,7 @@
};
NewPostView.prototype.updateVisibilityMessage = function($target, forceDivided) {
var $visEl = $('.group-visibility .field-label-text');
var $visEl = $('#wrapper-visibility-message');
var visTemplate = edx.HtmlUtils.template($('#new-post-visibility-template').html());
var groupName = null;
if (($target && $target.data('divided')) || forceDivided) {

View File

@@ -30,12 +30,7 @@
</label>
</div>
<% } else if (is_discussion_division_enabled) { %>
<div class="post-field group-visibility">
<label class="field-label">
<!-- Wrapper for the visibility message filled out by a separate template -->
<span class="field-label-text"></span>
</label>
</div>
<div class="post-field group-visibility" id="wrapper-visibility-message"></div>
<% } %>
<div class="post-field">
<label class="field-label">

View File

@@ -6,7 +6,7 @@
<%- gettext("Add your post to a relevant topic to help others find it. (Required)") %>
</div>
<div class="field-input">
<select class="post-topic field-input" aria-describedby="field_help_topic_area" required>
<select class="post-topic field-input" aria-describedby="field_help_topic_area wrapper-visibility-message" required>
<%= edx.HtmlUtils.ensureHtml(topics_html) %>
</select>
</div>