[KHistoryComboBox] Add method to set an icon provider
ClosedPublic

Authored by nicolasfella on Dec 2 2019, 10:52 PM.

Details

Summary

An icon provider can be any function that takes a QString and returns a QIcon. This shall replace the setPixmapProvider to allow defining icons by name.

Test Plan

Tested with KDirSelectDialog and a patched plasma-integration

Diff Detail

Repository
R284 KCompletion
Branch
icopro
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 19602
Build 19620: arc lint + arc unit
nicolasfella created this revision.Dec 2 2019, 10:52 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptDec 2 2019, 10:52 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
nicolasfella requested review of this revision.Dec 2 2019, 10:52 PM
  • Deprecation, fix, and docs
nicolasfella edited the summary of this revision. (Show Details)Dec 2 2019, 11:12 PM
apol added a subscriber: apol.Dec 2 2019, 11:48 PM

+1 Makes sense to me

nicolasfella updated this revision to Diff 71070.EditedDec 7 2019, 10:49 PM

Rebase

  • Adapt deprecation version
dfaure requested changes to this revision.Dec 8 2019, 10:31 PM
dfaure added inline comments.
src/khistorycombobox.cpp
415

s/ENABLE/BUILD/ in .cpp files

499

BUILD

src/khistorycombobox.h
195

Missing @since 5.66

This revision now requires changes to proceed.Dec 8 2019, 10:31 PM
nicolasfella marked 3 inline comments as done.
  • Comments
dfaure accepted this revision.Dec 8 2019, 10:59 PM
This revision is now accepted and ready to land.Dec 8 2019, 10:59 PM
This revision was automatically updated to reflect the committed changes.