[cosmetic] color_current_row takes precedence over color_column_sep

[mhoirup]

A small thing: the foreground color of the column separators gets overwritten by the foreground color of the current row. Would it be possible to have color_current_row ignore the column separators all together? In the picture you can see how the separators are colored white on the current row, even though I've specified a different color for separators.

[midichef]

For people who want to try out this proposed change, change the line in sheets.py:

visidata/visidata/sheets.py

Line 922 in efbe84e

basecellcattr = sepcattr = update_attr(rowcattr, color_current_row)

to not assign to sepcattr:

                basecellcattr = update_attr(rowcattr, color_current_row)

I'm too used to the existing behavior. So I can't tell what most users would prefer.

[mhoirup]

Thank you so much @midichef! Worked like a charm. I also defined my own color for the current row separators and used that in sheets.py like so:

The result is, in my subjective opinion, a much cleaner look:

[saulpw]

This was an intentional choice. If the background colors are different, the cursor row won't be continuous unless the color_current_row takes precedence. I don't know what the solution is here to make us both happy. Maybe the default 'reverse' is the problem?