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

Display tags in note template.

Browse Source
This commit is contained in:
cahrens
2015-04-29 12:28:52 -04:00
parent 64047478d3
commit 8e636677ff
3 changed files with 21 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
lms/static/js/edxnotes/models/note.js
View File

@@ -12,6 +12,7 @@ define(['backbone', 'js/edxnotes/utils/utils', 'underscore.string'], function (B
'text': '',
'quote': '',
'ranges': [],
'tags': [],
'unit': {
'display_name': '',
'url': '',

17
lms/static/js/spec/edxnotes/views/note_item_spec.js
View File

@@ -53,8 +53,21 @@ define([
it('should display update value and accompanying text', function() {
var view = getView();
expect(view.$('.reference-title').last()).toContainText('Last Edited:');
expect(view.$('.reference-meta').last()).toContainText('December 11, 2014 at 11:12AM');
expect(view.$('.reference-title')[1]).toContainText('Last Edited:');
expect(view.$('.reference-updated-date').last()).toContainText('December 11, 2014 at 11:12AM');
});
it('should not display tags if there are none', function() {
var view = getView();
expect(view.$el).not.toContain('.reference-tags');
expect(view.$('.reference-title').length).toBe(2);
});
it('should display tags if they exist', function() {
var view = getView({tags: ["First", "Second"]});
expect(view.$('.reference-title').length).toBe(3);
expect(view.$('.reference-title')[2]).toContainText('Tags:');
expect(view.$('.reference-tags').last()).toContainText('First, Second');
});
it('should log the edx.student_notes.used_unit_link event properly', function () {

5
lms/templates/edxnotes/note-item.underscore
View File

@@ -34,5 +34,10 @@
<h3 class="reference-title"><%- gettext("Last Edited:") %></h3>
<span class="reference-meta reference-updated-date"><%- updated %></span>
<% if (tags.length > 0) { %>
<h3 class="reference-title"><%- gettext("Tags:") %></h3>
<span class="reference-meta reference-tags"><%- tags.join(", ") %></span>
<% } %>
</div>
</footer>