Bug 434275

Summary: Row height determined by number of tags even if they are not visible
Product: [Applications] digikam Reporter: Quincy <bbc.quincy>
Component: Albums-TableViewAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: 7.1.0   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: https://invent.kde.org/graphics/digikam/commit/73247bf4567fed4dcbef40a83947d0c70904a24e Version Fixed In: 7.2.0
Sentry Crash Report:

Description Quincy 2021-03-11 10:17:28 UTC 
SUMMARY
The height of the rows in the TableView obviously is adjusted always by the number of tags assigned to the image. So rows for images with many tags are big enough to fit all of those at the same time. Height stays the same even if the "Tags" column is removed and not visible anymore.

STEPS TO REPRODUCE
1. Open an album with images tagged with several tags in parallel
2. Remove the (default) column "Tags"

OBSERVED RESULT
Height of table rows stays as before

EXPECTED RESULT
Row height should be reduced to the actually visible data

SOFTWARE/OS VERSIONS
Windows: 10

ADDITIONAL INFORMATION
Changing the tab/view back and forth does not change anything, so no update problem.
Comment 1 Quincy 2021-03-11 10:36:06 UTC 
Just found out because it was a quite long list in my case and further testing:

There is some update problem, as I finally could manage to display all entries in a "one-line" row after switching back and forth to different views.

Even when removing the column, some entries directly switch to appropriate height, but not all.

Additionally those who switched don't enlarge again when re-enabling the display of tags leading to cluttered display in those cases.
Comment 2 caulier.gilles 2021-03-11 11:06:29 UTC 
Hi,

Please test with 7.2.0 pre-release compiled with current code to see if problem remain :

https://files.kde.org/digikam/

Gilles Caulier
Comment 3 Maik Qualmann 2021-03-11 11:47:26 UTC 
Git commit 73247bf4567fed4dcbef40a83947d0c70904a24e by Maik Qualmann.
Committed on 11/03/2021 at 11:46.
Pushed by mqualmann into branch 'master'.

update table layout after add or remove column
FIXED-IN: 7.2.0

M  +2    -2    NEWS
M  +12   -0    core/app/views/tableview/tableview_model.cpp

https://invent.kde.org/graphics/digikam/commit/73247bf4567fed4dcbef40a83947d0c70904a24e
Comment 4 Quincy 2021-03-12 06:51:17 UTC 
With digiKam-7.2.0-20210311T200524-Win64.exe the issue is solved. Very nice and fast work, very much appreciated :-)