search

GoldenGnu / jeveassets

jEveAssets is an out-of-game asset manager for Eve-Online, written in Java
http://eve.nikr.net/jeveasset
GNU General Public License v2.0
147 stars 36 forks source link

"Grand Total" Sums should include an option to ignore filtered results #443

Closed Groot2C closed 1 month ago

Groot2C commented 4 months ago

Feature request: for the "Grand Total" row to have the option to exclude filtered results. For example, on the "Slots" page it sums up all 60 of my characters instead of just the 20 that do Reactions/Manufacturing. Therefore, I have an "extra" 40 Reaction/Manufacturing/Researching slots than I "actually" have.

This is even after I add a filter "Reactions Max - Does Not Equals 1".

GoldenGnu commented 1 month ago

I'm going to reject this feature, for the following reasons: The total for shown/filtered slots are already shown in the statusbar. Changing table data on filtering can create a paradoxical state. Since every time you filter on the data, the data change, so you have to filter again, which may change the data again etc. leading to an endless loop. Which means it's very hard to do right. In this case the data is already available, so I don't think it's worth the effort.

If you have a good argument for implementing this regardless of the above, I'm always open to discussing it, either here or on discord. Nothing is set in stone.