D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
This revision was automatically updated to reflect the committed changes. Closed by commit R824:38681e7d27f5: Show Date Modified information by default in Dolphins Information Panel and… (authored by ngraham). REPOSITORY R824 Baloo Widgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8303?vs=20768=20818 REVISION DETAIL https://phabricator.kde.org/D8303 AFFECTED FILES src/metadatafilter.cpp To: ngraham, #dolphin, #kde_applications, #frameworks, aacid, markg Cc: markg, elvisangelaccio, anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
ngraham added a comment. Great, thank you! REPOSITORY R824 Baloo Widgets BRANCH master REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks, aacid, markg Cc: markg, elvisangelaccio, anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
markg accepted this revision. markg added a comment. This revision is now accepted and ready to land. In https://phabricator.kde.org/D8303#155858, @ngraham wrote: > Honestly I think it's fine the way it is: in the details view, we have a nice compact string that doesn't make the column too wide, and in the Information Panel, we have a longer string that provides more information and isn't too big since text can wrap there. > > I do see your point though, and I agree that the long string should be customizable in some way for people who prefer a shorter string. But that seems out of scope for this change; can we stick to this change here, and discuss that other change (which I do approve of:) ) in another patch or bug report? Oke, then it's a +1 :) REPOSITORY R824 Baloo Widgets BRANCH master REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks, aacid, markg Cc: markg, elvisangelaccio, anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
ngraham added a comment. Honestly I think it's fine the way it is: in the details view, we have a nice compact string that doesn't make the column too wide, and in the Information Panel, we have a longer string that provides more information and isn't too big since text can wrap there. I do see your point though, and I agree that the long string should be customizable in some way for people who prefer a shorter string. But that seems out of scope for this change; can we stick to this change here, and discuss that other change (which I do approve of:) ) in another patch or bug report? REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks, aacid, markg Cc: markg, elvisangelaccio, anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
markg added a comment. In https://phabricator.kde.org/D8303#155840, @ngraham wrote: > @markg I actually like the current verbose format. That's somewhat unrelated to this; can we track that change, if any, with a new patch? That depends on how you want to do that ;) What i'm thinking about, but that might be way too specific, is providing a settings field somewhere where you fill in the date time format you want to see. And then use that format everywhere in Dolphin. But that would be a micro setting that probably not everyone would agree to. Another option is getting it from the settings as both your and my preference is there. Just go to System Settings -> Formats and there you see "Time". There are two lines there, long format and short format. I think it would be acceptable if dolphin is a "Time format" dropdown in it's settings that picks either the long format or the short format. Would that work for you? The default probably has to stay the short format though, as the long one will explode the detail view width. REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks, aacid, markg Cc: markg, elvisangelaccio, anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
ngraham marked an inline comment as done. ngraham added a comment. @markg I actually like the current verbose format. That's somewhat unrelated to this; can we track that change, if any, with a new patch? REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks, aacid, markg Cc: markg, elvisangelaccio, anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
markg requested changes to this revision. markg added a comment. This revision now requires changes to proceed. +1 for having this property -1 for the current rather verbose date format. Look at the "Modified" column in the any dolphin details view. It (on my setup) just shows a date and a HH:MM timestamp. That would be enough information, right? REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks, aacid, markg Cc: markg, elvisangelaccio, anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
elvisangelaccio added a comment. I guess +1 from me. Infopanel and tooltips are disabled by default anyway, so it's not a big deal if we also show the mtime there. REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks, aacid Cc: elvisangelaccio, anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
ngraham added a reviewer: aacid. REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks, aacid Cc: anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
ngraham updated this revision to Diff 20768. ngraham added a comment. Update current version REPOSITORY R824 Baloo Widgets CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8303?vs=20767=20768 BRANCH master REVISION DETAIL https://phabricator.kde.org/D8303 AFFECTED FILES src/metadatafilter.cpp To: ngraham, #dolphin, #kde_applications, #frameworks Cc: anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
anthonyfieroni added a comment. Baloo doesn't have active developers so you can add Aacid to accept it. INLINE COMMENTS > metadatafilter.cpp:44 > { > const int currentVersion = 6; // increase version, if the blacklist of > disabled > // properties should be updated Increase it like commet advice. REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks Cc: anthonyfieroni, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
D8303: Show Date Modified information by default in Dolphin's Information Panel and Tooltips
ngraham edited the test plan for this revision. ngraham added reviewers: Dolphin, KDE Applications, Frameworks. ngraham added a project: Dolphin. REPOSITORY R824 Baloo Widgets REVISION DETAIL https://phabricator.kde.org/D8303 To: ngraham, #dolphin, #kde_applications, #frameworks Cc: spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp