Bug 434275 - Row height determined by number of tags even if they are not visible
Summary: Row height determined by number of tags even if they are not visible
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Albums-TableView (show other bugs)
Version: 7.1.0
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-03-11 10:17 UTC by Quincy
Modified: 2021-03-12 06:51 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 7.2.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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 :-)