Minor improvements to constructors and MathMLLength

BenjaminTMilnes · BenjaminTMilnes · commit 3efded0a638c · 2016-03-30T20:46:07.000+01:00
diff --git a/MathML.Tests/MathMLGlyphTests.cs b/MathML.Tests/MathMLGlyphTests.cs
@@ -27,7 +27,7 @@ public void EmptyMathMLGlyphTest()
         [TestMethod]
         public void MathMLGlyphWithAllAttributesTest()
         {
-            Assert.AreEqual("<mglyph src=\"somefile.png\" alt=\"Some Text\" width=\"12pt\" height=\"14mm\" valign=\"16px\"></mglyph>", _serializer.SerializeMathMLNode(new MathMLGlyph() { AlternativeText = "Some Text", Source = "somefile.png", Width = new MathMLLength(12, MathMLLengthUnit.Points), Height = new MathMLLength(14, MathMLLengthUnit.Millimeters), VerticalAlignment = new MathMLLength(16, MathMLLengthUnit.Pixels) }));
+            Assert.AreEqual("<mglyph src=\"somefile.png\" alt=\"Some Text\" width=\"12pt\" height=\"14mm\" valign=\"16px\"></mglyph>", _serializer.SerializeMathMLNode(new MathMLGlyph() { AlternativeText = "Some Text", Source = "somefile.png", Width = new MathMLMeasuredLength(12, MathMLMeasuredLengthUnit.Points), Height = new MathMLMeasuredLength(14, MathMLMeasuredLengthUnit.Millimeters), VerticalAlignment = new MathMLMeasuredLength(16, MathMLMeasuredLengthUnit.Pixels) }));
         }
     }
 }
diff --git a/MathML.Tests/MathMLMultiscriptsTests.cs b/MathML.Tests/MathMLMultiscriptsTests.cs
@@ -27,7 +27,7 @@ public void EmptyMathMLMultiscriptsTest()
         [TestMethod]
         public void MathMLMultiscriptsWithAllAttributesTest()
         {
-            Assert.AreEqual("<mmultiscripts subscriptshift=\"4pt\" superscriptshift=\"8mm\"></mmultiscripts>", _serializer.SerializeMathMLNode(new MathMLMultiscripts() { SubscriptShift = new MathMLLength(4, MathMLLengthUnit.Points), SuperscriptShift = new MathMLLength(8, MathMLLengthUnit.Millimeters) }));
+            Assert.AreEqual("<mmultiscripts subscriptshift=\"4pt\" superscriptshift=\"8mm\"></mmultiscripts>", _serializer.SerializeMathMLNode(new MathMLMultiscripts() { SubscriptShift = new MathMLMeasuredLength(4, MathMLMeasuredLengthUnit.Points), SuperscriptShift = new MathMLMeasuredLength(8, MathMLMeasuredLengthUnit.Millimeters) }));
         }
     }
 }
diff --git a/MathML.Tests/MathMLOperatorTests.cs b/MathML.Tests/MathMLOperatorTests.cs
@@ -38,7 +38,7 @@ public void MathMLOperatorWithContentTest()
         [TestMethod]
         public void MathMLOperatorWithAllAttributes()
         {
-            Assert.AreEqual("<mo dir=\"rtl\" mathvariant=\"bold\" form=\"postfix\" largeop=\"true\" movablelimits=\"true\" accent=\"true\" fence=\"true\" stretchy=\"true\" symmetric=\"true\" lspace=\"10pt\" rspace=\"5mm\"></mo>", _serializer.SerializeMathMLNode(new MathMLOperator() { Accent = true, Fence = true, Form = MathMLOperatorForm.Postfix, LargeOperator = true, MovableLimits = true, Stretchy = true, Symmetric = true, LeftSpace = new MathMLLength(10, MathMLLengthUnit.Points), RightSpace = new MathMLLength(5, MathMLLengthUnit.Millimeters), TextDirection = MathMLTextDirection.RightToLeft, MathVariant = MathMLMathVariant.Bold }));
+            Assert.AreEqual("<mo dir=\"rtl\" mathvariant=\"bold\" form=\"postfix\" largeop=\"true\" movablelimits=\"true\" accent=\"true\" fence=\"true\" stretchy=\"true\" symmetric=\"true\" lspace=\"10pt\" rspace=\"5mm\"></mo>", _serializer.SerializeMathMLNode(new MathMLOperator() { Accent = true, Fence = true, Form = MathMLOperatorForm.Postfix, LargeOperator = true, MovableLimits = true, Stretchy = true, Symmetric = true, LeftSpace = new MathMLMeasuredLength(10, MathMLMeasuredLengthUnit.Points), RightSpace = new MathMLMeasuredLength(5, MathMLMeasuredLengthUnit.Millimeters), TextDirection = MathMLTextDirection.RightToLeft, MathVariant = MathMLMathVariant.Bold }));
         }
     }
 }
