D27864: KIO::iconNameForUrl(): handle the case of a file/folder under trash:/

2020-03-05 Thread Ahmad Samir
This revision was automatically updated to reflect the committed changes. Closed by commit R241:082e01b19a8e: KIO::iconNameForUrl(): handle the case of a file/folder under trash:/ (authored by ahmadsamir). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27864?vs=77

D27864: KIO::iconNameForUrl(): handle the case of a file/folder under trash:/

2020-03-05 Thread Méven Car
meven accepted this revision. REPOSITORY R241 KIO BRANCH l-trash-stuff (branched from master) REVISION DETAIL https://phabricator.kde.org/D27864 To: ahmadsamir, #frameworks, dfaure, meven, sitter, broulik Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

D27864: KIO::iconNameForUrl(): handle the case of a file/folder under trash:/

2020-03-05 Thread David Faure
dfaure accepted this revision. dfaure added a comment. This revision is now accepted and ready to land. Yay for unittests :-) REPOSITORY R241 KIO BRANCH l-trash-stuff (branched from master) REVISION DETAIL https://phabricator.kde.org/D27864 To: ahmadsamir, #frameworks, dfaure, meven,

D27864: KIO::iconNameForUrl(): handle the case of a file/folder under trash:/

2020-03-05 Thread Ahmad Samir
ahmadsamir created this revision. ahmadsamir added reviewers: Frameworks, dfaure, meven, sitter, broulik. Herald added a project: Frameworks. ahmadsamir requested review of this revision. REVISION SUMMARY For trash:/ we have two individual cases: - trash:/ itself, show empty/full trash icon