Skip to content

Commit cc385e1

Browse files
albert-githubalbert-github
committed
Improve headings of tables
- possibility of multi line table headings - possibility to have "heading" cells in the middle of a table (analogous to HTML output)
1 parent 445ff9a commit cc385e1

File tree

5 files changed

+23
-35
lines changed

5 files changed

+23
-35
lines changed

‎src/docnode.cpp

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2185,13 +2185,19 @@ const DocNodeVariant *DocHtmlTable::caption() const
21852185
return m_caption.get();
21862186
}
21872187

2188-
const DocNodeVariant *DocHtmlTable::firstRow() const
2188+
size_t DocHtmlTable::numberHeaderRows() const
21892189
{
2190-
if (!children().empty() && std::holds_alternative<DocHtmlRow>(children().front()))
2190+
size_t hl = 0;
2191+
for (auto &rowNode : children())
21912192
{
2192-
return &children().front();
2193+
const DocHtmlRow *row = std::get_if<DocHtmlRow>(&rowNode);
2194+
if (row)
2195+
{
2196+
if (!row->isHeading()) break;
2197+
hl++;
2198+
}
21932199
}
2194-
return nullptr;
2200+
return hl;
21952201
}
21962202

21972203
Token DocHtmlTable::parse()

‎src/docnode.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1277,7 +1277,7 @@ class DocHtmlTable : public DocCompoundNode
12771277
Token parseXml();
12781278
size_t numColumns() const { return m_numCols; }
12791279
const DocNodeVariant *caption() const;
1280-
const DocNodeVariant *firstRow() const;
1280+
size_t numberHeaderRows() const;
12811281

12821282
private:
12831283
void computeTableGrid();

‎src/latexdocvisitor.cpp

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1262,7 +1262,6 @@ void LatexDocVisitor::writeStartTableCommand(const DocNodeVariant *n,size_t cols
12621262
{
12631263
m_t << "\n\\begin{DoxyTable}{" << cols << "}";
12641264
}
1265-
//return isNested ? "TabularNC" : "TabularC";
12661265
}
12671266

12681267
void LatexDocVisitor::writeEndTableCommand(const DocNodeVariant *n)
@@ -1275,7 +1274,6 @@ void LatexDocVisitor::writeEndTableCommand(const DocNodeVariant *n)
12751274
{
12761275
m_t << "\\end{DoxyTable}\n";
12771276
}
1278-
//return isNested ? "TabularNC" : "TabularC";
12791277
}
12801278

12811279
void LatexDocVisitor::operator()(const DocHtmlTable &t)
@@ -1294,7 +1292,6 @@ void LatexDocVisitor::operator()(const DocHtmlTable &t)
12941292
m_t << "\n";
12951293
}
12961294

1297-
const DocHtmlRow *firstRow = std::get_if<DocHtmlRow>(t.firstRow());
12981295
writeStartTableCommand(t.parent(),t.numColumns());
12991296
if (!isTableNested(t.parent()))
13001297
{
@@ -1313,18 +1310,9 @@ void LatexDocVisitor::operator()(const DocHtmlTable &t)
13131310
}
13141311
m_t << "}";
13151312
}
1316-
// write head row
1317-
m_t << "{";
1318-
if (firstRow && firstRow->isHeading())
1319-
{
1320-
m_t << "1";
1321-
}
1322-
else
1323-
{
1324-
m_t << "0";
1325-
}
1326-
m_t << "}";
1327-
m_t << "\n";
1313+
1314+
// write head row(s)
1315+
m_t << "{" << t.numberHeaderRows() << "}\n";
13281316

13291317
setNumCols(t.numColumns());
13301318

@@ -1449,6 +1437,11 @@ void LatexDocVisitor::operator()(const DocHtmlCell &c)
14491437
setCurrentColumn(currentColumn()+cs-1);
14501438
appendOpt("c="+QCString().setNum(cs));
14511439
}
1440+
if (c.isHeading())
1441+
{
1442+
appendSpec("bg=\\tableheadbgcolor");
1443+
appendSpec("font=\\bfseries");
1444+
}
14521445
switch(va) // vertical alignment
14531446
{
14541447
case DocHtmlCell::Top:

‎src/latexdocvisitor.h

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -188,7 +188,6 @@ class LatexDocVisitor : public DocVisitor
188188
RowSpanList rowSpans;
189189
size_t numCols = 0;
190190
size_t currentColumn = 0;
191-
bool firstRow = false;
192191
};
193192
std::stack<TableState> m_tableStateStack; // needed for nested tables
194193
RowSpanList m_emptyRowSpanList;
@@ -227,14 +226,6 @@ class LatexDocVisitor : public DocVisitor
227226
{
228227
if (!m_tableStateStack.empty()) m_tableStateStack.top().numCols = num;
229228
}
230-
bool firstRow() const
231-
{
232-
return !m_tableStateStack.empty() ? m_tableStateStack.top().firstRow : FALSE;
233-
}
234-
void setFirstRow(bool b)
235-
{
236-
if (!m_tableStateStack.empty()) m_tableStateStack.top().firstRow = b;
237-
}
238229
RowSpanList &rowSpans()
239230
{
240231
return !m_tableStateStack.empty() ? m_tableStateStack.top().rowSpans : m_emptyRowSpanList;

‎templates/latex/doxygen.sty

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -393,17 +393,15 @@
393393
\begin{longtblr}{hlines,vlines,% automatically set horizontal and vertical cell lines
394394
measure=vbox,% needed to allow nested lists and tables
395395
colspec={*{#1}{X[-1]}},% set column type for all columns
396-
rowhead=#4,% set which row is the header (0=disable, 1=enable)
397-
row{#4}={bg=\tableheadbgcolor,font=\bfseries}}% set header background color and font weight
396+
rowhead=#4} % set which row is the header (0=disable, 1=enable)
398397
#5
399398
\end{longtblr}%
400399
\else% use fixed left aligned columns
401400
% now render the table for real
402401
\begin{longtblr}{hlines,vlines,% automatically set horizontal and vertical cell lines
403402
measure=vbox,% needed to allow nested lists and tables
404403
colspec={*{#1}{l}},% set column type for all columns
405-
rowhead=#4,% set which row is the header (0=disable, 1=enable)
406-
row{#4}={bg=\tableheadbgcolor,font=\bfseries}}% set header background color and font weight
404+
rowhead=#4} % set which row is the header (0=disable, 1=enable)
407405
#5
408406
\end{longtblr}%
409407
\fi%
@@ -419,8 +417,8 @@
419417
\begin{tblr}{hlines,vlines,% automatically set horizontal and vertical cell lines
420418
measure=vbox,% needed to allow nested lists and tables
421419
colspec={*{#1}{X[-1]}},% set column type for all columns
422-
rowhead=#2,% set which row is the header (0=disable, 1=enable)
423-
row{#2}={bg=\tableheadbgcolor,font=\bfseries}}% set header background color and font weight
420+
rowhead=#2% set which row is the header (0=disable, 1=enable)
421+
}
424422
}{% no end marker needed anymore
425423
\end{tblr}
426424
}%

0 commit comments

Comments
 (0)