https://bugs.kde.org/show_bug.cgi?id=118533

--- Comment #4 from [email protected] ---
Jay,

In you patch the approach is not the right way.

1/ You don't need in fact QDateTimeEdit. The request from this file is to
extract the time-stamp from image and to populate the text editor with these
info.
2/ add a QCheckBox to enable/disable this feature in setting view.
3/ When check box is turned on, call a slot to parse image metadata and extract
time stamp, and finaly pass the text the d->textEdit editor.

That all, in fact it's very simple.

Gilles Caulier

Notes:
- no need // NEW comment in source code. New line are visible in patch format.
- text in i18n() are literal strings for translators. Don't use extra
uppercases.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to