German comments in htmltab (squashed and fixed)

Change-Id: I37ffa94dc0edf58996fb59da838c02bb4a559519
Reviewed-on: https://gerrit.libreoffice.org/34325
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>

1 files changed, 346 insertions, 457 deletions

diff --git a/sw/source/filter/html/htmltab.cxx b/sw/source/filter/html/htmltab.cxx
index f0705e586555..e3d7f982b74c 100644
--- a/sw/source/filter/html/htmltab.cxx
+++ b/sw/source/filter/html/htmltab.cxx
@@ -442,7 +442,7 @@ class HTMLTable
     // the following parameters are from the <TABLE>-Tag
     sal_uInt16 m_nWidth;                  // width of the table
     sal_uInt16 m_nHeight;                 // absolute height of the table
-    SvxAdjust m_eTableAdjust;         // drawing::Alignment of the table
+    SvxAdjust m_eTableAdjust;             // drawing::Alignment of the table
     sal_Int16 m_eVertOrientation;         // Default vertical direction of the cells
     sal_uInt16 m_nBorder;                 // width of the external border
     HTMLTableFrame m_eFrame;          // frame around the table
@@ -515,7 +515,7 @@ class HTMLTable
 
 public:
 
-    bool m_bFirstCell;                // wurde schon eine Zelle angelegt?
+    bool m_bFirstCell;                // is there a cell created already?
 
     HTMLTable( SwHTMLParser* pPars, HTMLTable *pTopTab,
                bool bParHead, bool bHasParentSec,
@@ -568,10 +568,10 @@ public:
     void InsertCol( sal_uInt16 nSpan, sal_uInt16 nWidth, bool bRelWidth,
                     SvxAdjust eAdjust, sal_Int16 eVertOri );
 
-    // Beenden einer Tab-Definition (MUSS fuer ALLE Tabs aufgerufen werden)
+    // End a table definition (needs to be called for every table)
     void CloseTable();
 
-    // SwTable konstruieren (inkl. der Child-Tabellen)
+    // Construct a SwTable (including child tables)
     void MakeTable( SwTableBox *pUpper, sal_uInt16 nAbsAvail,
                     sal_uInt16 nRelAvail=0, sal_uInt16 nAbsLeftSpace=0,
                     sal_uInt16 nAbsRightSpace=0, sal_uInt16 nInhAbsSpace=0 );
@@ -639,7 +639,7 @@ HTMLTableCnts::HTMLTableCnts( HTMLTable* pTab ):
 
 HTMLTableCnts::~HTMLTableCnts()
 {
-    delete m_pTable;              // die Tabellen brauchen wir nicht mehr
+    delete m_pTable;              // we don't need the tables anymore
     delete m_pNext;
 }
 
@@ -693,8 +693,7 @@ HTMLTableCell::HTMLTableCell():
 
 HTMLTableCell::~HTMLTableCell()
 {
-    // der Inhalt ist in mehrere Zellen eingetragen, darf aber nur einmal
-    // geloescht werden
+    // the content is in multiple cells but mustn't be deleted more than once
     if( 1==nRowSpan && 1==nColSpan )
     {
         delete pContents;
@@ -733,13 +732,12 @@ inline void HTMLTableCell::SetWidth( sal_uInt16 nWdth, bool bRelWdth )
 
 void HTMLTableCell::SetProtected()
 {
-    // Die Inhalte dieser Zelle mussen nich irgenwo anders verankert
-    // sein, weil sie nicht geloescht werden!!!
+    // The content of this cell doesn't have to be anchored anywhere else,
+    // since they're not gonna be deleted
 
-    // Inhalt loeschen
     pContents = nullptr;
 
-    // Hintergrundfarbe kopieren.
+    // Copy background color
     if( pBGBrush )
         pBGBrush = new SvxBrushItem( *pBGBrush );
 
@@ -808,9 +806,8 @@ inline HTMLTableCell *HTMLTableRow::GetCell( sal_uInt16 nCell ) const
 
 void HTMLTableRow::Expand( sal_uInt16 nCells, bool bOneCell )
 {
-    // die Zeile wird mit einer einzigen Zelle aufgefuellt, wenn
-    // bOneCell gesetzt ist. Das geht, nur fuer Zeilen, in die keine
-    // Zellen mehr eingefuegt werden!
+    // This row will be filled with a single cell if bOneCell is set.
+    // This will only work for rows that don't allow adding cells!
 
     sal_uInt16 nColSpan = nCells - m_pCells->size();
     for (sal_uInt16 i = m_pCells->size(); i < nCells; ++i)
@@ -957,16 +954,14 @@ void HTMLTable::InitCtor( const HTMLTableOptions *pOptions )
     long nPHeight = nBorderOpt==USHRT_MAX ? 0 : nBorderOpt;
     SvxCSS1Parser::PixelToTwip( nPWidth, nPHeight );
 
-    // nBorder gibt die Breite der Umrandung an, wie sie in die
-    // Breitenberechnung in Netscape einfliesst. Wenn pOption->nBorder
-    // == USHRT_MAX, wurde keine BORDER-Option angegeben. Trotzdem fliesst
-    // eine 1 Pixel breite Umrandung in die Breitenberechnung mit ein.
+    // nBorder tells the width of the border as it's used in the width calculation of NetScape
+    // If pOption->nBorder == USHRT_MAX, there wasn't a BORDER option given
+    // Nonetheless, a 1 pixel wide border will be used for width calculation
     m_nBorder = (sal_uInt16)nPWidth;
     if( nBorderOpt==USHRT_MAX )
         nPWidth = 0;
 
-    // HACK: ein Pixel-breite Linien sollen zur Haarlinie werden, wenn
-    // wir mit doppelter Umrandung arbeiten
+    // HACK: one pixel wide lines should be hairlines when we'll use double bordering
     if( pOptions->nCellSpacing!=0 && nBorderOpt==1 )
     {
         nPWidth = 1;
@@ -1092,8 +1087,7 @@ HTMLTable::~HTMLTable()
 
     delete m_pContext;
 
-    // pLayoutInfo wurde entweder bereits geloescht oder muss aber es
-    // in den Besitz der SwTable uebergegangen.
+    // pLayoutInfo has either already been deleted or is now owned by SwTable
 }
 
 SwHTMLTableLayout *HTMLTable::CreateLayoutInfo()
@@ -1180,28 +1174,28 @@ void HTMLTable::ProtectRowSpan( sal_uInt16 nRow, sal_uInt16 nCol, sal_uInt16 nRo
     }
 }
 
-// Suchen des SwStartNodes der letzten belegten Vorgaengerbox
+// Search the SwStartNode of the last used predecessor box
 const SwStartNode* HTMLTable::GetPrevBoxStartNode( sal_uInt16 nRow, sal_uInt16 nCol ) const
 {
     const HTMLTableCnts *pPrevCnts = nullptr;
 
     if( 0==nRow )
     {
-        // immer die Vorgaenger-Zelle
+        // always the predecessor cell
         if( nCol>0 )
             pPrevCnts = GetCell( 0, nCol-1 )->GetContents();
         else
             return m_pPrevStartNode;
     }
     else if( USHRT_MAX==nRow && USHRT_MAX==nCol )
-        // der Contents der letzten Zelle
+        // contents of preceding cell
         pPrevCnts = GetCell( m_nRows-1, m_nCols-1 )->GetContents();
     else
     {
         sal_uInt16 i;
         HTMLTableRow *const pPrevRow = (*m_pRows)[nRow-1].get();
 
-        // evtl. eine Zelle in der aktuellen Zeile
+        // maybe a cell in the current row
         i = nCol;
         while( i )
         {
@@ -1213,7 +1207,7 @@ const SwStartNode* HTMLTable::GetPrevBoxStartNode( sal_uInt16 nRow, sal_uInt16 n
             }
         }
 
-        // sonst die letzte gefuellte Zelle der Zeile davor suchen
+        // otherwise the last filled cell of the row before
         if( !pPrevCnts )
         {
             i = m_nCols;
@@ -1283,12 +1277,12 @@ void HTMLTable::FixFrameFormat( SwTableBox *pBox,
                              sal_uInt16 nRowSpan, sal_uInt16 nColSpan,
                              bool bFirstPara, bool bLastPara ) const
 {
-    SwFrameFormat *pFrameFormat = nullptr;      // frame::Frame-Format
+    SwFrameFormat *pFrameFormat = nullptr;      // frame::Frame format
     sal_Int16 eVOri = text::VertOrientation::NONE;
-    const SvxBrushItem *pBGBrushItem = nullptr;   // Hintergrund
+    const SvxBrushItem *pBGBrushItem = nullptr;   // background
     std::shared_ptr<SvxBoxItem> pBoxItem;
     bool bTopLine = false, bBottomLine = false, bLastBottomLine = false;
-    bool bReUsable = false;     // Format nochmals verwendbar?
+    bool bReUsable = false;     // Format reusable?
     sal_uInt16 nEmptyRows = 0;
     bool bHasNumFormat = false;
     bool bHasValue = false;
@@ -1299,19 +1293,16 @@ void HTMLTable::FixFrameFormat( SwTableBox *pBox,
 
     if( pBox->GetSttNd() )
     {
-        // die Hintergrundfarbe/-grafik bestimmen
+        // Determine background color/graphic
         const HTMLTableCell *pCell = GetCell( nRow, nCol );
         pBoxItem = pCell->GetBoxItem();
         pBGBrushItem = pCell->GetBGBrush();
         if( !pBGBrushItem )
         {
-            // Wenn die Zelle ueber mehrere Zeilen geht muss ein evtl.
-            // an der Zeile gesetzter Hintergrund an die Zelle uebernommen
-            // werden.
-            // Wenn es sich um eine Tabelle in der Tabelle handelt und
-            // die Zelle ueber die gesamte Heoehe der Tabelle geht muss
-            // ebenfalls der Hintergrund der Zeile uebernommen werden, weil
-            // die Line von der GC (zu Recht) wegoptimiert wird.
+            // If a cell spans multiple rows, a background to that row should be copied to the cell.
+            // If it's a table in a table and that cell goes over the whole height of that table,
+            // the row's background has to be copied to the cell aswell,
+            // since the line is gonna be GC-ed (correctly).
             if( nRowSpan > 1 || (this != m_pTopTable && nRowSpan==m_nRows) )
             {
                 pBGBrushItem = (*m_pRows)[nRow]->GetBGBrush();
@@ -1351,16 +1342,16 @@ void HTMLTable::FixFrameFormat( SwTableBox *pBox,
     {
         pFrameFormat = pBox->ClaimFrameFormat();
 
-        // die Breite der Box berechnen
+        // calculate width of the box
         SwTwips nFrameWidth = (SwTwips)m_pLayoutInfo->GetColumn(nCol)
                                                 ->GetRelColWidth();
         for( sal_uInt16 i=1; i<nColSpan; i++ )
             nFrameWidth += (SwTwips)m_pLayoutInfo->GetColumn(nCol+i)
                                              ->GetRelColWidth();
 
-        // die Umrandung nur an Edit-Boxen setzen (bei der oberen und unteren
-        // Umrandung muss beruecks. werden, ob es sich um den ersten oder
-        // letzen Absatz der Zelle handelt)
+        // Only set the border on edit boxes.
+        // On setting the upper and lower border, keep in mind if
+        // it's the first or the last paragraph of the cell
         if( pBox->GetSttNd() )
         {
             bool bSet = (m_nCellPadding > 0);
@@ -1382,9 +1373,7 @@ void HTMLTable::FixFrameFormat( SwTableBox *pBox,
             {
                 if( nEmptyRows && !m_aBorderLine.GetInWidth() )
                 {
-                    // Leere Zeilen koennen zur Zeit nur dann ueber
-                    // dicke Linien simuliert werden, wenn die Linie
-                    // einfach ist.
+                    // For now, empty rows can only be emulated by thick lines, if it's a single line
                     SvxBorderLine aThickBorderLine( m_aBorderLine );
 
                     sal_uInt16 nBorderWidth = m_aBorderLine.GetOutWidth();
@@ -1421,14 +1410,12 @@ void HTMLTable::FixFrameFormat( SwTableBox *pBox,
             }
             else if (bSet)
             {
-                // BorderDist nicht mehr Bestandteil einer Zelle mit fixer Breite
+                // BorderDist is not part of a cell with fixed width
                 sal_uInt16 nBDist = static_cast< sal_uInt16 >(
                     (2*m_nCellPadding <= nInnerFrameWidth) ? m_nCellPadding
                                                       : (nInnerFrameWidth / 2) );
-                // wir setzen das Item nur, wenn es eine Umrandung gibt
-                // oder eine Border-Distanz vorgegeben ist. Fehlt letztere,
-                // dann gibt es eine Umrandung, und wir muessen die Distanz
-                // setzen
+                // We only set the item if there's a border or a border distance
+                // If the latter is missing, there's gonna be a border and we'll have to set the distance
                 aBoxItem.SetAllDistances((nBDist) ? nBDist : MIN_BORDER_DIST);
                 pFrameFormat->SetFormatAttr( aBoxItem );
             }
@@ -1442,7 +1429,7 @@ void HTMLTable::FixFrameFormat( SwTableBox *pBox,
             else
                 pFrameFormat->ResetFormatAttr( RES_BACKGROUND );
 
-            // Format nur setzten, wenn es auch einen Value gibt oder die Box leer ist.
+            // Only set format if there's a value or the box is empty
             if( bHasNumFormat && (bHasValue || IsBoxEmpty(pBox)) )
             {
                 bool bLock = pFrameFormat->GetDoc()->GetNumberFormatter()
@@ -1554,8 +1541,7 @@ SwTableBox *HTMLTable::NewTableBox( const SwStartNode *pStNd,
     if( m_pTopTable->m_pBox1 &&
         m_pTopTable->m_pBox1->GetSttNd() == pStNd )
     {
-        // wenn der StartNode dem StartNode der initial angelegten Box
-        // entspricht nehmen wir diese Box
+        // If the StartNode is the StartNode of the initially created box, we take that box
         pBox = m_pTopTable->m_pBox1;
         pBox->SetUpper( pUpper );
         m_pTopTable->m_pBox1 = nullptr;
@@ -1575,7 +1561,7 @@ static void ResetLineFrameFormatAttrs( SwFrameFormat *pFrameFormat )
             "Zeile hat vertikale Ausrichtung" );
 }
 
-// !!! kann noch vereinfacht werden
+// !!! could be simplified
 SwTableLine *HTMLTable::MakeTableLine( SwTableBox *pUpper,
                                        sal_uInt16 nTopRow, sal_uInt16 nLeftCol,
                                        sal_uInt16 nBottomRow, sal_uInt16 nRightCol )
@@ -1593,16 +1579,14 @@ SwTableLine *HTMLTable::MakeTableLine( SwTableBox *pUpper,
     const SvxBrushItem *pBGBrushItem = nullptr;
     if( this == m_pTopTable || nTopRow>0 || nBottomRow<m_nRows )
     {
-        // An der Line eine Frabe zu setzen macht keinen Sinn, wenn sie
-        // die auesserste und gleichzeitig einzige Zeile einer Tabelle in
-        // der Tabelle ist.
+        // It doesn't make sense to set a color on a line,
+        // if it's the outermost and simultaneously sole line of a table in a table
         pBGBrushItem = pTopRow->GetBGBrush();
 
         if( !pBGBrushItem && this != m_pTopTable )
         {
-            // Ein an einer Tabellen in der Tabelle gesetzter Hintergrund
-            // wird an den Rows gesetzt. Das gilt auch fuer den Hintergrund
-            // der Zelle, in dem die Tabelle vorkommt.
+            // A background that's set on a table in a table is set on the rows.
+            // It's the same for the background of the cell where that table is
             pBGBrushItem = GetBGBrush();
             if( !pBGBrushItem )
                 pBGBrushItem = GetInhBGBrush();
@@ -1615,10 +1599,8 @@ SwTableLine *HTMLTable::MakeTableLine( SwTableBox *pUpper,
 
         if( nRowHeight )
         {
-            // Tabellenhoehe einstellen. Da es sich um eine
-            // Mindesthoehe handelt, kann sie genauso wie in
-            // Netscape berechnet werden, also ohne Beruecksichtigung
-            // der tatsaechlichen Umrandungsbreite.
+            // set table height. Since it's a minimum height it can be calculated like in Netscape,
+            // so without considering the actual border width
             nRowHeight += GetTopCellSpace( nTopRow ) +
                        GetBottomCellSpace( nTopRow, 1 );
 
@@ -1633,8 +1615,7 @@ SwTableLine *HTMLTable::MakeTableLine( SwTableBox *pUpper,
     }
     else if( !m_pLineFrameFormatNoHeight )
     {
-        // sonst muessen wir die Hoehe aus dem Attribut entfernen
-        // und koennen uns das Format merken
+        // else, we'll have to remove the height from the attribute and remember the formatx
         m_pLineFrameFormatNoHeight = static_cast<SwTableLineFormat*>(pLine->ClaimFrameFormat());
 
         ResetLineFrameFormatAttrs( m_pLineFrameFormatNoHeight );
@@ -1662,8 +1643,7 @@ SwTableLine *HTMLTable::MakeTableLine( SwTableBox *pUpper,
                 HTMLTableCell *pCell2 = GetCell( nTopRow, nStartCol );
                 if( pCell2->GetColSpan() == (nCol+1-nStartCol) )
                 {
-                    // Die HTML-Tabellen-Zellen bilden genau eine Box.
-                    // Dann muss hinter der Box gesplittet werden
+                    // The HTML tables represent a box. So we need to split behind that box
                     nSplitCol = nCol + 1;
 
                     long nBoxRowSpan = pCell2->GetRowSpan();
@@ -1685,7 +1665,7 @@ SwTableLine *HTMLTable::MakeTableLine( SwTableBox *pUpper,
                         if( nBoxRowSpan < 0 )
                             pCurrCell->SetRowSpan( 0 );
 
-                        // ggf. COLSPAN beachten
+                        // check COLSPAN if needed
                         for( sal_uInt16 j=nStartCol+1; j<nSplitCol; j++ )
                         {
                             GetCell(nTopRow,j)->SetContents( pCnts );
@@ -1728,10 +1708,10 @@ SwTableBox *HTMLTable::MakeTableBox( SwTableLine *pUpper,
 
     if( !pCnts->Next() )
     {
-        // nur eine Inhalts-Section
+        // just one content section
         if( pCnts->GetStartNode() )
         {
-            // und die ist keine Tabelle
+            // ... that's not a table
             pBox = NewTableBox( pCnts->GetStartNode(), pUpper );
             pCnts->SetTableBox( pBox );
         }
@@ -1739,9 +1719,8 @@ SwTableBox *HTMLTable::MakeTableBox( SwTableLine *pUpper,
         {
             pCnts->GetTable()->InheritVertBorders( this, nLeftCol,
                                                    nRightCol-nLeftCol );
-            // und die ist eine Tabelle: dann bauen wir eine neue
-            // Box und fuegen die Zeilen der Tabelle in die Zeilen
-            // der Box ein
+            // ... that's a table. We'll build a new box and put the rows of the table
+            // in the rows of the box
             pBox = new SwTableBox( m_pBoxFormat, 0, pUpper );
             sal_uInt16 nAbs, nRel;
             m_pLayoutInfo->GetAvail( nLeftCol, nColSpan, nAbs, nRel );
@@ -1754,7 +1733,7 @@ SwTableBox *HTMLTable::MakeTableBox( SwTableLine *pUpper,
     }
     else
     {
-        // mehrere Inhalts Sections: dann brauchen wir eine Box mit Zeilen
+        // multiple content sections: we'll build a box with rows
         pBox = new SwTableBox( m_pBoxFormat, 0, pUpper );
         SwTableLines& rLines = pBox->GetTabLines();
         bool bFirstPara = true;
@@ -1763,14 +1742,13 @@ SwTableBox *HTMLTable::MakeTableBox( SwTableLine *pUpper,
         {
             if( pCnts->GetStartNode() )
             {
-                // normale Absaetze werden zu einer Box in einer Zeile
+                // normal paragraphs are gonna be boxes in a row
                 SwTableLine *pLine =
                     new SwTableLine( m_pLineFrameFormatNoHeight ? m_pLineFrameFormatNoHeight
                                                          : m_pLineFormat, 0, pBox );
                 if( !m_pLineFrameFormatNoHeight )
                 {
-                    // Wenn es noch kein Line-Format ohne Hoehe gibt, koennen
-                    // wir uns dieses her als soleches merken
+                    // If there's no line format without height yet, we can use that one
                     m_pLineFrameFormatNoHeight = static_cast<SwTableLineFormat*>(pLine->ClaimFrameFormat());
 
                     ResetLineFrameFormatAttrs( m_pLineFrameFormatNoHeight );
@@ -1789,7 +1767,7 @@ SwTableBox *HTMLTable::MakeTableBox( SwTableLine *pUpper,
             {
                 pCnts->GetTable()->InheritVertBorders( this, nLeftCol,
                                                        nRightCol-nLeftCol );
-                // Tabellen werden direkt eingetragen
+                // Tables are entered directly
                 sal_uInt16 nAbs, nRel;
                 m_pLayoutInfo->GetAvail( nLeftCol, nColSpan, nAbs, nRel );
                 sal_uInt16 nLSpace = m_pLayoutInfo->GetLeftCellSpace( nLeftCol,
@@ -1819,47 +1797,38 @@ void HTMLTable::InheritBorders( const HTMLTable *pParent,
     OSL_ENSURE( m_nRows>0 && m_nCols>0 && m_nCurrentRow==m_nRows,
             "Wurde CloseTable nicht aufgerufen?" );
 
-    // Die Child-Tabelle muss einen Rahmen bekommen, wenn die umgebende
-    // Zelle einen Rand an der betreffenden Seite besitzt.
-    // Der obere bzw. untere Rand wird nur gesetzt, wenn die Tabelle
-    // ale erster bzw. letzter Absatz in der Zelle vorkommt. Ansonsten
-    // Fuer den linken/rechten Rand kann noch nicht entschieden werden,
-    // ob eine Umrandung der Tabelle noetig/moeglich ist, weil das davon
-    // abhaengt, ob "Filler"-Zellen eingefuegt werden. Hier werden deshalb
-    // erstmal nur Informationen gesammelt
+    // The child table needs a border, if the surrounding cell has a margin on that side.
+    // The upper/lower border is only set if the table is the first/last paragraph in that cell
+    // It can't be determined if a border for that table is needed or possible for the left or right side,
+    // since that's depending on if filler cells are gonna be added. We'll only collect info for now
 
     if( 0==nRow && pParent->m_bTopBorder && bFirstPara )
     {
         m_bTopBorder = true;
-        m_bFillerTopBorder = true; // auch Filler bekommt eine Umrandung
+        m_bFillerTopBorder = true; // fillers get a border too
         m_aTopBorderLine = pParent->m_aTopBorderLine;
     }
     if ((*pParent->m_pRows)[nRow+nRowSpan-1]->bBottomBorder && bLastPara)
     {
         (*m_pRows)[m_nRows-1]->bBottomBorder = true;
-        m_bFillerBottomBorder = true; // auch Filler bekommt eine Umrandung
+        m_bFillerBottomBorder = true; // fillers get a border too
         m_aBottomBorderLine =
             nRow+nRowSpan==pParent->m_nRows ? pParent->m_aBottomBorderLine
                                           : pParent->m_aBorderLine;
     }
 
-    // Die Child Tabelle darf keinen oberen oder linken Rahmen bekommen,
-    // wenn der bereits durch die umgebende Tabelle gesetzt ist.
-    // Sie darf jedoch immer einen oberen Rand bekommen, wenn die Tabelle
-    // nicht der erste Absatz in der Zelle ist.
+    // The child table mustn't get an upper or lower border, if that's already done by the surrounding table
+    // It can get an upper border if the table is not the first paragraph in that cell
     m_bTopAllowed = ( !bFirstPara || (pParent->m_bTopAllowed &&
                  (0==nRow || !((*pParent->m_pRows)[nRow-1])->bBottomBorder)) );
 
-    // die Child-Tabelle muss die Farbe der Zelle erben, in der sie
-    // vorkommt, wenn sie keine eigene besitzt
+    // The child table has to inherit the color of the cell it's contained in, if it doesn't have one
     const SvxBrushItem *pInhBG = pParent->GetCell(nRow,nCol)->GetBGBrush();
     if( !pInhBG && pParent != m_pTopTable &&
         pParent->GetCell(nRow,nCol)->GetRowSpan() == pParent->m_nRows )
     {
-        // die ganze umgebende Tabelle ist eine Tabelle in der Tabelle
-        // und besteht nur aus einer Line, die bei der GC (zu Recht)
-        // wegoptimiert wird. Deshalb muss der Hintergrund der Line in
-        // diese Tabelle uebernommen werden.
+        // the whole surrounding table is a table in a table and consists only of a single line
+        // that's gonna be GC-ed (correctly). That's why the background of that line is copied.
         pInhBG = (*pParent->m_pRows)[nRow]->GetBGBrush();
         if( !pInhBG )
             pInhBG = pParent->GetBGBrush();
@@ -1878,7 +1847,7 @@ void HTMLTable::InheritVertBorders( const HTMLTable *pParent,
 
     if( nCol+nColSpan==pParent->m_nCols && pParent->m_bRightBorder )
     {
-        m_bInheritedRightBorder = true; // erstmal nur merken
+        m_bInheritedRightBorder = true; // just remember for now
         m_aInheritedRightBorderLine = pParent->m_aRightBorderLine;
         nInhRightBorderWidth =
             GetBorderWidth( m_aInheritedRightBorderLine, true ) + MIN_BORDER_DIST;
@@ -1886,7 +1855,7 @@ void HTMLTable::InheritVertBorders( const HTMLTable *pParent,
 
     if (((*pParent->m_pColumns)[nCol])->bLeftBorder)
     {
-        m_bInheritedLeftBorder = true;  // erstmal nur merken
+        m_bInheritedLeftBorder = true;  // just remember for now
         m_aInheritedLeftBorderLine = 0==nCol ? pParent->m_aLeftBorderLine
                                      : pParent->m_aBorderLine;
         nInhLeftBorderWidth =
@@ -2006,7 +1975,7 @@ SvxAdjust HTMLTable::GetInheritedAdjust() const
 
 sal_Int16 HTMLTable::GetInheritedVertOri() const
 {
-    // text::VertOrientation::TOP ist der default!
+    // text::VertOrientation::TOP is default!
     sal_Int16 eVOri = (*m_pRows)[m_nCurrentRow]->GetVertOri();
     if( text::VertOrientation::TOP==eVOri && m_nCurrentColumn<m_nCols )
         eVOri = ((*m_pColumns)[m_nCurrentColumn])->GetVertOri();
@@ -2031,12 +2000,11 @@ void HTMLTable::InsertCell( HTMLTableCnts *pCnts,
     if( !nColSpan || (sal_uInt32)m_nCurrentColumn + nColSpan > USHRT_MAX )
         nColSpan = 1;
 
-    sal_uInt16 nColsReq = m_nCurrentColumn + nColSpan;       // benoetigte Spalten
-    sal_uInt16 nRowsReq = m_nCurrentRow + nRowSpan;       // benoetigte Zeilen
+    sal_uInt16 nColsReq = m_nCurrentColumn + nColSpan;
+    sal_uInt16 nRowsReq = m_nCurrentRow + nRowSpan;
     sal_uInt16 i, j;
 
-    // falls wir mehr Spalten benoetigen als wir zur Zeit haben,
-    // muessen wir in allen Zeilen noch Zellen hinzufuegen
+    // if we need more colums than we currently have, we need to add cells for all rows
     if( m_nCols < nColsReq )
     {
         for( i=m_nCols; i<nColsReq; i++ )
@@ -2050,8 +2018,7 @@ void HTMLTable::InsertCell( HTMLTableCnts *pCnts,
     if( nColsReq > m_nFilledColumns )
         m_nFilledColumns = nColsReq;
 
-    // falls wir mehr Zeilen benoetigen als wir zur Zeit haben,
-    // muessen wir noch neue Zeilen hinzufuegen
+    // if we need more rows than we currently have, we need to add cells
     if( m_nRows < nRowsReq )
     {
         for( i=m_nRows; i<nRowsReq; i++ )
@@ -2060,8 +2027,7 @@ void HTMLTable::InsertCell( HTMLTableCnts *pCnts,
         OSL_ENSURE(m_nRows == m_pRows->size(), "wrong number of rows in Insert");
     }
 
-    // Testen, ob eine Ueberschneidung vorliegt und diese
-    // gegebenfalls beseitigen
+    // Check if we have an overlap and could remove that
     sal_uInt16 nSpanedCols = 0;
     if( m_nCurrentRow>0 )
     {
@@ -2071,11 +2037,9 @@ void HTMLTable::InsertCell( HTMLTableCnts *pCnts,
             HTMLTableCell *pCell = pCurRow->GetCell(i);
             if( pCell->GetContents() )
             {
-                // Der Inhalt reicht von einer weiter oben stehenden Zelle
-                // hier herein. Inhalt und Farbe der Zelle sind deshalb in
-                // jedem Fall noch dort verankert und koennen deshalb
-                // ueberschrieben werden bzw. von ProtectRowSpan geloescht
-                // (Inhalt) oder kopiert (Farbe) werden.
+                // A cell from a row further above overlaps this one.
+                // Content and colors are coming from that cell and can be overwritten
+                // or deleted (content) or copied (color) by ProtectRowSpan
                 nSpanedCols = i + pCell->GetColSpan();
                 FixRowSpan( m_nCurrentRow-1, i, pCell->GetContents() );
                 if( pCell->GetRowSpan() > nRowSpan )
@@ -2085,8 +2049,7 @@ void HTMLTable::InsertCell( HTMLTableCnts *pCnts,
         }
         for( i=nColsReq; i<nSpanedCols; i++ )
         {
-            // Auch diese Inhalte sind in jedem Fall nich in der Zeile
-            // darueber verankert.
+            // These contents are anchored in the row above in any case
             HTMLTableCell *pCell = pCurRow->GetCell(i);
             FixRowSpan( m_nCurrentRow-1, i, pCell->GetContents() );
             ProtectRowSpan( m_nCurrentRow, i, pCell->GetRowSpan() );
@@ -2112,32 +2075,32 @@ void HTMLTable::InsertCell( HTMLTableCnts *pCnts,
                     ->PixelToLogic( aTwipSz, MapMode( MapUnit::MapTwip ) );
     }
 
-    // die Breite nur in die erste Zelle setzen!
+    // Only set width on the first cell!
     if( nCellWidth )
     {
         sal_uInt16 nTmp = bRelWidth ? nCellWidth : (sal_uInt16)aTwipSz.Width();
         GetCell( m_nCurrentRow, m_nCurrentColumn )->SetWidth( nTmp, bRelWidth );
     }
 
-    // Ausserdem noch die Hoehe merken
+    // Remember height
     if( nCellHeight && 1==nRowSpan )
     {
         (*m_pRows)[m_nCurrentRow]->SetHeight(static_cast<sal_uInt16>(aTwipSz.Height()));
     }
 
-    // den Spaltenzaehler hinter die neuen Zellen setzen
+    // Set the column counter behind the new cells
     m_nCurrentColumn = nColsReq;
     if( nSpanedCols > m_nCurrentColumn )
         m_nCurrentColumn = nSpanedCols;
 
-    // und die naechste freie Zelle suchen
+    // and search for the next free cell
     while( m_nCurrentColumn<m_nCols && GetCell(m_nCurrentRow,m_nCurrentColumn)->IsUsed() )
         m_nCurrentColumn++;
 }
 
 inline void HTMLTable::CloseSection( bool bHead )
 {
-    // die vorhergende Section beenden, falls es schon eine Zeile gibt
+    // Close the preceding sections if there's already a row
     OSL_ENSURE( m_nCurrentRow<=m_nRows, "ungeultige aktuelle Zeile" );
     if( m_nCurrentRow>0 && m_nCurrentRow<=m_nRows )
         (*m_pRows)[m_nCurrentRow-1]->SetEndOfGroup();
@@ -2148,9 +2111,9 @@ inline void HTMLTable::CloseSection( bool bHead )
 void HTMLTable::OpenRow( SvxAdjust eAdjust, sal_Int16 eVertOrient,
                          SvxBrushItem *pBGBrushItem )
 {
-    sal_uInt16 nRowsReq = m_nCurrentRow+1;    // Anzahl benoetigter Zeilen;
+    sal_uInt16 nRowsReq = m_nCurrentRow+1;
 
-    // die naechste Zeile anlegen, falls sie nicht schon da ist
+    // create the next row if it's not there already
     if( m_nRows<nRowsReq )
     {
         for( sal_uInt16 i=m_nRows; i<nRowsReq; i++ )
@@ -2166,10 +2129,10 @@ void HTMLTable::OpenRow( SvxAdjust eAdjust, sal_Int16 eVertOrient,
     if( pBGBrushItem )
         (*m_pRows)[m_nCurrentRow]->SetBGBrush( pBGBrushItem );
 
-    // den Spaltenzaehler wieder an den Anfang setzen
+    // reset the column counter
     m_nCurrentColumn=0;
 
-    // und die naechste freie Zelle suchen
+    // and search for the next free cell
     while( m_nCurrentColumn<m_nCols && GetCell(m_nCurrentRow,m_nCurrentColumn)->IsUsed() )
         m_nCurrentColumn++;
 }
@@ -2178,7 +2141,7 @@ void HTMLTable::CloseRow( bool bEmpty )
 {
     OSL_ENSURE( m_nCurrentRow<m_nRows, "aktulle Zeile hinter dem Tabellenende" );
 
-    // leere Zellen bekommen einfach einen etwas dickeren unteren Rand!
+    // empty cells just get a slightly thicker lower border!
     if( bEmpty )
     {
         if( m_nCurrentRow > 0 )
@@ -2188,10 +2151,8 @@ void HTMLTable::CloseRow( bool bEmpty )
 
     HTMLTableRow *const pRow = (*m_pRows)[m_nCurrentRow].get();
 
-    // den COLSPAN aller leeren Zellen am Zeilenende so anpassen, dass
-    // eine Zelle daraus wird. Das kann man hier machen (und auf keinen
-    // Fall frueher), weill jetzt keine Zellen mehr in die Zeile eingefuegt
-    // werden.
+    // modify the COLSPAN of all empty cells at the row end in a way, that they're forming a single cell
+    // that can be done here (and not earlier) since there's no more cells in that row
     sal_uInt16 i=m_nCols;
     while( i )
     {
@@ -2233,7 +2194,7 @@ void HTMLTable::InsertCol( sal_uInt16 nSpan, sal_uInt16 nColWidth, bool bRelWidt
     if( !nSpan )
         nSpan = 1;
 
-    sal_uInt16 nColsReq = m_nCurrentColumn + nSpan;      // benoetigte Spalten
+    sal_uInt16 nColsReq = m_nCurrentColumn + nSpan;
 
     if( m_nCols < nColsReq )
     {
@@ -2267,11 +2228,9 @@ void HTMLTable::CloseTable()
 {
     sal_uInt16 i;
 
-    // Die Anzahl der Tabellenzeilen richtet sich nur nach den
-    // <TR>-Elementen (d.h. nach nCurRow). Durch ROWSPAN aufgespannte
-    // Zeilen hinter Zeile nCurRow muessen wir deshalb loeschen
-    // und vor allem aber den ROWSPAN in den darueberliegenden Zeilen
-    // anpassen.
+    // The number of table rows is only dependent on the <TR> elements (i.e. nCurRow).
+    // Rows that are spanned via ROWSPAN behind nCurRow need to be deleted
+    // and we need to adjust the ROWSPAN in the rows above
     if( m_nRows>m_nCurrentRow )
     {
         HTMLTableRow *const pPrevRow = (*m_pRows)[m_nCurrentRow-1].get();
@@ -2287,7 +2246,7 @@ void HTMLTable::CloseTable()
         m_nRows = m_nCurrentRow;
     }
 
-    // falls die Tabelle keine Spalte hat, muessen wir eine hinzufuegen
+    // if the table has no column, we need to add one
     if( 0==m_nCols )
     {
         m_pColumns->push_back(o3tl::make_unique<HTMLTableColumn>());
@@ -2297,7 +2256,7 @@ void HTMLTable::CloseTable()
         m_nFilledColumns = 1;
     }
 
-    // falls die Tabelle keine Zeile hat, muessen wir eine hinzufuegen
+    // if the table has no row, we need to add one
     if( 0==m_nRows )
     {
         m_pRows->push_back(o3tl::make_unique<HTMLTableRow>(m_nCols));
@@ -2319,8 +2278,6 @@ void HTMLTable::MakeTable_( SwTableBox *pBox )
     SwTableLines& rLines = (pBox ? pBox->GetTabLines()
                                  : const_cast<SwTable *>(m_pSwTable)->GetTabLines() );
 
-    // jetzt geht's richtig los ...
-
     for( sal_uInt16 i=0; i<m_nRows; i++ )
     {
         SwTableLine *pLine = MakeTableLine( pBox, i, 0, i+1, m_nCols );
@@ -2371,35 +2328,32 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
 
     if( this==m_pTopTable )
     {
-        // Umrandung der Tabelle und aller in ihr enthaltenen berechnen
+        // Calculate borders of the table and all contained tables
         SetBorders();
 
-        // Schritt 1: Die benoetigten Layout-Strukturen werden angelegt
-        // (inklusive Tabellen in Tabellen).
+        // Step 1: needed layout structures are created (including tables in tables)
         CreateLayoutInfo();
 
-        // Schritt 2: Die minimalen und maximalen Spaltenbreiten werden
-        // berechnet (inklusive Tabellen in Tabellen). Da wir noch keine
-        // Boxen haben, arabeiten wir noch auf den Start-Nodes.
+        // Step 2: the mininal and maximal column width is calculated
+        // (including tables in tables). Since we don't have boxes yet,
+        // we'll work on the start nodes
         m_pLayoutInfo->AutoLayoutPass1();
     }
 
-    // Schritt 3: Die tatsaechlichen Spaltenbreiten dieser Tabelle werden
-    // berechnet (nicht von Tabellen in Tabellen). Dies muss jetzt schon
-    // sein, damit wir entscheiden koennen ob Filler-Zellen benoetigt werden
-    // oder nicht (deshalb war auch Pass1 schon noetig).
+    // Step 3: the actual column widths of this table are calculated (not tables in tables)
+    // We need this now to decide if we need filler cells
+    // (Pass1 was needed because of this aswell)
     m_pLayoutInfo->AutoLayoutPass2( nAbsAvail, nRelAvail, nAbsLeftSpace,
                                   nAbsRightSpace, nInhAbsSpace );
 
     if( this!=m_pTopTable )
     {
-        // die linke und rechte Umrandung der Tabelle kann jetzt entgueltig
-        // festgelegt werden
+        // the right and left border of this table can be finally defined
         if( m_pLayoutInfo->GetRelRightFill() == 0 )
         {
             if( !m_bRightBorder )
             {
-                // linke Umrandung von auesserer Tabelle uebernehmen
+                // inherit left border of the outer table
                 if( m_bInheritedRightBorder )
                 {
                     m_bRightBorder = true;
@@ -2408,7 +2362,7 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
             }
             else
             {
-                // Umrandung nur setzen, wenn es erlaubt ist
+                // Only set border if allowed
                 m_bRightBorder = m_bRightAllowed;
             }
         }
@@ -2417,47 +2371,43 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
             !((*m_pColumns)[0])->bLeftBorder &&
             m_bInheritedLeftBorder )
         {
-            // ggf. rechte Umrandung von auesserer Tabelle uebernehmen
+            // If applicable, inherit right border of outer table
             ((*m_pColumns)[0])->bLeftBorder = true;
             m_aLeftBorderLine = m_aInheritedLeftBorderLine;
         }
     }
 
-    // Fuer die Top-Table muss die Ausrichtung gesetzt werden
+    // Set adjustment for the top table
     if( this==m_pTopTable )
     {
         sal_Int16 eHoriOri;
         if( m_bForceFrame )
         {
-            // Die Tabelle soll in einen Rahmen und ist auch schmaler
-            // als der verfuegbare Platz und nicht 100% breit.
-            // Dann kommt sie in einen Rahmen
+            // The table should go in a text frame and it's narrower than the
+            // available space and not 100% wide. So it gets a border
             eHoriOri = m_bPrcWidth ? text::HoriOrientation::FULL : text::HoriOrientation::LEFT;
         }
         else switch( m_eTableAdjust )
         {
-            // Die Tabelle passt entweder auf die Seite, soll aber in keinen
-            // Rahmen oder sie ist Breiter als die Seite und soll deshalb
-            // in keinen Rahmen
+            // The table either fits the page but shouldn't get a text frame,
+            // or it's wider than the page so it doesn't need a text frame
 
         case SVX_ADJUST_RIGHT:
-            // in rechtsbuendigen Tabellen kann nicht auf den rechten
-            // Rand Ruecksicht genommen werden
+            // Don't be considerate of the right margin in right-adjusted tables
             eHoriOri = text::HoriOrientation::RIGHT;
             break;
         case SVX_ADJUST_CENTER:
-            // zentrierte Tabellen nehmen keine Ruecksicht auf Raender!
+            // Centred tables are not considerate of margins
             eHoriOri = text::HoriOrientation::CENTER;
             break;
         case SVX_ADJUST_LEFT:
         default:
-            // linksbuendige Tabellen nehmen nur auf den linken Rand
-            // Ruecksicht
+            // left-adjusted tables are only considerate of the left margin
             eHoriOri = m_nLeftMargin ? text::HoriOrientation::LEFT_AND_WIDTH : text::HoriOrientation::LEFT;
             break;
         }
 
-        // das Tabellenform holen und anpassen
+        // get the table format and adapt it
         SwFrameFormat *pFrameFormat = m_pSwTable->GetFrameFormat();
         pFrameFormat->SetFormatAttr( SwFormatHoriOrient(0,eHoriOri) );
         if( text::HoriOrientation::LEFT_AND_WIDTH==eHoriOri )
@@ -2482,10 +2432,10 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
         }
     }
 
-    // die Default Line- und Box-Formate holen
+    // get the default line and box format
     if( this==m_pTopTable )
     {
-        // die erste Box merken und aus der ersten Zeile ausketten
+        // remember the first box and unlist it from the first row
         SwTableLine *pLine1 = (m_pSwTable->GetTabLines())[0];
         m_pBox1 = (pLine1->GetTabBoxes())[0];
         pLine1->GetTabBoxes().erase(pLine1->GetTabBoxes().begin());
@@ -2499,8 +2449,7 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
         m_pBoxFormat = m_pTopTable->m_pBoxFormat;
     }
 
-    // ggf. muessen fuer Tabellen in Tabellen "Filler"-Zellen eingefuegt
-    // werden
+    // If applicable, add filler cells for tables in tables
     if( this != m_pTopTable &&
         ( m_pLayoutInfo->GetRelLeftFill() > 0  ||
           m_pLayoutInfo->GetRelRightFill() > 0 ) )
@@ -2509,17 +2458,17 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
 
         SwTableLines& rLines = pBox->GetTabLines();
 
-        // dazu brauchen wir erstmal ein eine neue Table-Line in der Box
+        // first, we need a new table line in the box
         SwTableLine *pLine =
             new SwTableLine( m_pLineFrameFormatNoHeight ? m_pLineFrameFormatNoHeight
                                                  : m_pLineFormat, 0, pBox );
         rLines.push_back( pLine );
 
-        // Sicherstellen, dass wie ein Format ohne Hoehe erwischt haben
+        // Check that we have a format without height
         if( !m_pLineFrameFormatNoHeight )
         {
-            // sonst muessen wir die Hoehe aus dem Attribut entfernen
-            // und koennen uns das Format merken
+            // Otherwise, we need to remove the height from the attributes
+            // and remember that format
             m_pLineFrameFormatNoHeight = static_cast<SwTableLineFormat*>(pLine->ClaimFrameFormat());
 
             ResetLineFrameFormatAttrs( m_pLineFrameFormatNoHeight );
@@ -2528,11 +2477,11 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
         SwTableBoxes& rBoxes = pLine->GetTabBoxes();
         SwTableBox *pNewBox;
 
-        // ggf. links eine Zelle einfuegen
+        // If applicable, add a cell to the left
         if( m_pLayoutInfo->GetRelLeftFill() > 0 )
         {
-            // pPrevStNd ist der Vorgaenger-Start-Node der Tabelle. Den
-            // "Filler"-Node fuegen wir einfach dahinter ein ...
+            // pPrevStNd is the predecessor start node of the table
+            // We'll add the filler node just behind
             m_pPrevStartNode = m_pParser->InsertTableSection( m_pPrevStartNode );
 
             pNewBox = NewTableBox( m_pPrevStartNode, pLine );
@@ -2541,7 +2490,7 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
             m_pLayoutInfo->SetLeftFillerBox( pNewBox );
         }
 
-        // jetzt die Tabelle bearbeiten
+        // modify the table now
         pNewBox = new SwTableBox( m_pBoxFormat, 0, pLine );
         rBoxes.push_back( pNewBox );
 
@@ -2553,7 +2502,7 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
 
         MakeTable_( pNewBox );
 
-        // und noch ggf. rechts eine Zelle einfuegen
+        // and add a cell to the right if applicable
         if( m_pLayoutInfo->GetRelRightFill() > 0 )
         {
             const SwStartNode *pStNd =
@@ -2572,16 +2521,14 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
         MakeTable_( pBox );
     }
 
-    // zum Schluss fuehren wir noch eine Garbage-Collection fuer die
-    // Top-Level-Tabelle durch
+    // Finally, we'll do a garbage collection for the top level table
     if( this==m_pTopTable )
     {
         if( 1==m_nRows && m_nHeight && 1==m_pSwTable->GetTabLines().size() )
         {
-            // Hoehe einer einzeiligen Tabelle als Mindesthoehe der
-            // Zeile setzen. (War mal fixe Hoehe, aber das gibt manchmal
-            // Probleme (fix #34972#) und ist auch nicht Netscape 4.0
-            // konform
+            // Set height of a one-row table as the minimum width of the row
+            // Was originally a fixed height, but that made problems
+            // and is not Netscape 4.0 compliant
             m_nHeight = SwHTMLParser::ToTwips( m_nHeight );
             if( m_nHeight < MINLAY )
                 m_nHeight = MINLAY;
@@ -2604,10 +2551,9 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
             if (eAdjust != SVX_ADJUST_LEFT &&
                 eAdjust != SVX_ADJUST_RIGHT)
             {
-                // Wenn eine Tabelle ohne Breitenangabe nicht links oder
-                // rechts umflossen werden soll, dann stacken wir sie
-                // in einem Rahmen mit 100%-Breite, damit ihre Groesse
-                // angepasst wird. Der Rahmen darf nicht angepasst werden.
+                // If a table with a width attribute isn't flowed around left or right
+                // we'll stack it with a border of 100% width, so its size will
+                // be adapted. That text frame mustn't be modified
                 OSL_ENSURE( HasToFly(), "Warum ist die Tabelle in einem Rahmen?" );
                 sal_uInt32 nMin = m_pLayoutInfo->GetMin();
                 if( nMin > USHRT_MAX )
@@ -2619,9 +2565,8 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
             }
             else
             {
-                // Links und rechts ausgerichtete Tabellen ohne Breite
-                // duerfen leider nicht in der Breite angepasst werden, denn
-                // sie wuerden nur schrumpfen aber nie wachsen.
+                // left or right adjusted table without width mustn't be adjusted in width
+                // as they would only shrink but never grow
                 m_pLayoutInfo->SetMustNotRecalc( true );
                 if( m_pContext->GetFrameFormat()->GetAnchor().GetContentAnchor()
                     ->nNode.GetNode().FindTableNode() )
@@ -2641,8 +2586,7 @@ void HTMLTable::MakeTable( SwTableBox *pBox, sal_uInt16 nAbsAvail,
         }
         m_pLayoutInfo->SetMayBeInFlyFrame( bIsInFlyFrame );
 
-        // Nur Tabellen mit relativer Breite oder ohne Breite muessen
-        // angepasst werden.
+        // Only tables with relative width or without width should be modifed
         m_pLayoutInfo->SetMustResize( m_bPrcWidth || !m_nWidth );
 
         m_pLayoutInfo->SetWidths();
@@ -2855,15 +2799,12 @@ sal_Int32 SwHTMLParser::StripTrailingLF()
             {
                 if( nLFCount > 2 )
                 {
-                    // Bei Netscape entspricht ein Absatz-Ende zwei LFs
-                    // (mit einem kommt man in die naechste Zeile, das
-                    // zweite erzeugt eine Leerzeile) Diesen Abstand
-                    // erreichen wie aber schon mit dem unteren
-                    // Absatz-Abstand. Wenn nach den <BR> ein neuer
-                    // Absatz aufgemacht wird, wird das Maximum des Abstands,
-                    // der sich aus den BR und dem P ergibt genommen.
-                    // Deshalb muessen wir 2 bzw. alle bei weniger
-                    // als zweien loeschen
+                    // On Netscape, a paragraph end matches 2 LFs
+                    // (1 is just a newline, 2 creates a blank line)
+                    // We already have this space with the lower paragraph gap
+                    // If there's a paragraph after the <BR>, we take the maximum
+                    // of the gap that results from the <BR> and <P>
+                    // That's why we need to delete 2 respectively all if less than 2
                     nLFCount = 2;
                 }
 
@@ -2958,7 +2899,7 @@ SectionSaveStruct::SectionSaveStruct( SwHTMLParser& rParser ) :
     m_nDefListDeepSave(0), m_nContextStMinSave(0), m_nContextStAttrMinSave(0),
     m_pTable( nullptr )
 {
-    // Font-Stacks einfrieren
+    // Freeze font stacks
     m_nBaseFontStMinSave = rParser.m_nBaseFontStMin;
     rParser.m_nBaseFontStMin = rParser.m_aBaseFontStack.size();
 
@@ -2966,20 +2907,20 @@ SectionSaveStruct::SectionSaveStruct( SwHTMLParser& rParser ) :
     m_nFontStHeadStartSave = rParser.m_nFontStHeadStart;
     rParser.m_nFontStMin = rParser.m_aFontStack.size();
 
-    // Kontext-Stack einfrieren
+    // Freeze context stack
     m_nContextStMinSave = rParser.m_nContextStMin;
     m_nContextStAttrMinSave = rParser.m_nContextStAttrMin;
     rParser.m_nContextStMin = rParser.m_aContexts.size();
     rParser.m_nContextStAttrMin = rParser.m_nContextStMin;
 
-    // und noch ein par Zaehler retten
+    // And remember a few counters
     m_nDefListDeepSave = rParser.m_nDefListDeep;
     rParser.m_nDefListDeep = 0;
 }
 
 void SectionSaveStruct::Restore( SwHTMLParser& rParser )
 {
-    // Font-Stacks wieder auftauen
+    // Unfreeze font stacks
     sal_uInt16 nMin = rParser.m_nBaseFontStMin;
     if( rParser.m_aBaseFontStack.size() > nMin )
         rParser.m_aBaseFontStack.erase( rParser.m_aBaseFontStack.begin() + nMin,
@@ -2999,10 +2940,10 @@ void SectionSaveStruct::Restore( SwHTMLParser& rParser )
     rParser.m_nContextStMin = m_nContextStMinSave;
     rParser.m_nContextStAttrMin = m_nContextStAttrMinSave;
 
-    // und noch ein par Zaehler rekonstruieren
+    // Reconstruct a few counters
     rParser.m_nDefListDeep = m_nDefListDeepSave;
 
-    // und ein par Flags zuruecksetzen
+    // Reset a few flags
     rParser.m_bNoParSpace = false;
     rParser.m_nOpenParaToken = 0;
 
@@ -3017,16 +2958,16 @@ class CellSaveStruct : public SectionSaveStruct
     Color m_aBGColor;
     std::shared_ptr<SvxBoxItem> m_pBoxItem;
 
-    HTMLTableCnts* m_pCnts;           // Liste aller Inhalte
-    HTMLTableCnts* m_pCurrCnts;   // der aktuelle Inhalt oder 0
-    std::unique_ptr<SwNodeIndex> m_pNoBreakEndNodeIndex;// Absatz-Index eines </NOBR>
+    HTMLTableCnts* m_pCnts;           // List of all contents
+    HTMLTableCnts* m_pCurrCnts;   // current content or 0
+    std::unique_ptr<SwNodeIndex> m_pNoBreakEndNodeIndex;// Paragraph index of a <NOBR>
 
     double m_nValue;
 
     sal_uInt32 m_nNumFormat;
 
     sal_uInt16 m_nRowSpan, m_nColSpan, m_nWidth, m_nHeight;
-    sal_Int32 m_nNoBreakEndContentPos;     // Zeichen-Index eines </NOBR>
+    sal_Int32 m_nNoBreakEndContentPos;     // Character index of a <NOBR>
 
     SvxAdjust m_eAdjust;
     sal_Int16 m_eVertOri;
@@ -3036,8 +2977,8 @@ class CellSaveStruct : public SectionSaveStruct
     bool m_bHasNumFormat : 1;
     bool m_bHasValue : 1;
     bool m_bBGColor : 1;
-    bool m_bNoWrap : 1;       // NOWRAP-Option
-    bool m_bNoBreak : 1;      // NOBREAK-Tag
+    bool m_bNoWrap : 1;       // NOWRAP option
+    bool m_bNoBreak : 1;      // NOBREAK tag
 
 public:
 
@@ -3110,19 +3051,19 @@ CellSaveStruct::CellSaveStruct( SwHTMLParser& rParser, HTMLTable *pCurTable,
                                         aHTMLTableVAlignTable, m_eVertOri );
                 break;
             case HTML_O_WIDTH:
-                m_nWidth = (sal_uInt16)rOption.GetNumber();   // nur fuer Netscape
+                m_nWidth = (sal_uInt16)rOption.GetNumber();   // Just for Netscape
                 m_bPrcWidth = (rOption.GetString().indexOf('%') != -1);
                 if( m_bPrcWidth && m_nWidth>100 )
                     m_nWidth = 100;
                 break;
             case HTML_O_HEIGHT:
-                m_nHeight = (sal_uInt16)rOption.GetNumber();  // nur fuer Netscape
+                m_nHeight = (sal_uInt16)rOption.GetNumber();  // Just for Netscape
                 if( rOption.GetString().indexOf('%') != -1)
-                    m_nHeight = 0;    // keine %-Angaben beruecksichtigen
+                    m_nHeight = 0;    // don't consider % attributes
                 break;
             case HTML_O_BGCOLOR:
-                // Leere BGCOLOR bei <TABLE>, <TR> und <TD>/<TH> wie Netscape
-                // ignorieren, bei allen anderen Tags *wirklich* nicht.
+                // Ignore empty BGCOLOR on <TABLE>, <TR> and <TD>/<TH> like Netscape
+                // *really* not on other tags
                 if( !rOption.GetString().isEmpty() )
                 {
                     rOption.GetColor( m_aBGColor );
@@ -3170,9 +3111,8 @@ CellSaveStruct::CellSaveStruct( SwHTMLParser& rParser, HTMLTable *pCurTable,
                             *rParser.m_xDoc->GetNumberFormatter() );
     }
 
-    // einen neuen Kontext anlegen, aber das drawing::Alignment-Attribut
-    // nicht dort verankern, weil es noch ger keine Section gibt, in der
-    // es gibt.
+    // Create a new context but don't anchor the drawing::Alignment attribute there,
+    // since there's no section yet
     sal_uInt16 nToken, nColl;
     if( m_bHead )
     {
@@ -3227,20 +3167,18 @@ void CellSaveStruct::InsertCell( SwHTMLParser& rParser,
                                   HTMLTable *pCurTable )
 {
 #if OSL_DEBUG_LEVEL > 0
-    // Die Attribute muessen schon beim Auefrauemen des Kontext-Stacks
-    // entfernt worden sein, sonst ist etwas schiefgelaufen. Das
-    // Checken wir mal eben ...
-    // MIB 8.1.98: Wenn ausserhalb einer Zelle Attribute geoeffnet
-    // wurden stehen diese noch in der Attribut-Tabelle und werden erst
-    // ganz zum Schluss durch die CleanContext-Aufrufe in BuildTable
-    // geloescht. Damit es in diesem Fall keine Asserts gibt findet dann
-    // keine Ueberpruefung statt. Erkennen tut man diesen Fall an
-    // nContextStAttrMin: Der gemerkte Wert nContextStAttrMinSave ist der
-    // Wert, den nContextStAttrMin beim Start der Tabelle hatte. Und
-    // der aktuelle Wert von nContextStAttrMin entspricht der Anzahl der
-    // Kontexte, die beim Start der Zelle vorgefunden wurden. Sind beide
-    // Werte unterschiedlich, wurden ausserhalb der Zelle Kontexte
-    // angelegt und wir ueberpruefen nichts.
+    // The attributes need to have been removed when tidying up the context stack,
+    // Otherwise something's wrong. Let's check that...
+
+    // MIB 8.1.98: When attributes were opened outside of a cell,
+    // they're still in the attribut table and will only be deleted at the end
+    // through the CleanContext calls in BuildTable. We don't check that there
+    // so that we get no assert [violations, by translator]
+    // We can see this on nContextStAttrMin: the remembered value of nContextStAttrMinSave
+    // is the value that nContextStAttrMin had at the start of the table. And the
+    // current value of nContextStAttrMin corresponds to the number of contexts
+    // we found at the start of the cell. If the values differ, contexts
+    // were created and we don't check anything.
 
     if( rParser.m_nContextStAttrMin == GetContextStAttrMin() )
     {
@@ -3254,7 +3192,7 @@ void CellSaveStruct::InsertCell( SwHTMLParser& rParser,
     }
 #endif
 
-    // jetzt muessen wir noch die Zelle an der aktuellen Position einfuegen
+    // we need to add the cell on the current position
     SvxBrushItem *pBrushItem =
         rParser.CreateBrushItem( m_bBGColor ? &m_aBGColor : nullptr, m_aBGImage,
                                  m_aStyle, m_aId, m_aClass );
@@ -3293,7 +3231,7 @@ void CellSaveStruct::CheckNoBreak( const SwPosition& rPos, SwDoc * /*pDoc*/ )
     {
         if( m_bNoBreak )
         {
-            // <NOBR> wurde nicht beendet
+            // <NOBR> wasn't closed
             m_pCnts->SetNoBreak();
         }
         else if( m_pNoBreakEndNodeIndex &&
@@ -3301,7 +3239,7 @@ void CellSaveStruct::CheckNoBreak( const SwPosition& rPos, SwDoc * /*pDoc*/ )
         {
             if( m_nNoBreakEndContentPos == rPos.nContent.GetIndex() )
             {
-                // <NOBR> wurde unmittelbar vor dem Zellen-Ende beendet
+                // <NOBR> was closed immediately before the cell end
                 m_pCnts->SetNoBreak();
             }
             else if( m_nNoBreakEndContentPos + 1 == rPos.nContent.GetIndex() )
@@ -3313,8 +3251,7 @@ void CellSaveStruct::CheckNoBreak( const SwPosition& rPos, SwDoc * /*pDoc*/ )
                             pTextNd->GetText()[m_nNoBreakEndContentPos];
                     if( ' '==cLast || '\x0a'==cLast )
                     {
-                        // Zwischem dem </NOBR> und dem Zellen-Ende gibt es nur
-                        // ein Blank oder einen Zeilenumbruch.
+                        // There's just a blank or a newline between the <NOBR> and the cell end
                         m_pCnts->SetNoBreak();
                     }
                 }
@@ -3326,20 +3263,20 @@ void CellSaveStruct::CheckNoBreak( const SwPosition& rPos, SwDoc * /*pDoc*/ )
 HTMLTableCnts *SwHTMLParser::InsertTableContents(
                                         bool bHead )
 {
-    // eine neue Section anlegen, der PaM steht dann darin
+    // create a new section, the PaM is gonna be there
     const SwStartNode *pStNd =
         InsertTableSection( static_cast< sal_uInt16 >(bHead ? RES_POOLCOLL_TABLE_HDLN
                                            : RES_POOLCOLL_TABLE) );
 
     if( GetNumInfo().GetNumRule() )
     {
-        // 1. Absatz auf nicht numeriert setzen
+        // Set the first paragraph to non-enumerated
         sal_uInt8 nLvl = GetNumInfo().GetLevel();
 
         SetNodeNum( nLvl );
     }
 
-    // Attributierungs-Anfang neu setzen
+    // Reset attributation start
     const SwNodeIndex& rSttPara = m_pPam->GetPoint()->nNode;
     sal_Int32 nSttCnt = m_pPam->GetPoint()->nContent.GetIndex();
 
@@ -3397,7 +3334,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
     }
     else
     {
-        // <TH> bzw. <TD> wurde bereits gelesen
+        // <TH> resp. <TD> were already read
         if( m_pTable->IsOverflowing() )
         {
             SaveState( 0 );
@@ -3408,8 +3345,8 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
         {
             bool bTopTable = m_pTable==pCurTable;
 
-            // die Tabelle besitzt noch keinen Inhalt, d.h. die eigentliche
-            // Tabelle muss erst noch angelegt werden
+            // the table has no content yet, this means the actual table needs
+            // to be created first
 
             static sal_uInt16 aWhichIds[] =
             {
@@ -3450,13 +3387,12 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                 }
             }
 
-            // Den linken/rechten Absatzeinzug ermitteln
             sal_uInt16 nLeftSpace = 0;
             sal_uInt16 nRightSpace = 0;
             short nIndent;
             GetMarginsFromContextWithNumBul( nLeftSpace, nRightSpace, nIndent );
 
-            // die aktuelle Position an die wir irgendwann zurueckkehren
+            // save the current position we'll get back to some time
             SwPosition *pSavePos = nullptr;
             bool bForceFrame = false;
             bool bAppended = false;
@@ -3465,32 +3401,29 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             {
                 SvxAdjust eTableAdjust = m_pTable->GetTableAdjust(false);
 
-                // Wenn die Tabelle links oder rechts ausgerivchtet ist,
-                // oder in einen Rahmen soll, dann kommt sie auch in einen
-                // solchen.
+                // If the table is left or right adjusted or should be in a text frame,
+                // it'll get one
                 bForceFrame = eTableAdjust == SVX_ADJUST_LEFT ||
                               eTableAdjust == SVX_ADJUST_RIGHT ||
                               pCurTable->HasToFly();
 
-                // Entweder kommt die Tabelle in keinen Rahmen und befindet
-                // sich in keinem Rahmen (wird also durch Zellen simuliert),
-                // oder es gibt bereits Inhalt an der entsprechenden Stelle.
+                // The table either shouldn't get in a text frame and isn't in one
+                // (it gets simulated through cells),
+                // or there's already content at that position
                 OSL_ENSURE( !bForceFrame || pCurTable->HasParentSection(),
                         "Tabelle im Rahmen hat keine Umgebung!" );
 
                 bool bAppend = false;
                 if( bForceFrame )
                 {
-                    // Wenn die Tabelle in einen Rahmen kommt, muss
-                    // nur ein neuer Absatz aufgemacht werden, wenn
-                    // der Absatz Rahmen ohne Umlauf enthaelt.
+                    // If the table gets in a border, we only need to open a new
+                    //paragraph if the paragraph has text frames that don't fly
                     bAppend = HasCurrentParaFlys(true);
                 }
                 else
                 {
-                    // Sonst muss ein neuer Absatz aufgemacht werden,
-                    // wenn der Absatz nicht leer ist, oder Rahmen
-                    // oder text::Bookmarks enthaelt.
+                    // Otherwise, we need to open a new paragraph if the paragraph
+                    // is empty or contains text frames or bookmarks
                     bAppend =
                         m_pPam->GetPoint()->nContent.GetIndex() ||
                         HasCurrentParaFlys() ||
@@ -3521,9 +3454,9 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
 
                         pTmp = new HTMLAttr( *m_pPam->GetPoint(),
                                             SvxULSpaceItem( 0, 0, RES_UL_SPACE ) );
-                        m_aSetAttrTab.push_front( pTmp ); // ja, 0, weil schon
-                                                        // vom Tabellenende vorher
-                                                        // was gesetzt sein kann.
+                        m_aSetAttrTab.push_front( pTmp ); // Position 0, since
+                                                          // something can be set by
+                                                          // the table end before
                     }
                     AppendTextNode( AM_NOSPACE );
                     bAppended = true;
@@ -3532,9 +3465,8 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                 {
                     if( !bForceFrame )
                     {
-                        // Der Absatz wird gleich hinter die Tabelle
-                        // verschoben. Deshalb entfernen wir alle harten
-                        // Attribute des Absatzes
+                        // The paragraph will be moved right behind the table.
+                        // That's why we remove all hard attributes of that paragraph
 
                         for(HTMLAttr* i : m_aParaAttrs)
                             i->Invalidate();
@@ -3549,33 +3481,30 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             {
                 bParentLFStripped = StripTrailingLF() > 0;
 
-                // Absaetze bzw. ueberschriften beeenden
+                // Close paragraph resp. headers
                 m_nOpenParaToken = 0;
                 m_nFontStHeadStart = m_nFontStMin;
 
-                // die harten Attribute an diesem Absatz werden nie mehr ungueltig
+                // The hard attributes on that paragraph are never gonna be invalid anymore
                 if( !m_aParaAttrs.empty() )
                     m_aParaAttrs.clear();
             }
 
-            // einen Tabellen Kontext anlegen
+            // create a table context
             HTMLTableContext *pTCntxt =
                         new HTMLTableContext( pSavePos, m_nContextStMin,
                                                m_nContextStAttrMin );
 
-            // alle noch offenen Attribute beenden und hinter der Tabelle
-            // neu aufspannen
+            // end all open attributes and open them again behind the table
             HTMLAttrs *pPostIts = nullptr;
             if( !bForceFrame && (bTopTable || pCurTable->HasParentSection()) )
             {
                 SplitAttrTab( pTCntxt->aAttrTab, bTopTable );
-                // Wenn wir einen schon vorhandenen Absatz verwenden, duerfen
-                // in den keine PostIts eingefuegt werden, weil der Absatz
-                // ja hinter die Tabelle wandert. Sie werden deshalb in den
-                // ersten Absatz der Tabelle verschoben.
-                // Bei Tabellen in Tabellen duerfen ebenfalls keine PostIts
-                // in einen noch leeren Absatz eingefuegt werden, weil
-                // der sonat nicht geloescht wird.
+                // If we reuse a already existing paragraph, we can't add
+                // PostIts since the paragraph gets behind that table.
+                // They're gonna be moved into the first paragraph of the table
+                // If we have tables in tables, we also can't add PostIts to a
+                // still empty paragraph, since it's not gonna be deleted that way
                 if( (bTopTable && !bAppended) ||
                     (!bTopTable && !bParentLFStripped &&
                      !m_pPam->GetPoint()->nContent.GetIndex()) )
@@ -3593,7 +3522,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             }
             m_bNoParSpace = false;
 
-            // Aktuelle Numerierung retten und auschalten.
+            // Save current numbering and turn it off
             pTCntxt->SetNumInfo( GetNumInfo() );
             GetNumInfo().Clear();
             pTCntxt->SavePREListingXMP( *this );
@@ -3602,7 +3531,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             {
                 if( bForceFrame )
                 {
-                    // Die Tabelle soll in einen Rahmen geschaufelt werden.
+                    // the table should be put in a text frame
 
                     SfxItemSet aFrameSet( m_xDoc->GetAttrPool(),
                                         RES_FRMATR_BEGIN, RES_FRMATR_END-1 );
@@ -3658,10 +3587,9 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
 
                 }
 
-                // eine SwTable mit einer Box anlegen und den PaM in den
-                // Inhalt der Box-Section bewegen (der Ausrichtungs-Parameter
-                // ist erstmal nur ein Dummy und wird spaeter noch richtig
-                // gesetzt)
+                // create a SwTable with a box and set the PaM to the content of
+                // the box section (the adjustment parameter is a dummy for now
+                // and will be corrected later)
                 OSL_ENSURE( !m_pPam->GetPoint()->nContent.GetIndex(),
                         "Der Absatz hinter der Tabelle ist nicht leer!" );
                 const SwTable* pSwTable = m_xDoc->InsertTable(
@@ -3719,8 +3647,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
 
                 if( !bAppended && pPostIts )
                 {
-                    // noch vorhandene PostIts in den ersten Absatz
-                    // der Tabelle setzen
+                    // set still-existing PostIts to the first paragraph of the table
                     InsertAttrs( *pPostIts );
                     delete pPostIts;
                     pPostIts = nullptr;
@@ -3736,22 +3663,20 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             }
             else
             {
-                // noch offene Bereiche muessen noch entfernt werden
+                // still open sections need to be deleted
                 if( EndSections( bParentLFStripped ) )
                     bParentLFStripped = false;
 
                 if( pCurTable->HasParentSection() )
                 {
-                    // dannach entfernen wir ein ggf. zu viel vorhandenen
-                    // leeren Absatz, aber nur, wenn er schon vor dem
-                    // entfernen von LFs leer war
+                    // after that, we remove a possibly redundant empty paragraph,
+                    // but only if it was empty before we stripped the LFs
                     if( !bParentLFStripped )
                         StripTrailingPara();
 
                     if( pPostIts )
                     {
-                        // noch vorhandene PostIts an das Ende des jetzt
-                        // aktuellen Absatzes schieben
+                        // move still existing PostIts to the end of the current paragraph
                         InsertAttrs( *pPostIts );
                         delete pPostIts;
                         pPostIts = nullptr;
@@ -3765,10 +3690,9 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                 pCurTable->SetTable( pStNd, pTCntxt, nLeftSpace, nRightSpace );
             }
 
-            // Den Kontext-Stack einfrieren, denn es koennen auch mal
-            // irgendwo ausserhalb von Zellen Attribute gesetzt werden.
-            // Darf nicht frueher passieren, weil eventuell noch im
-            // Stack gesucht wird!!!
+            // Freeze the context stack, since there could be attributes set
+            // outside of cells. Can't happen earlier, since there may be
+            // searches in the stack
             m_nContextStMin = m_aContexts.size();
             m_nContextStAttrMin = m_nContextStMin;
         }
@@ -3776,13 +3700,12 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
         pSaveStruct = new CellSaveStruct( *this, pCurTable, bHead,
                                             bReadOptions );
 
-        // ist beim ersten GetNextToken schon pending, muss bei
-        // wiederaufsetzen auf jedenfall neu gelesen werden!
+        // If the first GetNextToken() doesn't succeed (pending input), must re-read from the beginning.
         SaveState( 0 );
     }
 
     if( !nToken )
-        nToken = GetNextToken();    // Token nach <TABLE>
+        nToken = GetNextToken();    // Token after <TABLE>
 
     bool bDone = false;
     while( (IsParserWorking() && !bDone) || bPending )
@@ -3795,8 +3718,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                 "Wo ist die Section gebieben?" );
         if( !m_pPendStack && m_bCallNextToken && pSaveStruct->IsInSection() )
         {
-            // NextToken direkt aufrufen (z.B. um den Inhalt von
-            // Floating-Frames oder Applets zu ignorieren)
+            // Call NextToken directly (e.g. ignore the content of floating frames or applets)
             NextToken( nToken );
         }
         else switch( nToken )
@@ -3824,17 +3746,15 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                 SvxAdjust eTabAdjust = SVX_ADJUST_END;
                 if( !m_pPendStack )
                 {
-                    // nur wenn eine neue Tabelle aufgemacht wird, aber
-                    // nicht wenn nach einem Pending in der Tabelle
-                    // weitergelesen wird!
+                    // only if we create a new table, but not if we're still
+                    // reading in the table after a Pending
                     pSaveStruct->m_pTable = m_pTable;
 
-                    // HACK: Eine Section fuer eine Tabelle anlegen, die
-                    // in einen Rahmen kommt.
+                    // HACK: create a section for a table that goes in a text frame
                     if( !pSaveStruct->IsInSection() )
                     {
-                        // Diese Schleife muss vorwartes sein, weil die
-                        // erste Option immer gewinnt.
+                        // The loop needs to be forward, since the
+                        // first option always wins
                         bool bNeedsSection = false;
                         const HTMLOptions& rHTMLOptions = GetOptions();
                         for (const auto & rOption : rHTMLOptions)
@@ -3857,12 +3777,12 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                     }
                     else
                     {
-                        // Wenn im aktuellen Absatz Flys verankert sind,
-                        // muss die neue Tabelle in einen Rahmen.
+                        // If Flys are anchored in the current paragraph,
+                        // the table needs to get in a text frame
                         bHasToFly = HasCurrentParaFlys(false,true);
                     }
 
-                    // in der Zelle kann sich ein Bereich befinden!
+                    // There could be a section in the cell
                     eTabAdjust = m_aAttrTab.pAdjust
                         ? static_cast<const SvxAdjustItem&>(m_aAttrTab.pAdjust->GetItem()).
                                                  GetAdjust()
@@ -3875,7 +3795,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                                                    bHasToFly );
                 if( SvParserState::Pending != GetStatus() )
                 {
-                    // nur wenn die Tabelle wirklich zu Ende ist!
+                    // Only if the table is really complete
                     if( pSubTable )
                     {
                         OSL_ENSURE( pSubTable->GetTableAdjust(false)!= SVX_ADJUST_LEFT &&
@@ -3889,8 +3809,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                             OSL_ENSURE( !pSaveStruct->IsInSection(),
                                     "Wo ist die Section geblieben" );
 
-                            // Wenn jetzt keine Tabelle kommt haben wir eine
-                            // Section
+                            // If there's no table coming, we have a section
                             pSaveStruct->AddContents( pParentContents );
                         }
 
@@ -3917,18 +3836,17 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
                                     new HTMLTableCnts(pCapStNd) );
                             }
 
-                            // Jetzt haben wir keine Section mehr
+                            // We don't have a section anymore
                             pSaveStruct->ClearIsInSection();
                         }
                         else if( pCapStNd )
                         {
-                            // Da wir diese Section nicht mehr loeschen
-                            // koennen (sie koeente zur erster Box
-                            // gehoeren), fuegen wir sie ein.
+                            // Since we can't delete this section (it might
+                            // belong to the first box), we'll add it
                             pSaveStruct->AddContents(
                                 new HTMLTableCnts(pCapStNd) );
 
-                            // Jetzt haben wir keine Section mehr
+                            // We don't have a section anymore
                             pSaveStruct->ClearIsInSection();
                         }
                     }
@@ -3939,7 +3857,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             break;
 
         case HTML_NOBR_ON:
-            // HACK fuer MS: Steht das <NOBR> zu beginn der Zelle?
+            // HACK for MS: Is the <NOBR> at the start of the cell?
             pSaveStruct->StartNoBreak( *m_pPam->GetPoint() );
             break;
 
@@ -3948,16 +3866,15 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             break;
 
         case HTML_COMMENT:
-            // Mit Kommentar-Feldern werden Spaces nicht mehr geloescht
-            // ausserdem wollen wir fuer einen Kommentar keine neue Zelle
-            // anlegen !!!
+            // Spaces are not gonna be deleted with comment fields,
+            // and we don't want a new cell for a comment
             NextToken( nToken );
             break;
 
         case HTML_MARQUEE_ON:
             if( !pSaveStruct->IsInSection() )
             {
-                // eine neue Section anlegen, der PaM steht dann darin
+                // create a new section, the PaM is gonna be there
                 pSaveStruct->AddContents(
                     InsertTableContents( bHead ) );
             }
@@ -3966,7 +3883,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             break;
 
         case HTML_TEXTTOKEN:
-            // keine Section fuer einen leeren String anlegen
+            // Don't add a section for an empty string
             if( !pSaveStruct->IsInSection() && 1==aToken.getLength() &&
                 ' '==aToken[0] )
                 break;
@@ -3974,7 +3891,7 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
         default:
             if( !pSaveStruct->IsInSection() )
             {
-                // eine neue Section anlegen, der PaM steht dann darin
+                // add a new section, the PaM's gonna be there
                 pSaveStruct->AddContents(
                     InsertTableContents( bHead ) );
             }
@@ -4003,11 +3920,10 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
         return;
     }
 
-    // Falls der Inhalt der Zelle leer war, muessen wir noch einen
-    // leeren Inhalt anlegen. Ausserdem legen wir einen leeren Inhalt
-    // an, wenn die Zelle mit einer Tabelle aufgehoert hat und keine
-    // COL-Tags hatte (sonst wurde sie wahrscheinlich von uns exportiert,
-    // und dann wollen wir natuerlich keinen zusaetzlichen Absatz haben).
+    // If the content of the cell was empty, we need to create an epty content
+    // We also create an empty content if the cell ended with a table and had no
+    // COL tags. Otherwise, it was probably exported by us and we don't
+    // want to have an additional paragraph
     if( !pSaveStruct->GetFirstContents() ||
         (!pSaveStruct->IsInSection() && !pCurTable->HasColTags()) )
     {
@@ -4036,9 +3952,8 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
     {
         pSaveStruct->CheckNoBreak( *m_pPam->GetPoint(), m_xDoc.get() );
 
-        // Alle noch offenen Kontexte beenden. Wir nehmen hier
-        // AttrMin, weil nContxtStMin evtl. veraendert wurde.
-        // Da es durch EndContext wieder restauriert wird, geht das.
+        // End all open contexts. We'll take AttrMin because nContextStMin might
+        // have been modified. Since it's gonna be restored by EndContext, it's okay
         while( m_aContexts.size() > m_nContextStAttrMin+1 )
         {
             HTMLAttrContext *pCntxt = PopContext();
@@ -4046,19 +3961,18 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
             delete pCntxt;
         }
 
-        // LFs am Absatz-Ende entfernen
+        // Remove LFs at the paragraph end
         if( StripTrailingLF()==0 && !m_pPam->GetPoint()->nContent.GetIndex() )
             StripTrailingPara();
 
-        // falls fuer die Zelle eine Ausrichtung gesetzt wurde, muessen
-        // wir die beenden
+        // If there was an adjustment set for the cell, we need to close it
         HTMLAttrContext *pCntxt = PopContext();
         EndContext( pCntxt );
         delete pCntxt;
     }
     else
     {
-        // Alle noch offenen Kontexte beenden
+        // Close all still open contexts
         while( m_aContexts.size() > m_nContextStAttrMin )
         {
             HTMLAttrContext *pCntxt = PopContext();
@@ -4067,14 +3981,14 @@ void SwHTMLParser::BuildTableCell( HTMLTable *pCurTable, bool bReadOptions,
         }
     }
 
-    // auch eine Numerierung muss beendet werden
+    // end an enumeration
     GetNumInfo().Clear();
 
     SetAttr( false );
 
     pSaveStruct->InsertCell( *this, pCurTable );
 
-    // wir stehen jetzt (wahrschenlich) vor <TH>, <TD>, <TR> oder </TABLE>
+    // we're probably before a <TH>, <TD>, <TR> or </TABLE>
     delete pSaveStruct;
 }
 
@@ -4094,7 +4008,7 @@ void SwHTMLParser::BuildTableRow( HTMLTable *pCurTable, bool bReadOptions,
                                   SvxAdjust eGrpAdjust,
                                   sal_Int16 eGrpVertOri )
 {
-    // <TR> wurde bereist gelesen
+    // <TR> was already read
 
     if( !IsParserWorking() && !m_pPendStack )
         return;
@@ -4144,8 +4058,8 @@ void SwHTMLParser::BuildTableRow( HTMLTable *pCurTable, bool bReadOptions,
                                     aHTMLTableVAlignTable, eVertOri );
                     break;
                 case HTML_O_BGCOLOR:
-                    // Leere BGCOLOR bei <TABLE>, <TR> und <TD>/<TH> wie Netsc.
-                    // ignorieren, bei allen anderen Tags *wirklich* nicht.
+                    // Ignore empty BGCOLOR on <TABLE>, <TR> and <TD>/>TH> like Netscape
+                    // *really* not on other tags
                     if( !rOption.GetString().isEmpty() )
                     {
                         rOption.GetColor( aBGColor );
@@ -4172,13 +4086,12 @@ void SwHTMLParser::BuildTableRow( HTMLTable *pCurTable, bool bReadOptions,
             CreateBrushItem( bBGColor ? &aBGColor : nullptr, aBGImage, aStyle,
                              aId, aClass );
         pCurTable->OpenRow( eAdjust, eVertOri, pBrushItem );
-        // ist beim ersten GetNextToken schon pending, muss bei
-        // wiederaufsetzen auf jedenfall neu gelesen werden!
+        // If the first GetNextToken() doesn't succeed (pending input), must re-read from the beginning.
         SaveState( 0 );
     }
 
     if( !nToken )
-        nToken = GetNextToken();    // naechstes Token
+        nToken = GetNextToken();
 
     bool bDone = false;
     while( (IsParserWorking() && !bDone) || bPending )
@@ -4193,8 +4106,7 @@ void SwHTMLParser::BuildTableRow( HTMLTable *pCurTable, bool bReadOptions,
         if( !m_pPendStack && m_bCallNextToken &&
             (pCurTable->GetContext() || pCurTable->HasParentSection()) )
         {
-            // NextToken direkt aufrufen (z.B. um den Inhalt von
-            // Floating-Frames oder Applets zu ignorieren)
+            /// Call NextToken directly (e.g. ignore the content of floating frames or applets)
             NextToken( nToken );
         }
         else switch( nToken )
@@ -4240,25 +4152,23 @@ void SwHTMLParser::BuildTableRow( HTMLTable *pCurTable, bool bReadOptions,
         case HTML_COLGROUP_OFF:
         case HTML_COL_ON:
         case HTML_COL_OFF:
-            // wo keine Zelle anfing kann auch keine aufhoehren, oder?
-            // und die ganzen anderen Tokens haben hier auch nicht zu
-            // suchen und machen nur die Tabelle kaputt
+            // Where no cell started, there can't be a cell ending
+            // all the other tokens are bogus anyway and only break the table
             break;
         case HTML_MULTICOL_ON:
-            // spaltige Rahmen koennen wir hier leider nicht einguegen
+            // we can't add columned text frames here
             break;
         case HTML_FORM_ON:
-            NewForm( false );   // keinen neuen Absatz aufmachen!
+            NewForm( false );   // don't create a new paragraph
             break;
         case HTML_FORM_OFF:
-            EndForm( false );   // keinen neuen Absatz aufmachen!
+            EndForm( false );   // don't create a new paragraph
             break;
         case HTML_COMMENT:
             NextToken( nToken );
             break;
         case HTML_MAP_ON:
-            // eine Image-Map fuegt nichts ein, deshalb koennen wir sie
-            // problemlos auch ohne Zelle parsen
+            // an image map doesn't add anything, so we can parse it without a cell
             NextToken( nToken );
             break;
         case HTML_TEXTTOKEN:
@@ -4294,14 +4204,14 @@ void SwHTMLParser::BuildTableRow( HTMLTable *pCurTable, bool bReadOptions,
         delete pSaveStruct;
     }
 
-    // wir stehen jetzt (wahrscheinlich) vor <TR> oder </TABLE>
+    // we're probably before <TR> or </TABLE>
 }
 
 void SwHTMLParser::BuildTableSection( HTMLTable *pCurTable,
                                       bool bReadOptions,
                                       bool bHead )
 {
-    // <THEAD>, <TBODY> bzw. <TFOOT> wurde bereits gelesen
+    // <THEAD>, <TBODY> resp. <TFOOT> were read already
     if( !IsParserWorking() && !m_pPendStack )
         return;
 
@@ -4350,13 +4260,12 @@ void SwHTMLParser::BuildTableSection( HTMLTable *pCurTable,
             }
         }
 
-        // ist beim ersten GetNextToken schon pending, muss bei
-        // wiederaufsetzen auf jedenfall neu gelesen werden!
+        // If the first GetNextToken() doesn't succeed (pending input), must re-read from the beginning.
         SaveState( 0 );
     }
 
     if( !nToken )
-        nToken = GetNextToken();    // naechstes Token
+        nToken = GetNextToken();
 
     bool bDone = false;
     while( (IsParserWorking() && !bDone) || bPending )
@@ -4371,8 +4280,7 @@ void SwHTMLParser::BuildTableSection( HTMLTable *pCurTable,
         if( !m_pPendStack && m_bCallNextToken &&
             (pCurTable->GetContext() || pCurTable->HasParentSection()) )
         {
-            // NextToken direkt aufrufen (z.B. um den Inhalt von
-            // Floating-Frames oder Applets zu ignorieren)
+            // Call NextToken directly (e.g. ignore the content of floating frames or applets)
             NextToken( nToken );
         }
         else switch( nToken )
@@ -4415,16 +4323,16 @@ void SwHTMLParser::BuildTableSection( HTMLTable *pCurTable,
             bDone = m_pTable->IsOverflowing();
             break;
         case HTML_MULTICOL_ON:
-            // spaltige Rahmen koennen wir hier leider nicht einguegen
+            // we can't add columned text frames here
             break;
         case HTML_FORM_ON:
-            NewForm( false );   // keinen neuen Absatz aufmachen!
+            NewForm( false );   // don't create a new paragraph
             break;
         case HTML_FORM_OFF:
-            EndForm( false );   // keinen neuen Absatz aufmachen!
+            EndForm( false );   // don't create a new paragraph
             break;
         case HTML_TEXTTOKEN:
-            // Blank-Strings sind Folge von CR+LF und kein Text
+            // blank strings may be a series of CR+LF and no text
             if( (pCurTable->GetContext() ||
                  !pCurTable->HasParentSection()) &&
                 1==aToken.getLength() && ' ' == aToken[0] )
@@ -4488,7 +4396,7 @@ inline void TableColGrpSaveStruct::CloseColGroup( HTMLTable *pTable )
 void SwHTMLParser::BuildTableColGroup( HTMLTable *pCurTable,
                                        bool bReadOptions )
 {
-    // <COLGROUP> wurde bereits gelesen, wenn bReadOptions
+    // <COLGROUP> was read already if bReadOptions is set
 
     if( !IsParserWorking() && !m_pPendStack )
         return;
@@ -4545,8 +4453,7 @@ void SwHTMLParser::BuildTableColGroup( HTMLTable *pCurTable,
                 }
             }
         }
-        // ist beim ersten GetNextToken schon pending, muss bei
-        // wiederaufsetzen auf jedenfall neu gelesen werden!
+        // If the first GetNextToken() doesn't succeed (pending input), must re-read from the beginning.
         SaveState( 0 );
     }
 
@@ -4566,8 +4473,7 @@ void SwHTMLParser::BuildTableColGroup( HTMLTable *pCurTable,
         if( !m_pPendStack && m_bCallNextToken &&
             (pCurTable->GetContext() || pCurTable->HasParentSection()) )
         {
-            // NextToken direkt aufrufen (z.B. um den Inhalt von
-            // Floating-Frames oder Applets zu ignorieren)
+            // Call NextToken directly (e.g. ignore the content of floating frames or applets)
             NextToken( nToken );
         }
         else switch( nToken )
@@ -4631,15 +4537,14 @@ void SwHTMLParser::BuildTableColGroup( HTMLTable *pCurTable,
                 pCurTable->InsertCol( nColSpan, nColWidth, bRelColWidth,
                                       eColAdjust, eColVertOri );
 
-                // die Angaben in <COLGRP> sollen ignoriert werden, wenn
-                // <COL>-Elemente existieren
+                // the attributes in <COLGRP> should be ignored, if there are <COL> elements
                 pSaveStruct->nColGrpSpan = 0;
             }
             break;
         case HTML_COL_OFF:
-            break;      // Ignorieren
+            break;      // Ignore
         case HTML_MULTICOL_ON:
-            // spaltige Rahmen koennen wir hier leider nicht einguegen
+            // we can't add columned text frames here
             break;
         case HTML_TEXTTOKEN:
             if( (pCurTable->GetContext() ||
@@ -4677,19 +4582,18 @@ void SwHTMLParser::BuildTableColGroup( HTMLTable *pCurTable,
 class CaptionSaveStruct : public SectionSaveStruct
 {
     SwPosition aSavePos;
-    SwHTMLNumRuleInfo aNumRuleInfo; // gueltige Numerierung
+    SwHTMLNumRuleInfo aNumRuleInfo; // valid numbering
 
 public:
 
-    HTMLAttrTable aAttrTab;        // und die Attribute
+    HTMLAttrTable aAttrTab;        // attributes
 
     CaptionSaveStruct( SwHTMLParser& rParser, const SwPosition& rPos ) :
         SectionSaveStruct( rParser ), aSavePos( rPos )
     {
         rParser.SaveAttrTab( aAttrTab );
 
-        // Die aktuelle Numerierung wurde gerettet und muss nur
-        // noch beendet werden.
+        // The current numbering was remembered and just needs to be closed
         aNumRuleInfo.Set( rParser.GetNumInfo() );
         rParser.GetNumInfo().Clear();
     }
@@ -4698,20 +4602,20 @@ public:
 
     void RestoreAll( SwHTMLParser& rParser )
     {
-        // Die alten Stack wiederherstellen
+        // Recover the old stack
         Restore( rParser );
 
-        // Die alte Attribut-Tabelle wiederherstellen
+        // Recover the old attribute tables
         rParser.RestoreAttrTab( aAttrTab );
 
-        // Die alte Numerierung wieder aufspannen
+        // Re-open the old numbering
         rParser.GetNumInfo().Set( aNumRuleInfo );
     }
 };
 
 void SwHTMLParser::BuildTableCaption( HTMLTable *pCurTable )
 {
-    // <CAPTION> wurde bereits gelesen
+    // <CAPTION> was read already
 
     if( !IsParserWorking() && !m_pPendStack )
         return;
@@ -4754,11 +4658,11 @@ void SwHTMLParser::BuildTableCaption( HTMLTable *pCurTable )
             }
         }
 
-        // Alte PaM-Position retten.
+        // Remember old PaM position
         pSaveStruct = new CaptionSaveStruct( *this, *m_pPam->GetPoint() );
 
-        // Eine Text-Section im Icons-Bereich als Container fuer die
-        // Ueberschrift anlegen und PaM dort reinstellen.
+        // Add a text section in the icon section as a container for the header
+        // and set the PaM there
         const SwStartNode *pStNd;
         if( m_pTable == pCurTable )
             pStNd = InsertTempTableCaptionSection();
@@ -4767,7 +4671,7 @@ void SwHTMLParser::BuildTableCaption( HTMLTable *pCurTable )
 
         HTMLAttrContext *pCntxt = new HTMLAttrContext( HTML_CAPTION_ON );
 
-        // Tabellen-Ueberschriften sind immer zentriert.
+        // Table headers are always centered
         NewAttr( &m_aAttrTab.pAdjust, SvxAdjustItem(SVX_ADJUST_CENTER, RES_PARATR_ADJUST) );
 
         HTMLAttrs &rAttrs = pCntxt->GetAttrs();
@@ -4775,18 +4679,17 @@ void SwHTMLParser::BuildTableCaption( HTMLTable *pCurTable )
 
         PushContext( pCntxt );
 
-        // StartNode der Section an der Tabelle merken.
+        // Remember the start node of the section at the table
         pCurTable->SetCaption( pStNd, bTop );
 
-        // ist beim ersten GetNextToken schon pending, muss bei
-        // wiederaufsetzen auf jedenfall neu gelesen werden!
+        // If the first GetNextToken() doesn't succeed (pending input), must re-read from the beginning.
         SaveState( 0 );
     }
 
     if( !nToken )
-        nToken = GetNextToken();    // naechstes Token
+        nToken = GetNextToken();
 
-    // </CAPTION> wird laut DTD benoetigt
+    // </CAPTION> is needed according to DTD
     bool bDone = false;
     while( IsParserWorking() && !bDone )
     {
@@ -4863,37 +4766,33 @@ void SwHTMLParser::BuildTableCaption( HTMLTable *pCurTable )
         delete pCntxt;
     }
 
-    // LF am Absatz-Ende entfernen
     bool bLFStripped = StripTrailingLF() > 0;
 
     if( m_pTable==pCurTable )
     {
-        // Beim spaeteren verschieben der Beschriftung vor oder hinter
-        // die Tabelle wird der letzte Absatz nicht mitverschoben.
-        // Deshalb muss sich am Ende der Section immer ein leerer
-        // Absatz befinden.
+        // On moving the caption later, the last paragraph isn't moved aswell.
+        // That means, there has to be an empty paragraph at the end of the section
         if( m_pPam->GetPoint()->nContent.GetIndex() || bLFStripped )
             AppendTextNode( AM_NOSPACE );
     }
     else
     {
-        // LFs am Absatz-Ende entfernen
+        // Strip LFs at the end of the paragraph
         if( !m_pPam->GetPoint()->nContent.GetIndex() && !bLFStripped )
             StripTrailingPara();
     }
 
-    // falls fuer die Zelle eine Ausrichtung gesetzt wurde, muessen
-    // wir die beenden
+    // If there's an adjustment for the cell, we need to close it
     HTMLAttrContext *pCntxt = PopContext();
     EndContext( pCntxt );
     delete pCntxt;
 
     SetAttr( false );
 
-    // Stacks und Attribut-Tabelle wiederherstellen
+    // Recover stack and attribute table
     pSaveStruct->RestoreAll( *this );
 
-    // PaM wiederherstellen.
+    // Recover PaM
     *m_pPam->GetPoint() = pSaveStruct->GetPos();
 
     delete pSaveStruct;
@@ -4908,9 +4807,8 @@ public:
         m_pCurrentTable( pCurTable )
     {}
 
-    // Aufbau der Tabelle anstossen und die Tabelle ggf. in einen
-    // Rahmen packen. Wenn true zurueckgegeben wird muss noch ein
-    // Absatz eingefuegt werden!
+    // Initiate creation of the table and put the table in a text frame if
+    // needed. If it returns true, we need to insert a paragraph.
     void MakeTable( sal_uInt16 nWidth, SwPosition& rPos, SwDoc *pDoc );
 };
 
@@ -4926,8 +4824,7 @@ void TableSaveStruct::MakeTable( sal_uInt16 nWidth, SwPosition& rPos, SwDoc *pDo
 
     if( pDoc->getIDocumentLayoutAccess().GetCurrentViewShell() && pTableNd )
     {
-        // Existiert schon ein Layout, dann muss an dieser Tabelle die
-        // BoxFrames neu erzeugt werden.
+        // If there's already a layout, the BoxFrames need to be regenerated at this table
 
         if( pTCntxt->GetFrameFormat() )
         {
@@ -4985,7 +4882,7 @@ HTMLTableOptions::HTMLTableOptions( const HTMLOptions& rOptions,
         case HTML_O_HEIGHT:
             nHeight = (sal_uInt16)rOption.GetNumber();
             if( rOption.GetString().indexOf('%') != -1 )
-                nHeight = 0;    // keine %-Anagben benutzen!!!
+                nHeight = 0;    // don't use % attributes
             break;
         case HTML_O_CELLPADDING:
             nCellPadding = (sal_uInt16)rOption.GetNumber();
@@ -5007,7 +4904,7 @@ HTMLTableOptions::HTMLTableOptions( const HTMLOptions& rOptions,
             eVertOri = rOption.GetEnum( aHTMLTableVAlignTable, eVertOri );
             break;
         case HTML_O_BORDER:
-            // BORDER und BORDER=BORDER wie BORDER=1 behandeln
+            // Handle BORDER and BORDER=BORDER like BORDER=1
             if (!rOption.GetString().isEmpty() &&
                 !rOption.GetString().equalsIgnoreAsciiCase(
                         OOO_STRING_SVTOOLS_HTML_O_border))
@@ -5031,8 +4928,8 @@ HTMLTableOptions::HTMLTableOptions( const HTMLOptions& rOptions,
             bHasRules = true;
             break;
         case HTML_O_BGCOLOR:
-            // Leere BGCOLOR bei <TABLE>, <TR> und <TD>/<TH> wie Netscape
-            // ignorieren, bei allen anderen Tags *wirklich* nicht.
+            // Ignore empty BGCOLOR on <TABLE>, <TR> and <TD>/<TH> like Netscape
+            // *really* not on other tags
             if( !rOption.GetString().isEmpty() )
             {
                 rOption.GetColor( aBGColor );
@@ -5074,8 +4971,7 @@ HTMLTableOptions::HTMLTableOptions( const HTMLOptions& rOptions,
         bPrcWidth = true;
     }
 
-    // Wenn BORDER=0 oder kein BORDER gegeben ist, daan darf es auch
-    // keine Umrandung geben
+    // If BORDER=0 or no BORDER given, then there shouldn't be a border
     if( 0==nBorder || USHRT_MAX==nBorder )
     {
         eFrame = HTML_TF_VOID;
@@ -5128,16 +5024,15 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
 
         delete pTableOptions;
 
-        // ist beim ersten GetNextToken schon pending, muss bei
-        // wiederaufsetzen auf jedenfall neu gelesen werden!
+        // Is pending on the first GetNextToken, needs to be re-read on each construction
         SaveState( 0 );
     }
 
     HTMLTable *pCurTable = pSaveStruct->m_pCurrentTable;
 
-    // </TABLE> wird laut DTD benoetigt
+    // </TABLE> is needed according to DTD
     if( !nToken )
-        nToken = GetNextToken();    // naechstes Token
+        nToken = GetNextToken();
 
     bool bDone = false;
     while( (IsParserWorking() && !bDone) || bPending )
@@ -5152,8 +5047,7 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
         if( !m_pPendStack && m_bCallNextToken &&
             (pCurTable->GetContext() || pCurTable->HasParentSection()) )
         {
-            // NextToken direkt aufrufen (z.B. um den Inhalt von
-            // Floating-Frames oder Applets zu ignorieren)
+            /// Call NextToken directly (e.g. ignore the content of floating frames or applets)
             NextToken( nToken );
         }
         else switch( nToken )
@@ -5161,8 +5055,7 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
         case HTML_TABLE_ON:
             if( !pCurTable->GetContext() )
             {
-                // Wenn noch keine Tabelle eingefuegt wurde,
-                // die naechste Tabelle lesen
+                // If there's no table added, read the next table'
                 SkipToken();
                 bDone = true;
             }
@@ -5196,16 +5089,16 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
             bDone = m_pTable->IsOverflowing();
             break;
         case HTML_MULTICOL_ON:
-            // spaltige Rahmen koennen wir hier leider nicht einguegen
+            // we can't add columned text frames here
             break;
         case HTML_FORM_ON:
-            NewForm( false );   // keinen neuen Absatz aufmachen!
+            NewForm( false );   // don't add a new paragraph
             break;
         case HTML_FORM_OFF:
-            EndForm( false );   // keinen neuen Absatz aufmachen!
+            EndForm( false );   // don't add a new paragraph
             break;
         case HTML_TEXTTOKEN:
-            // Blank-Strings sind u. U. eine Folge von CR+LF und kein Text
+            // blank strings may be a series of CR+LF and no text
             if( (pCurTable->GetContext() ||
                  !pCurTable->HasParentSection()) &&
                 1==aToken.getLength() && ' '==aToken[0] )
@@ -5237,14 +5130,12 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
     HTMLTableContext *pTCntxt = pCurTable->GetContext();
     if( pTCntxt )
     {
-        // Die Tabelle wurde auch angelegt
 
-        // Tabellen-Struktur anpassen
+        // Modify table structure
         pCurTable->CloseTable();
 
-        // ausserhalb von Zellen begonnene Kontexte beenden
-        // muss vor(!) dem Umsetzten der Attribut Tabelle existieren,
-        // weil die aktuelle danach nicht mehr existiert
+        // end contexts that began out of cells. Needs to exist before (!) we move the table,
+        // since the current one doesn't exist anymore afterwards
         while( m_aContexts.size() > m_nContextStAttrMin )
         {
             HTMLAttrContext *pCntxt = PopContext();
@@ -5257,16 +5148,16 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
 
         if( m_pTable==pCurTable )
         {
-            // Tabellen-Beschriftung setzen
+            // Set table caption
             const SwStartNode *pCapStNd = m_pTable->GetCaptionStartNode();
             if( pCapStNd )
             {
-                // Der letzte Absatz der Section wird nie mitkopiert. Deshalb
-                // muss die Section mindestens zwei Absaetze enthalten.
+                // The last paragraph of the section is never part of the copy.
+                // That's why the section needs to contain at least two paragraphs
 
                 if( pCapStNd->EndOfSectionIndex() - pCapStNd->GetIndex() > 2 )
                 {
-                    // Start-Node und letzten Absatz nicht mitkopieren.
+                    // Don't copy start node and the last paragraph
                     SwNodeRange aSrcRg( *pCapStNd, 1,
                                     *pCapStNd->EndOfSectionNode(), -1 );
 
@@ -5287,11 +5178,10 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
                     m_xDoc->getIDocumentContentOperations().MoveNodeRange( aSrcRg, aDstIdx,
                         SwMoveFlags::DEFAULT );
 
-                    // Wenn die Caption vor der Tabelle eingefuegt wurde muss
-                    // eine an der Tabelle gestzte Seitenvorlage noch in den
-                    // ersten Absatz der Ueberschrift verschoben werden.
-                    // Ausserdem muessen alle gemerkten Indizes, die auf den
-                    // Tabellen-Node zeigen noch verschoben werden.
+                    // If the caption was added before the table, a page style on that table
+                    // needs to be moved to the first paragraph of the header.
+                    // Additionally, all remembered indices that point to the table node
+                    // need to be moved
                     if( bTop )
                     {
                         MovePageDescAttrs( pTableStNd, aSrcRg.aStart.GetIndex(),
@@ -5299,14 +5189,14 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
                     }
                 }
 
-                // Die Section wird jetzt nicht mehr gebraucht.
+                // The section isn't needed anymore
                 m_pPam->SetMark();
                 m_pPam->DeleteMark();
                 m_xDoc->getIDocumentContentOperations().DeleteSection( const_cast<SwStartNode *>(pCapStNd) );
                 m_pTable->SetCaption( nullptr, false );
             }
 
-            // SwTable aufbereiten
+            // Process SwTable
             sal_uInt16 nBrowseWidth = (sal_uInt16)GetCurrentBrowseWidth();
             pSaveStruct->MakeTable( nBrowseWidth, *m_pPam->GetPoint(), m_xDoc.get() );
         }
@@ -5317,14 +5207,14 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
 
         if( m_pTable==pCurTable )
         {
-            // oberen Absatz-Abstand einstellen
+            // Set upper paragraph spacing
             m_bUpperSpace = true;
             SetTextCollAttrs();
 
             m_nParaCnt = m_nParaCnt - std::min(m_nParaCnt,
                 pTCntxt->GetTableNode()->GetTable().GetTabSortBoxes().size());
 
-            // ggfs. eine Tabelle anspringen
+            // Jump to a table if needed
             if( JUMPTO_TABLE == m_eJumpTo && m_pTable->GetSwTable() &&
                 m_pTable->GetSwTable()->GetFrameFormat()->GetName() == m_sJmpMark )
             {
@@ -5332,21 +5222,20 @@ HTMLTable *SwHTMLParser::BuildTable( SvxAdjust eParentAdjust,
                 m_eJumpTo = JUMPTO_NONE;
             }
 
-            // Wenn Import abgebrochen wurde kein erneutes Show
-            // aufrufen, weil die SwViewShell schon geloescht wurde!
-            // Genuegt nicht. Auch im ACCEPTING_STATE darf
-            // kein Show aufgerufen werden, weil sonst waehrend des
-            // Reschedules der Parser zerstoert wird, wenn noch ein
-            // DataAvailable-Link kommt. Deshalb: Nur im WORKING-State.
+            // If the import was canceled, don't call Show again here since
+            // the SwViewShell was already deleted
+            // That's not enough. Even in the ACCEPTING_STATE, a Show mustn't be called
+            // because otherwise the parser's gonna be destroyed on the reschedule,
+            // if there's still a DataAvailable link coming. So: only in the WORKING state
             if( !m_nParaCnt && SvParserState::Working == GetStatus() )
                 Show();
         }
     }
     else if( m_pTable==pCurTable )
     {
-        // Es wurde gar keine Tabelle gelesen.
+        // There was no table read
 
-        // Dann muss eine evtl gelesene Beschriftung noch geloescht werden.
+        // We maybe need to delete a read caption
         const SwStartNode *pCapStNd = pCurTable->GetCaptionStartNode();
         if( pCapStNd )
         {