diff --git a/MathML.Tests/MathMLPaddedTests.cs b/MathML.Tests/MathMLPaddedTests.cs
@@ -27,7 +27,7 @@ public void EmptyMathMLPaddedTest()
         [TestMethod]
         public void MathMLPaddedWithAllAttributes()
         {
-            Assert.AreEqual("<mpadded width=\"10pt\" height=\"5mm\" depth=\"20px\" lspace=\"1in\" voffset=\"15em\"></mpadded>", _serializer.SerializeMathMLNode(new MathMLPadded() { Width = new MathMLLength(10, MathMLLengthUnit.Points), Height = new MathMLLength(5, MathMLLengthUnit.Millimeters), Depth = new MathMLLength(20, MathMLLengthUnit.Pixels), LeftSpace = new MathMLLength(1, MathMLLengthUnit.Inches), VerticalOffset = new MathMLLength(15, MathMLLengthUnit.Em) }));
+            Assert.AreEqual("<mpadded width=\"10pt\" height=\"5mm\" depth=\"20px\" lspace=\"1in\" voffset=\"15em\"></mpadded>", _serializer.SerializeMathMLNode(new MathMLPadded() { Width = new MathMLMeasuredLength(10, MathMLMeasuredLengthUnit.Points), Height = new MathMLMeasuredLength(5, MathMLMeasuredLengthUnit.Millimeters), Depth = new MathMLMeasuredLength(20, MathMLMeasuredLengthUnit.Pixels), LeftSpace = new MathMLMeasuredLength(1, MathMLMeasuredLengthUnit.Inches), VerticalOffset = new MathMLMeasuredLength(15, MathMLMeasuredLengthUnit.Em) }));
         }
     }
 }
diff --git a/MathML.Tests/MathMLSpaceTests.cs b/MathML.Tests/MathMLSpaceTests.cs
@@ -27,7 +27,7 @@ public void EmptyMathMLSpaceTest()
         [TestMethod]
         public void MathMLSpaceWithAllAttributesTest()
         {
-            Assert.AreEqual("<mspace width=\"10pt\" height=\"5mm\" depth=\"20px\" linebreak=\"newline\"></mspace>", _serializer.SerializeMathMLNode(new MathMLSpace() { Width = new MathMLLength(10, MathMLLengthUnit.Points), Height = new MathMLLength(5, MathMLLengthUnit.Millimeters), Depth = new MathMLLength(20, MathMLLengthUnit.Pixels), LineBreak = MathMLLineBreak.NewLine }));
+            Assert.AreEqual("<mspace width=\"10pt\" height=\"5mm\" depth=\"20px\" linebreak=\"newline\"></mspace>", _serializer.SerializeMathMLNode(new MathMLSpace() { Width = new MathMLMeasuredLength(10, MathMLMeasuredLengthUnit.Points), Height = new MathMLMeasuredLength(5, MathMLMeasuredLengthUnit.Millimeters), Depth = new MathMLMeasuredLength(20, MathMLMeasuredLengthUnit.Pixels), LineBreak = MathMLLineBreak.NewLine }));
         }
     }
 }
