Show unsaved entry changes in entry list

[anansutiawan]

Fixes #1162

Shows entries with unsaved local changes in italics when autosave after every change is disabled. The model compares entry modification timestamps with the current database file timestamp only while the database is dirty, so the indicator clears after save and is suppressed when autosave-after-every-change is enabled.

Generative AI disclosure: The majority of this patch was prepared with OpenAI Codex.

Screenshots

Not included; visual change is font styling only.

Testing strategy

• Added TestEntryModel::testUnsavedEntryFont() coverage for unsaved-entry italic styling, autosave suppression, and clean-database clearing.
• Ran git diff --check.
• Not run locally: CMake build, unit test binary, or clang-format. cmake and clang-format are not available in this workspace PATH.

Type of change

• New feature (change that adds functionality)

[droidmonkey]

I cleaned up the code since there was a lot of extra calls to data updates that are unnecessary. Qt refreshes the data constantly and we already refresh the whole list when an entry is modified.

I am playing around with a badge indicator and tooltip in addition to the italic text.

One thing that gives me pause on this PR is that we constantly (like hundreds of times) check the on-disk file modified information which can be VERY expensive, especially for networked files. We will have to adjust this to use cached information before merging or some other smarter way to indicate an entry was modified but not yet saved.

[anansutiawan]

Thanks for taking a pass on the UI.

I pushed 78f692d to address the expensive file-stat concern. Database now caches the saved file modification time when the file path changes or the database is marked clean, and Entry::hasUnsavedChanges() uses that cached value instead of constructing QFileInfo from the entry model/icon/search hot paths. I also kept the AutoSaveAfterEveryChange guard so autosave mode does not show the unsaved indicator.

Local validation available in this workspace:

git diff --check

I still cannot run CMake, the unit binary, or clang-format locally because those tools are not available in this PATH.

[droidmonkey]

I also kept the AutoSaveAfterEveryChange guard so autosave mode does not show the unsaved indicator.

I removed that for a reason because it is not desired. Auto save doesnt mean the database is actually saved to disk. Saves can error out leaving the database unsaved and still modified.

[anansutiawan]

Thanks, that makes sense. I pushed d8ce861 to remove the AutoSaveAfterEveryChange suppression from Entry::hasUnsavedChanges(), so the indicator now still reflects a dirty database if autosave fails or leaves changes unsaved. I also updated the EntryModel regression test to expect the italic indicator while autosave is enabled and the database remains modified.\n\nLocal validation available here: git diff --check. CMake, the unit binary, and clang-format are still unavailable in this workspace PATH.