D24450: [dolphin/search] Reset search options when needed

Ismael Asensio noreply at phabricator.kde.org
Sun Oct 6 23:40:31 BST 2019


iasensio created this revision.
iasensio added reviewers: Dolphin, elvisangelaccio, meven.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
iasensio requested review of this revision.

REVISION SUMMARY
  Adds a method to reset the options in `facetsWidget` in two cases: when disabled and before parsing a new search URL.
  Otherwise, controls for a parameter (ex. `rating`) which are not found in the new URL would stay on the old positions, instead of the default "any". 
  See D24422 <https://phabricator.kde.org/D24422>

TEST PLAN
  Controls on `facetsWidget` go back to the default in such two cases.

REPOSITORY
  R318 Dolphin

BRANCH
  search_parse_reset

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

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

To: iasensio, #dolphin, elvisangelaccio, meven
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/20191006/c895c525/attachment.htm>


More information about the kfm-devel mailing list