diff --git a/MathML.Tests/MathMLStyleTests.cs b/MathML.Tests/MathMLStyleTests.cs
@@ -27,7 +27,7 @@ public void EmptyMathMLStyleTest()
         [TestMethod]
         public void MathMLStyleWithAllAttributesTest()
         {
-            Assert.AreEqual("<mstyle dir=\"rtl\" displaystyle=\"false\" decimalpoint=\",\" infixlinebreakstyle=\"duplicate\" scriptlevel=\"10\" scriptminsize=\"15mm\" scriptsizemultiplier=\"2.5\"></mstyle>", _serializer.SerializeMathMLNode(new MathMLStyle() { TextDirection = MathMLTextDirection.RightToLeft, DisplayStyle = false, DecimalPoint = ',', InfixLineBreakStyle = MathMLInfixLineBreakStyle.Duplicate, ScriptLevel = 10, ScriptMinimumSize = new MathMLLength(15, MathMLLengthUnit.Millimeters), ScriptSizeMultiplier = 2.5 }));
+            Assert.AreEqual("<mstyle dir=\"rtl\" displaystyle=\"false\" decimalpoint=\",\" infixlinebreakstyle=\"duplicate\" scriptlevel=\"10\" scriptminsize=\"15mm\" scriptsizemultiplier=\"2.5\"></mstyle>", _serializer.SerializeMathMLNode(new MathMLStyle() { TextDirection = MathMLTextDirection.RightToLeft, DisplayStyle = false, DecimalPoint = ',', InfixLineBreakStyle = MathMLInfixLineBreakStyle.Duplicate, ScriptLevel = 10, ScriptMinimumSize = new MathMLMeasuredLength(15, MathMLMeasuredLengthUnit.Millimeters), ScriptSizeMultiplier = 2.5 }));
         }
     }
 }
diff --git a/MathML.Tests/MathMLSubscriptSuperscriptTests.cs b/MathML.Tests/MathMLSubscriptSuperscriptTests.cs
@@ -27,7 +27,7 @@ public void EmptyMathMLSubscriptSuperscriptTest()
         [TestMethod]
         public void MathMLSubscriptSuperscriptWithAllAttributesTest()
         {
-            Assert.AreEqual("<msubsup subscriptshift=\"10pt\" superscriptshift=\"15mm\"></msubsup>", _serializer.SerializeMathMLNode(new MathMLSubscriptSuperscript() { SubscriptShift = new MathMLLength(10, MathMLLengthUnit.Points), SuperscriptShift = new MathMLLength(15, MathMLLengthUnit.Millimeters) }));
+            Assert.AreEqual("<msubsup subscriptshift=\"10pt\" superscriptshift=\"15mm\"></msubsup>", _serializer.SerializeMathMLNode(new MathMLSubscriptSuperscript() { SubscriptShift = new MathMLMeasuredLength(10, MathMLMeasuredLengthUnit.Points), SuperscriptShift = new MathMLMeasuredLength(15, MathMLMeasuredLengthUnit.Millimeters) }));
         }
     }
 }
diff --git a/MathML.Tests/MathMLSubscriptTests.cs b/MathML.Tests/MathMLSubscriptTests.cs
@@ -27,7 +27,7 @@ public void EmptyMathMLSubscriptTest()
         [TestMethod]
         public void MathMLSubscriptWithAllAttributesTest()
         {
-            Assert.AreEqual("<msub subscriptshift=\"10pt\"></msub>", _serializer.SerializeMathMLNode(new MathMLSubscript() { Shift = new MathMLLength(10, MathMLLengthUnit.Points) }));
+            Assert.AreEqual("<msub subscriptshift=\"10pt\"></msub>", _serializer.SerializeMathMLNode(new MathMLSubscript() { Shift = new MathMLMeasuredLength(10, MathMLMeasuredLengthUnit.Points) }));
         }
     }
 }
diff --git a/MathML.Tests/MathMLSuperscriptTests.cs b/MathML.Tests/MathMLSuperscriptTests.cs
@@ -27,7 +27,7 @@ public void EmptyMathMLSuperscriptTest()
         [TestMethod]
         public void MathMLSuperscriptWithAllAttributes()
         {
-            Assert.AreEqual("<msup superscriptshift=\"10pt\"></msup>", _serializer.SerializeMathMLNode(new MathMLSuperscript() { Shift = new MathMLLength(10, MathMLLengthUnit.Points) }));
+            Assert.AreEqual("<msup superscriptshift=\"10pt\"></msup>", _serializer.SerializeMathMLNode(new MathMLSuperscript() { Shift = new MathMLMeasuredLength(10, MathMLMeasuredLengthUnit.Points) }));
         }
     }
 }
diff --git a/MathML/MathML.csproj b/MathML/MathML.csproj
@@ -59,8 +59,8 @@
     <Compile Include="MathMLInfixLineBreakStyle.cs" />
     <Compile Include="MathMLLabeledTableRow.cs" />
     <Compile Include="MathMLLabeledTableRowPlacement.cs" />
