D14980: Restore settings label and prevent warning when Baloo is unavailable
Henrik Fehlauer
noreply at phabricator.kde.org
Tue Aug 21 20:13:09 BST 2018
rkflx created this revision.
rkflx added a reviewer: elvisangelaccio.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
rkflx requested review of this revision.
REVISION SUMMARY
As discussed in D14814 <https://phabricator.kde.org/D14814>, 6674c9c387d0 <https://phabricator.kde.org/R318:6674c9c387d0986e99698332727bf797257bf07d> removed the Show tooltips
checkbox in Dolphin's preferences when Baloo is unavailabe at compile
time and thus the tooltip would not be functional. The Miscellaneous
label in front of the checkbox was also removed, even though it relates
to all options in that group.
In addition, after 6674c9c387d0 <https://phabricator.kde.org/R318:6674c9c387d0986e99698332727bf797257bf07d> running Dolphin without Baloo support
would print
QWidget::insertAction: Attempt to insert null action
on startup.
Both issues can be fixed with more `#ifdefs`.
TEST PLAN
After building with `-DCMAKE_DISABLE_FIND_PACKAGE_KF5Baloo=ON`, the
Show tooltips checkbox is removed, the Miscellaneous label
is still there, and there is no warning printed.
REPOSITORY
R318 Dolphin
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D14980
AFFECTED FILES
src/dolphinmainwindow.cpp
src/settings/general/behaviorsettingspage.cpp
To: rkflx, elvisangelaccio
Cc: kfm-devel, spoorun, navarromorales, firef, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180821/50d2d253/attachment.htm>
More information about the kfm-devel
mailing list