Feature request: Style table rows / cells individually

[SkyLeite]

I would like to be able to provide styling for rows and / or cells individually. Currently the only way to do this is through the Selected property, but that approach is limited in the sense that you can only have two styles (one for the base, and one for the selected row).

[maaslalani]

Hey, this is a good idea and natural extension of the table. I think we should probably introduce a row that can take any struct that implements the Stringer interface so that you can pass lipgloss Styles and anything else for displaying the table.

[pja237]

Hey all,
@maaslalani this is my attempt at this feature, goal for me was:

1. be backward compatible, all the code using the table package continues to work as-is
2. not have to change too much of the table.go code (which tbh. i don't know how to achieve by changing the Row to an interface, i end up having to refactor half of the package, or break no 1. and do ton of code changes in the existing app 😆 )

and this is the best i could come up with, if you'd be interested to take a look and let me know what you think of this approach (it's a local, older copy of tables code i'm still using, but i can easily PR this here)

1. i've composed a new CustomStyles struct in the table model
   https://github.com/CLIP-HPC/SlurmCommander/blob/alt_table_2/internal/table/table.go#L22

which holds interfaces for Cell/Row/Selected functions.

// int is the row/col number, row is the row content, used in the methods to generate custom styles
type CustomStyle interface {
	Style(int, Row) lipgloss.Style
}

https://github.com/CLIP-HPC/SlurmCommander/blob/alt_table_2/internal/table/table.go#L28-L32

2. modified the renderRow() method to include checks if functions are present in the model and do the "overlaying" of cell<-row<-selected styles (using .Inherit()) so we can compose them.
   https://github.com/CLIP-HPC/SlurmCommander/blob/alt_table_2/internal/table/table.go#L406-L454

This way, separate styles for a single Cell/Column of cells/Row/Selected rows can be programmatically done, like i'm doing here (first working prototype):
https://github.com/CLIP-HPC/SlurmCommander/blob/alt_table_2/internal/model/tabs/clustertab/clustertabtable.go#L69-L90

[jlec]

@pja237 Could you create a PR so that we can work together on refining your idea?

[pja237]

@jlec Was hoping someone would be willing to help me with this 👍
I've done the PR, it's above. Looking forward to hearing your thoughts.

[hedhyw]

I've implemented the first part for stylizing cells: #397
Please, review.

[fsareshwala]

I've implemented the first part for stylizing cells: #397
Please, review.

Code looks good to me. I've personally been waiting for this for a personal project of mine so I'm excited to have this merged.

[bashbunni]

Hey this was introduced in #504 and will be in the next release :)