-    <Compile Include="MathMLLength.cs" />
-    <Compile Include="MathMLLengthUnit.cs" />
+    <Compile Include="MathMLMeasuredLength.cs" />
+    <Compile Include="MathMLMeasuredLengthUnit.cs" />
     <Compile Include="MathMLLineBreak.cs" />
     <Compile Include="MathMLLongDivision.cs" />
     <Compile Include="MathMLMath.cs" />
diff --git a/MathML/MathMLColor.cs b/MathML/MathMLColor.cs
@@ -6,7 +6,7 @@
 
 namespace MathML
 {
-    public class MathMLColor
+    public abstract class MathMLColor
     {
     }
 }
diff --git a/MathML/MathMLEnclose.cs b/MathML/MathMLEnclose.cs
@@ -10,6 +10,14 @@ namespace MathML
     public class MathMLEnclose : MathMLElement
     {
         [MathMLAttributeName("notation")]
+        [MathMLAttributeOrderIndex(7)]
         public MathMLEncloseNotation Notation { get; set; }
+
+        public MathMLEnclose() : this(MathMLEncloseNotation.LongDivision) { }
+
+        public MathMLEnclose(MathMLEncloseNotation notation)
+        {
+            Notation = notation;
+        }
     }
 }
diff --git a/MathML/MathMLGlyph.cs b/MathML/MathMLGlyph.cs
@@ -19,14 +19,14 @@ public class MathMLGlyph : MathMLElement
 
         [MathMLAttributeName("width")]
         [MathMLAttributeOrderIndex(9)]
-        public MathMLLength Width { get; set; }
+        public MathMLMeasuredLength Width { get; set; }
 
         [MathMLAttributeName("height")]
         [MathMLAttributeOrderIndex(10)]
-        public MathMLLength Height { get; set; }
+        public MathMLMeasuredLength Height { get; set; }
 
         [MathMLAttributeName("valign")]
         [MathMLAttributeOrderIndex(11)]
-        public MathMLLength VerticalAlignment { get; set; }
+        public MathMLMeasuredLength VerticalAlignment { get; set; }
     }
 }
diff --git a/MathML/MathMLMath.cs b/MathML/MathMLMath.cs
@@ -17,10 +17,14 @@ public class MathMLMath : MathMLElement
         [DefaultValue(MathMLOverflow.LineBreak)]
         public MathMLOverflow Overflow { get; set; }
 
-        public MathMLMath()
+        public MathMLMath() : this(MathMLDisplay.Block, MathMLOverflow.LineBreak) { }
+
+        public MathMLMath(MathMLDisplay display) : this(display, MathMLOverflow.LineBreak) { }
+
+        public MathMLMath(MathMLDisplay display, MathMLOverflow overflow)
         {
-            Display = MathMLDisplay.Block;
-            Overflow = MathMLOverflow.LineBreak;
+            Display = display;
+            Overflow = overflow;
         }
     }
 }
