Display "Downloaded From" by default, if file supports it
ClosedPublic

Authored by ngraham on Oct 27 2017, 6:22 PM.

Details

Summary

FEATURE: 386261

By default, show "Downloaded From" information from files' extended attributes, if present. No change for files without the xattr.

Test Plan

Tested in KDE Neon. Blew away dolphinrc and baloofileinformationrc, deployed the baloo-widgets change, rebuilt dolphin, downloaded a file using Chromium (which writes the user.xdg.origin.url xattr), opened Dolphin, navigated to the file, selected it, and turned on the Information panel. Downloaded From was visible. It was (correctly) not visible for any other file that lacked the user.xdg.origin.url xattr.

Diff Detail

Repository
R824 Baloo Widgets
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Oct 27 2017, 6:22 PM
ngraham edited the test plan for this revision. (Show Details)Oct 27 2017, 6:23 PM
ngraham added reviewers: Dolphin, Frameworks.
ngraham added a project: Dolphin.
ngraham edited the summary of this revision. (Show Details)
vhanda accepted this revision.Oct 29 2017, 5:06 AM
vhanda added a subscriber: vhanda.

Looks good.

Maybe we should wait and see if someone who works on Dolphin has an opinion. This widget is only used in Dolphin.

This revision is now accepted and ready to land.Oct 29 2017, 5:06 AM

Do any Dolphin people object? If not, I'll land this in a few days (sooner if I get some more thumbs up :) ).

markg accepted this revision.Oct 31 2017, 8:51 AM

Lovely! Less code, more features, those are the cool changes :)
I think it's safe to push.

This revision was automatically updated to reflect the committed changes.