Skip to content

Commit 82e910f

Browse files
committed
Merge pull request #17 from learn-javascript/feature-topic-ordering
Fixed Topic Ordering
2 parents 9bfa3f5 + 3fef81e commit 82e910f

File tree

1 file changed

+14
-6
lines changed

1 file changed

+14
-6
lines changed

assets/js/main.js

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
1-
21
(function () {
3-
42
// add more files here if needed
5-
var files = ['javascript.json', 'text-editors.json', 'jquery.json', 'node.json']
3+
var files = ['javascript.json', 'node.json', 'jquery.json', 'text-editors.json']
4+
5+
files.forEach(function (filename) {
6+
// create placeholder divs to enforce topic order
7+
var el = getTargetElement(filename.split('.')[0])
8+
document.getElementById('content').appendChild(el)
69

7-
.forEach(function (filename) {
810
getFile(filename).then(function (data) {
911
render(data)
1012
})
@@ -13,9 +15,10 @@
1315
function render (data) {
1416
var template = Handlebars.compile(document.getElementById('main_template').text)
1517
var html = template(data)
16-
var el = document.createElement('div')
18+
19+
var id = 'topic-' + data.title.replace(/ /g, '-')
20+
var el = document.getElementById(id)
1721
el.innerHTML = html
18-
document.getElementById('content').appendChild(el)
1922
}
2023

2124
function getFile (filename) {
@@ -26,4 +29,9 @@
2629
})
2730
}
2831

32+
function getTargetElement (id) {
33+
var el = document.createElement('div')
34+
el.setAttribute('id', 'topic-' + id)
35+
return el
36+
}
2937
})()

0 commit comments

Comments
 (0)