Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #14680 from edx/bjacobel/discussion-dupe-ids

Browse Source 
Fix duplicated ids in discussion action menus
This commit is contained in:
Brian Jacobel
2017-03-15 13:28:51 -04:00
committed by GitHub
parent 0300fb1c6e 85fc5871d6
commit de32e69b8d
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
common/static/common/templates/discussion/forum-actions.underscore
View File

@@ -1,13 +1,13 @@
<% if (!readOnly) { %>
<ul class="<%= contentType %>-actions-list">
<ul class="<%- contentType %>-actions-list">
<% _.each(primaryActions, function(action) { print(_.template($('#forum-action-' + action).html())({})) }) %>
<li class="actions-item is-visible">
<div class="more-wrapper">
<button class="btn-link action-button action-more" aria-label="<%- gettext('More') %>" aria-haspopup="true" aria-controls="action-menu-<%= contentType %>">
<button class="btn-link action-button action-more" aria-label="<%- gettext('More') %>" aria-haspopup="true" aria-controls="action-menu-<%- contentType %>-<%- contentId %>">
<span class="action-label"><%- gettext('More') %></span>
<span class="action-icon"><span class="icon fa fa-ellipsis-h" aria-hidden="true"></span></span>
</button>
<div class="actions-dropdown" id="action-menu-<%= contentType %>" aria-expanded="false">
<div class="actions-dropdown" id="action-menu-<%- contentType %>-<%- contentId %>" aria-expanded="false">
<ul class="actions-dropdown-list">
<% _.each(secondaryActions, function(action) { print(_.template($('#forum-action-' + action).html())({})) }) %>
</ul>