p-ranav / tabulate

Table Maker for Modern C++

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Color is not working in nested tables

maximiliank opened this issue · comments

When I try to put a Table with a colored cell into another Table the color is not working anymore.

Here is an example:

#include "tabulate/tabulate.hpp"
#include <iostream>

int main()
{
    using namespace tabulate;

    Table inner;
    inner.add_row({"A"});
    inner[0][0].format().font_color(Color::green);
    std::cout << "Here 'A' is green:\n" << inner << std::endl;

    Table outer;
    outer.add_row({inner});
    std::cout << "Here 'A' is not green:\n" << outer << std::endl;
    return 0;
}

which gives the following output on Ubuntu:

Here 'A' is green:
+---+
| A |
+---+
Here 'A' is not green:
+-------+
| +---+ |
| | A | |
| +---+ |
+-------+

Yeah this is a known issue and it has to do with std::stringstream not retaining the termcolor colors.

Essentially, how nested tables are implemented are as follows:

  • The inner table is converted to string (using std::stringstream as the stream, instead of std::cout)
    • Relevant source here.
  • The outer table cell value is now the inner table string, serialized in the previous step
  • The outer table is then fully printed to std::cout

If the inner table colors were preserved in the first step, this would work as we want. Unfortunately, this is not the case.

Nested table support was added as sort of an after-thought/bonus feature. I may have to just implement printing of nested tables with iteration (starting from the outer table and going in), which is non-ideal.

Thank you for your reply.

I tried to add << termcolor::colorize, see ikalnytskyi/termcolor#20, to the inline void Printer::print_table(std::ostream &stream, TableInternal &table) function. The color works, but the right borders are messed up.
Do you think it could work this way?

ansi escape sequence should be considered in get_sequence_length to support colorful nested table. I have implemented by myself https://github.com/vxfury/tabulate. and with true color supported, too.

ansi escape sequence should be considered in get_sequence_length to support colorful nested table. I have implemented by myself https://github.com/vxfury/tabulate. and with true color supported, too.

截屏2021-10-20 11 58 22

A pull request would be greatly appreciated :)

Is there a support plan for this feature?

An easy bypass of this issue is to convert the table to string manually with termcolor::colorize, enable the multibyte char, and then limit the width manually.

@edisonhello can you give code for this 'bypass' ?
@vxfury link https://github.com/vxfury/tabulate is not available - is it a chance to get this piece of code?