Deprecate KIO::pixmapForUrl
AbandonedPublic

Authored by nicolasfella on Dec 3 2019, 12:09 AM.

Details

Reviewers
dfaure
Group Reviewers
Frameworks
Summary

Can easily be done otherwise.

Diff Detail

Repository
R241 KIO
Branch
arcpatch-D25705_1
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 25274
Build 25292: arc lint + arc unit
nicolasfella created this revision.Dec 3 2019, 12:09 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptDec 3 2019, 12:09 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
nicolasfella requested review of this revision.Dec 3 2019, 12:09 AM
apol added a subscriber: apol.Dec 3 2019, 12:38 PM

What do you mean by Can easily be done otherwise.?

kossebau added inline comments.
src/widgets/pixmaploader.h
51

This line should be before the API dox comment.

In D25705#571458, @apol wrote:

What do you mean by Can easily be done otherwise.?

You can get an icon name for the icon with KIO::iconNameForUrl and then use QIcon API if you really need a pixmap

  • Fix update
nicolasfella planned changes to this revision.Apr 15 2020, 1:48 PM

Seems like I missed some usages

kossebau added inline comments.Apr 15 2020, 1:55 PM
src/widgets/pixmaploader.h
37 ↗(On Diff #72413)