Review Request: Enable next/previous search buttons only when needed

Francesco Nwokeka francesco.nwokeka at gmail.com
Sun Apr 3 16:46:16 CEST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101017/
-----------------------------------------------------------

(Updated April 3, 2011, 2:46 p.m.)


Review request for Telepathy.


Changes
-------

Added the enable/disable feature for the actions in the KXmlGUIWindow as well. 
Plus I check if these are to be enabled even with multiple tabs. 
For example, if you have two tabs open and in one of these you are doing a search, if you switch to the other one( without search activated ) the actions will be deactivated. When switching back they will be reactivated.


Summary
-------

Simple patch that disables the next and previous buttons in the search bar when there is no text to look for.
Requested by martin


Diffs (updated)
-----

  app/chat-window.h 45ba617 
  app/chat-window.cpp 0725ca6 
  lib/chat-search-bar.h 796280f 
  lib/chat-search-bar.cpp 6b137b3 

Diff: http://git.reviewboard.kde.org/r/101017/diff


Testing
-------


Thanks,

Francesco

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110403/14b3f374/attachment.htm 


More information about the KDE-Telepathy mailing list