lnav icon indicating copy to clipboard operation
lnav copied to clipboard

Modification to avoid color conflicts for colored bars

Open trantor opened this issue 3 years ago • 1 comments

Fix to avoid reusing the same color for numeric columns when graphed in the SQL view. A problem with columns graphed from JSON fields remains. Addresses #1047 .

Again @tstack , since I am not really a C++ developer, any advice/correction is appreciated.

trantor avatar Sep 08 '22 22:09 trantor

@tstack what do you think?

trantor avatar Sep 10 '22 19:09 trantor

I pushed a change that should address the common case. I would say a redesign of the UI needs to be done before trying to work on edge cases.

tstack avatar Sep 23 '22 16:09 tstack

Ok, thanks @tstack . I agree that a bit of refactoring might be in order to address other cases properly.

trantor avatar Sep 23 '22 17:09 trantor