D24446: Hide search extra options when not available

Ismael Asensio noreply at phabricator.kde.org
Wed Oct 9 22:20:15 BST 2019


iasensio updated this revision to Diff 67579.
iasensio marked an inline comment as done.
iasensio added a comment.


  - Add setFacetsVisible() method
  - Fuse setFacetsVisible() and updateFacetsToggleWidget()
  
  The fusing is to avoid an issue on first launch, where if `SearchSettings::showFacetsWidget()` is `true`, we set the facets visibility to `true`, but the `isVisible()` method returns `false`, because it is not yet actually shown.

REPOSITORY
  R318 Dolphin

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D24446?vs=67467&id=67579

BRANCH
  facets_hide

REVISION DETAIL
  https://phabricator.kde.org/D24446

AFFECTED FILES
  src/search/dolphinsearchbox.cpp
  src/search/dolphinsearchbox.h

To: iasensio, #dolphin, elvisangelaccio, meven, ngraham
Cc: kfm-devel, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20191009/9ec23738/attachment.htm>


More information about the kfm-devel mailing list