[Libreoffice-commits] core.git: sw/source
Lukas Röllin
lsroellin at gmail.com
Sat Feb 18 16:33:59 UTC 2017
sw/source/filter/html/htmltab.cxx | 803 ++++++++++++++++----------------------
1 file changed, 346 insertions(+), 457 deletions(-)
New commits:
commit 423beaf852aefb6a62882ac0ca55443113fda8b1
Author: Lukas Röllin <lsroellin at gmail.com>
Date: Wed Feb 15 19:39:35 2017 +0100
German comments in htmltab (squashed and fixed)
Change-Id: I37ffa94dc0edf58996fb59da838c02bb4a559519
Reviewed-on: https://gerrit.libreoffice.org/34325
Reviewed-by: Michael Stahl <mstahl at redhat.com>
Tested-by: Michael Stahl <mstahl at redhat.com>
diff --git a/sw/source/filter/html/htmltab.cxx b/sw/source/filter/html/htmltab.cxx
index f0705e5..e3d7f98 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
... etc. - the rest is truncated
More information about the Libreoffice-commits
mailing list