[FIX] OWTable: Include attributes from meta attributes in header row labels

[janezd]

Issue

Fixes #2777

Includes

• Code changes
• Tests

[codecov]

Codecov Report

Merging #3633 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3633      +/-   ##
==========================================
+ Coverage   84.31%   84.32%   +<.01%     
==========================================
  Files         370      370              
  Lines       67838    67850      +12     
==========================================
+ Hits        57197    57213      +16     
+ Misses      10641    10637       -4

[markotoplak]

This is certainly a fix of the point (2) in #2777.

But you decided that you won't fix the point (1), right?

[janezd]

I didn't decide, I just overlooked it. :)

But if I didn't overlook it, I still wouldn't implement it. I think it doesn't hurt to have this information there. If we however decide to remove it, we can easily do this by renaming "color" to "_color".

What's your opinion? And @BlazZupan, @brunap, others?

[markotoplak]

@janezd, I am against hiding "color". I have no opinion about "_color".

[janezd]

If you're against hiding, that's it. We would rename it to _color if we wanted to hide it -- all attributes that start with _ are automatically hidden.