Review Request 110260: Plugin to set status "Away" when screen gets locked.

[Martin Klapetek]

> On May 1, 2013, 9:08 p.m., Martin Klapetek wrote:
> > config/telepathy-kded-config.cpp, line 91
> > <http://git.reviewboard.kde.org/r/110260/diff/3/?file=141623#file141623line91>
> >
> >     I wonder if we should cache this i18n string so the translators don't have to translate it 3x (not your fault, I'll probably do it in a separate patch).
> 
> Lucas Betschart wrote:
>     They don't have to. Or at least I think they don't have to, because the new entry already appears translated in my UI (it's default German).
>     So no changes needed.

Yeah I'd expect just that, but I've been advised many times to not duplicate i18n strings as they might get translated differently...*shrug*


- Martin


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


On May 1, 2013, 9:57 p.m., Lucas Betschart wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110260/
> -----------------------------------------------------------
> 
> (Updated May 1, 2013, 9:57 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Plugin to set status "Away" when screen gets locked.
> Forked from autoaway.h/cpp.
> 
> 
> This addresses bug https://bugs.kde.org/show_bug.cgi?id=290998.
>     http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=290998
> 
> 
> Diffs
> -----
> 
>   telepathy-module.h 748437d6268fce49ef276ab996d18cbcb93da025 
>   telepathy-module.cpp d3f223a8391e232c74a5a03b9277f6c518c4d2ed 
>   screensaveraway.cpp PRE-CREATION 
>   screensaveraway.h PRE-CREATION 
>   config/telepathy-kded-config.ui 54ebc542d7d04c14c7122d2ca3c2e0533b2c3b21 
>   config/telepathy-kded-config.cpp 50c176d583aafdeedcf70442d58336afb5db3c19 
>   CMakeLists.txt a339495773875954b668875a28c074f7ac7f733f 
>   config/telepathy-kded-config.h b1bf626e0fbb9fc0705d95c9d190be5023ace876 
> 
> Diff: http://git.reviewboard.kde.org/r/110260/diff/
> 
> 
> Testing
> -------
> 
> Manually by myself.
> 
> 
> Thanks,
> 
> Lucas Betschart
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130501/a3219ab8/attachment.html>