Skip to content

Commit d85dd19

Browse files
committed
Update slide rendering to reduce DOM wrap and avoid duplicate rendering slides
1 parent eab984d commit d85dd19

2 files changed

Lines changed: 9 additions & 5 deletions

File tree

public/js/reveal-markdown.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,7 @@
355355
var rendered = md.render(markdown);
356356
rendered = preventXSS(rendered);
357357
var result = postProcess(rendered);
358-
section.innerHTML = result[0].outerHTML;
358+
section.innerHTML = result[0].innerHTML;
359359
addAttributes( section, section, null, section.getAttribute( 'data-element-attributes' ) ||
360360
section.parentNode.getAttribute( 'data-element-attributes' ) ||
361361
DEFAULT_ELEMENT_ATTRIBUTES_SEPARATOR,

public/js/slide.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,10 +55,14 @@ viewAjaxCallback = function () {
5555
};
5656

5757
function renderSlide(event) {
58-
var title = document.title;
59-
finishView($(event.currentSlide));
60-
document.title = title;
61-
Reveal.layout();
58+
var markdown = $(event.currentSlide);
59+
if (!markdown.attr('data-rendered')) {
60+
var title = document.title;
61+
finishView(markdown);
62+
markdown.attr('data-rendered', 'true');
63+
document.title = title;
64+
Reveal.layout();
65+
}
6266
}
6367

6468
Reveal.addEventListener('ready', renderSlide);

0 commit comments

Comments
 (0)