D23232: [dolphin] Add an action to toggle the searchbar

[Nathaniel Graham]

ngraham added a comment.


  In D23232#514449 <https://phabricator.kde.org/D23232#514449>, @ndavis wrote:
  
  > I also think that using toggle for search makes more sense. It would be nice to have a general pattern of using toggle buttons in the toolbar for things that can be opened and closed, especially when they're activated via the toolbar in the first place.
  
  
  +1. In fact, we already do have this pattern; there are just a few that don't conform.

REPOSITORY
  R318 Dolphin

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

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