Completely hide chart titles if blank

Before this change, a chart with a blank title would show up in Excel with a default title like “Chart title” or “Diagrammtitel”.

It was not possible to completely hide a chart title: Setting it to an empty space would override Excel’s default, but the empty title did still occupy vertical space.

Per OOXML standard the `title` element is optional, so omitting it if the title is blank looks like the right thing to do and seems to work correctly in Excel.

1 files changed, 14 insertions, 0 deletions

diff --git a/test/drawing/tc_chart.rb b/test/drawing/tc_chart.rb
index 302d13cd..58622621 100644
--- a/test/drawing/tc_chart.rb
+++ b/test/drawing/tc_chart.rb
@@ -1,3 +1,5 @@
+$LOAD_PATH.unshift "#{File.dirname(__FILE__)}/../"
+
 require 'tc_helper.rb'
 
 class TestChart < Test::Unit::TestCase
@@ -25,6 +27,8 @@ class TestChart < Test::Unit::TestCase
     @chart.title = @row.cells.first
     assert_equal(@chart.title.text, "one", "the title text was set via cell reference")
     assert_equal(@chart.title.cell, @row.cells.first)
+    @chart.title = ""
+    assert(@chart.title.empty?)
   end
 
   def test_style
@@ -121,4 +125,14 @@ class TestChart < Test::Unit::TestCase
     doc = Nokogiri::XML(@chart.to_xml_string)
     assert_equal("span", doc.xpath("//c:dispBlanksAs").attr("val").value, "did not use the display_blanks_as configuration")
   end
+
+  def test_to_xml_string_for_title
+    @chart.title = "foobar"
+    doc = Nokogiri::XML(@chart.to_xml_string)
+    assert_equal("foobar", doc.xpath("//c:title//c:tx//a:t").text)
+
+    @chart.title = ""
+    doc = Nokogiri::XML(@chart.to_xml_string)
+    assert_equal(0, doc.xpath("//c:title").size)
+  end
 end