diff --git a/MathML/MathMLMeasuredLength.cs b/MathML/MathMLMeasuredLength.cs
@@ -6,12 +6,12 @@
 
 namespace MathML
 {
-    public class MathMLLength
+    public class MathMLMeasuredLength
     {
         public double Quantity { get; private set; }
-        public MathMLLengthUnit Units { get; private set; }
+        public MathMLMeasuredLengthUnit Units { get; private set; }
 
-        public MathMLLength(double quantity, MathMLLengthUnit units)
+        public MathMLMeasuredLength(double quantity, MathMLMeasuredLengthUnit units)
         {
             Quantity = quantity;
             Units = units;
diff --git a/MathML/MathMLMeasuredLengthUnit.cs b/MathML/MathMLMeasuredLengthUnit.cs
@@ -6,7 +6,7 @@
 
 namespace MathML
 {
-    public enum MathMLLengthUnit
+    public enum MathMLMeasuredLengthUnit
     {
         [MathMLAttributeValue("em")]
         Em = 0,
diff --git a/MathML/MathMLMultiscripts.cs b/MathML/MathMLMultiscripts.cs
@@ -10,9 +10,9 @@ namespace MathML
     public class MathMLMultiscripts : MathMLElement
     {
         [MathMLAttributeName("subscriptshift")]
-        public MathMLLength SubscriptShift { get; set; }
+        public MathMLMeasuredLength SubscriptShift { get; set; }
 
         [MathMLAttributeName("superscriptshift")]
-        public MathMLLength SuperscriptShift { get; set; }
+        public MathMLMeasuredLength SuperscriptShift { get; set; }
     }
 }
diff --git a/MathML/MathMLOperator.cs b/MathML/MathMLOperator.cs
@@ -51,11 +51,11 @@ public class MathMLOperator : MathMLElement
 
         [MathMLAttributeName("lspace")]
         [MathMLAttributeOrderIndex(17)]
-        public MathMLLength LeftSpace { get; set; }
+        public MathMLMeasuredLength LeftSpace { get; set; }
 
         [MathMLAttributeName("rspace")]
         [MathMLAttributeOrderIndex(18)]
-        public MathMLLength RightSpace { get; set; }
+        public MathMLMeasuredLength RightSpace { get; set; }
 
         [MathMLAttributeName("dir")]
         [MathMLAttributeOrderIndex(7)]
diff --git a/MathML/MathMLOver.cs b/MathML/MathMLOver.cs
@@ -13,5 +13,12 @@ public class MathMLOver : MathMLElement
         [MathMLAttributeOrderIndex(7)]
         [DefaultValue(false)]
         public bool Accent { get; set; }
+
+        public MathMLOver() : this(false) { }
+
+        public MathMLOver(bool accent)
+        {
+            Accent = accent;
+        }
     }
 }
diff --git a/MathML/MathMLPadded.cs b/MathML/MathMLPadded.cs
@@ -11,22 +11,22 @@ public class MathMLPadded : MathMLElement
     {
         [MathMLAttributeName("width")]
         [MathMLAttributeOrderIndex(7)]
-        public MathMLLength Width { get; set; }
+        public MathMLMeasuredLength Width { get; set; }
 
         [MathMLAttributeName("height")]
         [MathMLAttributeOrderIndex(8)]
-        public MathMLLength Height { get; set; }
+        public MathMLMeasuredLength Height { get; set; }
 
         [MathMLAttributeName("depth")]
         [MathMLAttributeOrderIndex(9)]
-        public MathMLLength Depth { get; set; }
+        public MathMLMeasuredLength Depth { get; set; }
 
         [MathMLAttributeName("lspace")]
         [MathMLAttributeOrderIndex(10)]
-        public MathMLLength LeftSpace { get; set; }
+        public MathMLMeasuredLength LeftSpace { get; set; }
 
         [MathMLAttributeName("voffset")]
         [MathMLAttributeOrderIndex(11)]
-        public MathMLLength VerticalOffset { get; set; }
+        public MathMLMeasuredLength VerticalOffset { get; set; }
     }
 }
diff --git a/MathML/MathMLSerializer.cs b/MathML/MathMLSerializer.cs
@@ -162,7 +162,7 @@ private string SerializeAttributeValue(object value)
             else if (value is MathMLTableVerticalAlignment) return SerializeEnumeration((MathMLTableVerticalAlignment)value);
             else if (value is MathMLTextDirection) return SerializeEnumeration((MathMLTextDirection)value);
             else if (value is MathMLUnderOverAlignment) return SerializeEnumeration((MathMLUnderOverAlignment)value);
-            else if (value is MathMLLength) return SerializeMathMLLength((MathMLLength)value);
+            else if (value is MathMLMeasuredLength) return SerializeMathMLLength((MathMLMeasuredLength)value);
             else if (value is MathMLColor) return SerializeMathMLColor((MathMLColor)value);
 
             throw new UnknownMathMLAttributeTypeException();
@@ -183,7 +183,7 @@ private string SerializeEnumeration<T>(T value)
             return ((MathMLAttributeValue)attribute).Value;
         }
 
-        private string SerializeMathMLLength(MathMLLength length)
+        private string SerializeMathMLLength(MathMLMeasuredLength length)
         {
             return string.Format("{0}{1}", length.Quantity.ToString(), SerializeEnumeration(length.Units));
         }
