Review Request: focus on searchbox in activity manager when it is opened

Marco Martin notmart at gmail.com
Mon Feb 28 12:16:05 CET 2011


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

Ship it!


good work!

- Marco


On Feb. 28, 2011, 10:54 a.m., Farhad Hedayati Fard wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100772/
> -----------------------------------------------------------
> 
> (Updated Feb. 28, 2011, 10:54 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> focus on activity manager when it is opened and pass the focus to the searchbox using focusInEvent() , btw, this doesn't work if I dont use QTimer::singleshot() , If you know a better way for doing this, please let me know :)
> 
> 
> Diffs
> -----
> 
>   plasma/desktop/shell/activitymanager/activitymanager.h e347fac 
>   plasma/desktop/shell/activitymanager/activitymanager.cpp 0f61351 
>   plasma/desktop/shell/activitymanager/filterbar.h 21ba49c 
>   plasma/desktop/shell/activitymanager/filterbar.cpp 3d6d4dc 
>   plasma/desktop/shell/controllerwindow.cpp cf6fcc4 
> 
> Diff: http://git.reviewboard.kde.org/r/100772/diff
> 
> 
> Testing
> -------
> 
> works fine here! 
> 
> 
> Thanks,
> 
> Farhad
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110228/5b6e9501/attachment.htm 


More information about the Plasma-devel mailing list