Bug 397727 - Don't show "Mod."" label when no modified timestamp exists on an image
Summary: Don't show "Mod."" label when no modified timestamp exists on an image
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Usability-Ergonomy (show other bugs)
Version: 6.0.0
Platform: macOS (DMG) macOS
: NOR minor
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-22 00:30 UTC by bcr
Modified: 2020-12-19 05:35 UTC (History)
2 users (show)

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


Attachments
Two images, only one is 'modified' (23.24 KB, image/png)
2018-08-22 00:30 UTC, bcr
Details

Note You need to log in before you can comment on or make changes to this bug.
Description bcr 2018-08-22 00:30:41 UTC
Created attachment 114538 [details]
Two images, only one is 'modified'

Per the attached screenshot, I think it would be best if the "Mod.:" label was left off if the image doesn't actually have a modification timestamp.

In lieu of a text label, how about showing a tiny gear/wrench/tool instead?

Not sure why most of my images aren't showing up, I guess thats a database problem being worked on?
Comment 1 caulier.gilles 2018-08-22 06:06:45 UTC
If i remember, the modification time stamp must be show only if it differs than creation time stamp.

The "Mod:" label with nothing after is just a side effect. The label must be hidden as well..

So, for me it's a little ergonomy bug.

Gilles Caulier
Comment 2 Maik Qualmann 2018-08-22 06:08:40 UTC
Presumably, the modification date is not valid. It is easy to fix the "Mod.:" string then not print. What if you re-read the metadata on this image?

Maik
Comment 3 Maik Qualmann 2018-08-27 20:02:07 UTC
Git commit 9e78d36825ef7913f24a8ea8f93a6b137f965902 by Maik Qualmann.
Committed on 27/08/2018 at 20:00.
Pushed by mqualmann into branch 'master'.

show modification label only if the date is valid
FIXED-IN: 6.0.0

M  +2    -1    NEWS
M  +2    -1    core/app/items/imagedelegate.cpp

https://commits.kde.org/digikam/9e78d36825ef7913f24a8ea8f93a6b137f965902