Add Cell Level Controll for Table Borders

[bbrenter]

Intent
It would be great to have control over table borders by defining them for each cell.

Describe the solution you'd like
This could for example be implemented by adding the border string slot to the Cell Class and use it inside the get_cell_border function to determine the border for each cell. A possible implementation could look like this...

[andersonhc]

@bbrenter are you interested in implementing the feature and submitting a PR?

[Kaustbh]

Hi I'm new to open source, but I'm eager to dive in and help out. I came across this issue and I think I can tackle it. Can I work on it?

[Lucas-C]

Yes, given the history on this issue, you are very welcome to tackle this @Kaustbh 🙂

As a minor change to the interface suggested by @bbrenter,
I think it would be better if the new border property of Cell was typed with a new subclass of CoerciveEnum, maybe named CellBordersLayout (in a similar fashion as TableBordersLayout). It could have those values:

• NONE = 0
• LEFT = 1
• RIGHT = 2
• TOP = 4
• BOTTOM = 8
• ALL = LEFT | RIGHT | TOP | BOTTOM = 15
• INHERIT = 16 would be the default value, meaning to inherit from the table layout scheme

A starting point for you @Kaustbh could to be to read our development guidelines.

Then, you could add a new unit test to test/table/test_table.py.

Please ask away if you have any question! 🙂

[Kaustbh]

Hi @Lucas-C ,

I'm currently working on implementing the CellBordersLayout as suggested. However, I'm having trouble understanding table rendering logic and get_cell_border function, could you please help me?

[Kaustbh]

I understood the logic of rendering and get_cell_border function, can you please tell me how should I write CellBordersLayout class whether it should be of type CoerciveEnum or CoerciveIntFlag ?