-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://mattr.info:8080/r/302/#review335
-----------------------------------------------------------

Ship it!


Please commit, if you haven't already. Thanks!

- Matt


On 2008-03-16 14:41:00, Charles Connell wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://mattr.info:8080/r/302/
> -----------------------------------------------------------
> 
> (Updated 2008-03-16 14:41:00)
> 
> 
> Review request for Kopete.
> 
> 
> Summary
> -------
> 
> The system tray menu has an option called "Minimize" or "Restore," depending 
> on which is relevant at the time. This is, in my opinion, completely useless 
> since clicking on the tray icon itself has the same effect. It takes up 
> precious space in the tray menu, and it is only in Kopete (out of all KDE 
> apps), as far as I can tell. This patch simply removes the menu option.
> 
> 
> This addresses bug 146835.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdenetwork/kopete/kopete/kopetewindow.cpp
> 
> Diff: http://mattr.info:8080/r/302/diff
> 
> 
> Testing
> -------
> 
> I ran it a couple of times and it works fine.
> 
> 
> Screenshots
> -----------
> 
> new menu
>   http://mattr.info:8080/r/302/s/35/
> 
> 
> Thanks,
> 
> Charles
> 
>

_______________________________________________
kopete-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to