Differential D25485

Remove KIconThemes use
ClosedPublic
Actions

Authored by broulik on Nov 23 2019, 9:02 AM.

Details

Reviewers
cullmann
Group Reviewers
Frameworks
Commits
R39:c7d64230aa34: Remove KIconThemes use
Summary

It was effectively unused.

Test Plan
  • help button on the variable editor still shows and hides as I enter and leave

Diff Detail

Repository
R39 KTextEditor
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Nov 23 2019, 9:02 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptNov 23 2019, 9:02 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
broulik requested review of this revision.Nov 23 2019, 9:02 AM
cullmann accepted this revision.Nov 23 2019, 9:04 AM
This revision is now accepted and ready to land.Nov 23 2019, 9:04 AM
Closed by commit R39:c7d64230aa34: Remove KIconThemes use (authored by broulik). · Explain WhyNov 23 2019, 9:11 AM
This revision was automatically updated to reflect the committed changes.
Restricted Application added a project: Kate. · View Herald TranscriptNov 23 2019, 9:11 AM
Restricted Application added a subscriber: kwrite-devel. · View Herald Transcript
dhaumann added a subscriber: dhaumann.Nov 23 2019, 9:51 AM

Makes sense, thanks.

Revision Contents
Changeset List

Diff 70199

View Options

CMakeLists.txt

Loading...
View Options

autotests/CMakeLists.txt

Loading...
View Options

src/CMakeLists.txt

Loading...
View Options

src/dialogs/katedialogs.cpp

Loading...
View Options

src/variableeditor/katehelpbutton.h

Loading...
View Options

src/variableeditor/katehelpbutton.cpp

Loading...