Column sum for views is incorrectly calculated

[AndyScherzinger]

Steps to reproduce

1. create a table with columns
2. create views with one less column

Expected behavior

sum of columns for views matches the views

Actual behavior

sum of columns for views matches the number of columns of the table

Tables app version

0.6.5

Browser

FF121

Client operating system

Win11Pro

Operating system

No response

Web server

None

PHP engine version

None

Database

None

Additional info

c.nc.c

[enjeck]

To be sure, we expect total to be the sum of the rows and column values? Because it appears even the rows sum isn't always right too.

[juliusknorr]

Total is not supposed to be the sum but the overall row/column count, as a row could be part of multiple views. However I get how the visual presentation can be a bit confusing.

Maybe @nextcloud/designers have any input to that.

[datenangebot]

Maybe just renaming the "Total" into "Total counter"...

[jancborchardt]

So I understand the "Total" for rows, but do we need a total for columns at all? That's not really a common/relevant thing to be counted in a table or spreadsheet, which also causes this confusion.

[enjeck]

So we can remove the column count to address this issue?

[AndyScherzinger]

Yes 👍

[datenangebot]

So I understand the "Total" for rows, but do we need a total for columns at all? That's not really a common/relevant thing to be counted in a table or spreadsheet, which also causes this confusion.

@jancborchardt @AndyScherzinger I would say it is relevant. There is no other way to get the number of columns. Specially when you have a ton of columns, you don't like to count them one by one?! And in the table it might help to get an idea how many columns exists in comparison of the numbers for the views. That way you can easily see if a view contains all columns or how many are missing.

So I would like to keep them, but maybe we can make it more clear what this number means?

[jancborchardt]

That way you can easily see if a view contains all columns or how many are missing.

But this also becomes a calculation you have to do on your own, especially when you have a ton of columns.

I would say this info is not 100% relevant for this overview, but rather in the vuew itself – where it is visible via the modal which columns are enabled or not.

[enjeck]

Related: #606