diff --git a/MathML/MathMLSpace.cs b/MathML/MathMLSpace.cs
@@ -11,15 +11,15 @@ public class MathMLSpace : MathMLElement
     {
         [MathMLAttributeName("width")]
         [MathMLAttributeOrderIndex(7)]
-        public MathMLLength Width { get; set; }
+        public MathMLMeasuredLength Width { get; set; }
 
         [MathMLAttributeName("height")]
         [MathMLAttributeOrderIndex(8)]
-        public MathMLLength Height { get; set; }
+        public MathMLMeasuredLength Height { get; set; }
 
         [MathMLAttributeName("depth")]
         [MathMLAttributeOrderIndex(9)]
-        public MathMLLength Depth { get; set; }
+        public MathMLMeasuredLength Depth { get; set; }
 
         [MathMLAttributeName("linebreak")]
         [MathMLAttributeOrderIndex(10)]
diff --git a/MathML/MathMLStyle.cs b/MathML/MathMLStyle.cs
@@ -36,7 +36,7 @@ public class MathMLStyle : MathMLElement
 
         [MathMLAttributeName("scriptminsize")]
         [MathMLAttributeOrderIndex(12)]
-        public MathMLLength ScriptMinimumSize { get; set; }
+        public MathMLMeasuredLength ScriptMinimumSize { get; set; }
 
         [MathMLAttributeName("scriptsizemultiplier")]
         [MathMLAttributeOrderIndex(13)]
diff --git a/MathML/MathMLSubscript.cs b/MathML/MathMLSubscript.cs
@@ -10,6 +10,6 @@ namespace MathML
     public class MathMLSubscript : MathMLElement
     {
         [MathMLAttributeName("subscriptshift")]
-        public MathMLLength Shift { get; set; }
+        public MathMLMeasuredLength Shift { get; set; }
     }
 }
diff --git a/MathML/MathMLSubscriptSuperscript.cs b/MathML/MathMLSubscriptSuperscript.cs
@@ -10,9 +10,9 @@ namespace MathML
     public class MathMLSubscriptSuperscript : MathMLElement
     {
         [MathMLAttributeName("subscriptshift")]
-        public MathMLLength SubscriptShift { get; set; }
+        public MathMLMeasuredLength SubscriptShift { get; set; }
 
         [MathMLAttributeName("superscriptshift")]
-        public MathMLLength SuperscriptShift { get; set; }
+        public MathMLMeasuredLength SuperscriptShift { get; set; }
     }
 }
diff --git a/MathML/MathMLSuperscript.cs b/MathML/MathMLSuperscript.cs
@@ -10,6 +10,6 @@ namespace MathML
     public class MathMLSuperscript : MathMLElement
     {
         [MathMLAttributeName("superscriptshift")]
-        public MathMLLength Shift { get; set; }
+        public MathMLMeasuredLength Shift { get; set; }
     }
 }
