Skip to content

Commit fada93e

Browse files
committed
Added a rendering guard that actually works.
1 parent 7981609 commit fada93e

File tree

3 files changed

+10
-15
lines changed

3 files changed

+10
-15
lines changed

lib/macruby-docs.coffee

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -95,13 +95,10 @@ if !window.in_tests
9595
check = ->
9696
try
9797
className = jQuery("#pageTitle", window.parent.frames[0].document).html().split(" ")[0]
98+
return if jQuery(".declaration .macruby", window.parent.frames[0].document).length > 0
9899
jQuery.each jQuery(".declaration", window.parent.frames[0].document), (i, element) ->
99100
content = element.innerHTML
100-
if content.indexOf("MacRuby") == -1
101-
element.innerHTML = element.innerHTML + "<h5 style='margin-top: 20px'>MacRuby</h5>" + new DocRenderer(className, content).render()
102-
else
103-
# If a page already has the annotations, don't check more than once. (the pages are loaded by ajax).
104-
return
101+
element.innerHTML = element.innerHTML + "<h5 class='macruby' style='margin-top: 20px'>MacRuby</h5>" + new DocRenderer(className, content).render()
105102
catch err
106103
console.log(err)
107104

lib/macruby-docs.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -117,14 +117,13 @@ if (!window.in_tests) {
117117
var className;
118118
try {
119119
className = jQuery("#pageTitle", window.parent.frames[0].document).html().split(" ")[0];
120+
if (jQuery(".declaration .macruby", window.parent.frames[0].document).length > 0) {
121+
return;
122+
}
120123
return jQuery.each(jQuery(".declaration", window.parent.frames[0].document), function(i, element) {
121124
var content;
122125
content = element.innerHTML;
123-
if (content.indexOf("MacRuby") === -1) {
124-
return element.innerHTML = element.innerHTML + "<h5 style='margin-top: 20px'>MacRuby</h5>" + new DocRenderer(className, content).render();
125-
} else {
126-
127-
}
126+
return element.innerHTML = element.innerHTML + "<h5 class='macruby' style='margin-top: 20px'>MacRuby</h5>" + new DocRenderer(className, content).render();
128127
});
129128
} catch (err) {
130129
return console.log(err);

lib/macruby-docs.stable.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -117,14 +117,13 @@ if (!window.in_tests) {
117117
var className;
118118
try {
119119
className = jQuery("#pageTitle", window.parent.frames[0].document).html().split(" ")[0];
120+
if (jQuery(".declaration .macruby", window.parent.frames[0].document).length > 0) {
121+
return;
122+
}
120123
return jQuery.each(jQuery(".declaration", window.parent.frames[0].document), function(i, element) {
121124
var content;
122125
content = element.innerHTML;
123-
if (content.indexOf("MacRuby") === -1) {
124-
return element.innerHTML = element.innerHTML + "<h5 style='margin-top: 20px'>MacRuby</h5>" + new DocRenderer(className, content).render();
125-
} else {
126-
127-
}
126+
return element.innerHTML = element.innerHTML + "<h5 class='macruby' style='margin-top: 20px'>MacRuby</h5>" + new DocRenderer(className, content).render();
128127
});
129128
} catch (err) {
130129
return console.log(err);

0 commit comments

Comments
 (0)