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