Review Request: Turn config app into a KCModule

[Florian Reinhard]

> On March 14, 2011, 10:47 p.m., Florian Reinhard wrote:
> > config/kcm_telepathy_chat_config.desktop, line 1
> > <http://git.reviewboard.kde.org/r/100858/diff/1/?file=11382#file11382line1>
> >
> >     the .desktop file needs a name "Chat Window Design", otherwise systemsettings shows kcm_telepathy_chat_config as fallback.
> 
> Martin Klapetek wrote:
>     Just a small nitpick in this nitpick: instead of "Design", there should be "Appearance" ;)

and the icon could be apps/preferences-desktop-theme.png


- Florian


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


On March 14, 2011, 7:24 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100858/
> -----------------------------------------------------------
> 
> (Updated March 14, 2011, 7:24 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Summary
> -------
> 
> Turn config app into a KCModule, changed the chat-ui app to load it when requested.
> 
> 
> This addresses bug 268136.
>     http://bugs.kde.org/show_bug.cgi?id=268136
> 
> 
> Diffs
> -----
> 
>   app/chat-window.cpp c0e825c0cedba9cd32f5834edf335c414b6bc149 
>   config/CMakeLists.txt 2b944693e5fe684ccb1b426aaf7ee12f51c317a2 
>   config/chatwindowconfig.ui c0572229545287a7a2453c99d3e6046a3c51b8b3 
>   config/kcm_telepathy_chat_config.desktop PRE-CREATION 
>   config/main-window.h 4bb91477c30f15c0dc5b7f93cf0e8faf8957443d 
>   config/main-window.cpp 7e28346de2684ea356c838b1e910b18f67ceac3f 
> 
> Diff: http://git.reviewboard.kde.org/r/100858/diff
> 
> 
> Testing
> -------
> 
> Checked it loaded. Checked it still saved settings correctly
> 
> 
> Thanks,
> 
> David
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110315/23c7f263/attachment.htm