Skip to content

Commit 58d602e

Browse files
author
Sven Riedel
committed
Prevent lots of veritical whitespace caused by empty rdoc nodes
1 parent 6aaba60 commit 58d602e

File tree

3 files changed

+10
-5
lines changed

3 files changed

+10
-5
lines changed

lib/bri/match/base.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ def to_s
88

99
private
1010
def build_description( source )
11-
source.map { |element| Bri::Renderer.render( element ) }
11+
source.map { |element| Bri::Renderer.render( element ) }.compact
1212
end
1313
end
1414
end

lib/bri/renderer.rb

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,14 @@ def self.render( element, width = Bri.width, alignment_width = 0 )
5454

5555
else
5656
text = extract_text( element, width, alignment_width )
57-
styled_text = replace_markup( text )
58-
wrapped_text = wrap_to_width( styled_text, width )
59-
indent( wrapped_text )
57+
58+
if text == "\n"
59+
nil
60+
else
61+
styled_text = replace_markup( text )
62+
wrapped_text = wrap_to_width( styled_text, width )
63+
indent( wrapped_text )
64+
end
6065
end
6166
end
6267

spec/lib/bri/match/class_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@
4747

4848
describe "#description_paragraphs" do
4949
it "should be empty for an undocumented class" do
50-
empty_class.description_paragraphs.should == [ '' ]
50+
empty_class.description_paragraphs.should == []
5151
end
5252

5353
it "should contain rendered text for a documented class" do

0 commit comments

Comments
 (0)