diff --git a/MathML/MathMLTable.cs b/MathML/MathMLTable.cs
@@ -10,6 +10,7 @@ namespace MathML
     public class MathMLTable : MathMLElement
     {
         [MathMLAttributeName("align")]
+        [DefaultValue(MathMLTableVerticalAlignment.Axis)]
         public MathMLTableVerticalAlignment VerticalAlignment { get; set; }
 
         [MathMLAttributeName("columnalign")]
@@ -19,28 +20,32 @@ public class MathMLTable : MathMLElement
         public IList<MathMLTableLineStyle> ColumnLineStyle { get; set; }
 
         [MathMLAttributeName("columnspacing")]
-        public MathMLLength ColumnSpacing { get; set; }
+        public MathMLMeasuredLength ColumnSpacing { get; set; }
 
         [MathMLAttributeName("columnwidth")]
-        public MathMLLength ColumnWidth { get; set; }
+        public MathMLMeasuredLength ColumnWidth { get; set; }
 
         [MathMLAttributeName("displaystyle")]
+        [DefaultValue(true)]
         public bool DisplayStyle { get; set; }
 
         [MathMLAttributeName("equalcolumns")]
+        [DefaultValue(false)]
         public bool EqualColumnHeights { get; set; }
 
         [MathMLAttributeName("equalrows")]
+        [DefaultValue(false)]
         public bool EqualRowHeights { get; set; }
 
         [MathMLAttributeName("frame")]
+        [DefaultValue(MathMLTableLineStyle.None)]
         public MathMLTableLineStyle Frame { get; set; }
 
         [MathMLAttributeName("framespacing")]
-        public MathMLLength FrameSpacing { get; set; }
+        public MathMLMeasuredLength FrameSpacing { get; set; }
 
         [MathMLAttributeName("minlabelspacing")]
-        public MathMLLength MinimumLabelSpacing { get; set; }
+        public MathMLMeasuredLength MinimumLabelSpacing { get; set; }
 
         [MathMLAttributeName("rowalign")]
         public IList<MathMLTableVerticalAlignment> RowAlignment { get; set; }
@@ -49,12 +54,23 @@ public class MathMLTable : MathMLElement
         public IList<MathMLTableLineStyle> RowLineStyle { get; set; }
 
         [MathMLAttributeName("rowspacing")]
-        public MathMLLength RowSpacing { get; set; }
+        public MathMLMeasuredLength RowSpacing { get; set; }
 
         [MathMLAttributeName("side")]
+        [DefaultValue(MathMLLabeledTableRowPlacement.Right)]
         public MathMLLabeledTableRowPlacement Side { get; set; }
 
         [MathMLAttributeName("width")]
-        public MathMLLength Width { get; set; }
+        public MathMLMeasuredLength Width { get; set; }
+
+        public MathMLTable()
+        {
+            VerticalAlignment = MathMLTableVerticalAlignment.Axis;
+            DisplayStyle = true;
+            EqualColumnHeights = false;
+            EqualRowHeights = false;
+            Frame = MathMLTableLineStyle.None;
+            Side = MathMLLabeledTableRowPlacement.Right;
+        }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ public void EmptyMathMLGlyphTest()`
`27`	`27`	`[TestMethod]`
`28`	`28`	`public void MathMLGlyphWithAllAttributesTest()`
`29`	`29`	`{`
`30`		`- Assert.AreEqual("<mglyph src=\"somefile.png\" alt=\"Some Text\" width=\"12pt\" height=\"14mm\" valign=\"16px\"></mglyph>", _serializer.SerializeMathMLNode(new MathMLGlyph() { AlternativeText = "Some Text", Source = "somefile.png", Width = new MathMLLength(12, MathMLLengthUnit.Points), Height = new MathMLLength(14, MathMLLengthUnit.Millimeters), VerticalAlignment = new MathMLLength(16, MathMLLengthUnit.Pixels) }));`
	`30`	`+ Assert.AreEqual("<mglyph src=\"somefile.png\" alt=\"Some Text\" width=\"12pt\" height=\"14mm\" valign=\"16px\"></mglyph>", _serializer.SerializeMathMLNode(new MathMLGlyph() { AlternativeText = "Some Text", Source = "somefile.png", Width = new MathMLMeasuredLength(12, MathMLMeasuredLengthUnit.Points), Height = new MathMLMeasuredLength(14, MathMLMeasuredLengthUnit.Millimeters), VerticalAlignment = new MathMLMeasuredLength(16, MathMLMeasuredLengthUnit.Pixels) }));`
`31`	`31`	`}`
`32`	`32`	`}`
`33`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public void MathMLOperatorWithContentTest()`
`38`	`38`	`[TestMethod]`
`39`	`39`	`public void MathMLOperatorWithAllAttributes()`
`40`	`40`	`{`
`41`		- Assert.AreEqual("<mo dir=\"rtl\" mathvariant=\"bold\" form=\"postfix\" largeop=\"true\" movablelimits=\"true\" accent=\"true\" fence=\"true\" stretchy=\"true\" symmetric=\"true\" lspace=\"10pt\" rspace=\"5mm\"></mo>", _serializer.SerializeMathMLNode(new MathMLOperator() { Accent = true, Fence = true, Form = MathMLOperatorForm.Postfix, LargeOperator = true, MovableLimits = true, Stretchy = true, Symmetric = true, LeftSpace = new MathMLLength(10, MathMLLengthUnit.Points), RightSpace = new MathMLLength(5, MathMLLengthUnit.Millimeters), TextDirection = MathMLTextDirection.RightToLeft, MathVariant = MathMLMathVariant.Bold }));
	`41`	+ Assert.AreEqual("<mo dir=\"rtl\" mathvariant=\"bold\" form=\"postfix\" largeop=\"true\" movablelimits=\"true\" accent=\"true\" fence=\"true\" stretchy=\"true\" symmetric=\"true\" lspace=\"10pt\" rspace=\"5mm\"></mo>", _serializer.SerializeMathMLNode(new MathMLOperator() { Accent = true, Fence = true, Form = MathMLOperatorForm.Postfix, LargeOperator = true, MovableLimits = true, Stretchy = true, Symmetric = true, LeftSpace = new MathMLMeasuredLength(10, MathMLMeasuredLengthUnit.Points), RightSpace = new MathMLMeasuredLength(5, MathMLMeasuredLengthUnit.Millimeters), TextDirection = MathMLTextDirection.RightToLeft, MathVariant = MathMLMathVariant.Bold }));
`42`	`42`	`}`
`43`	`43`	`}`
`44`	`44`	`}`