<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added inline comments.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D16425">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D16425#inline-115032">View Inline</a><span style="color: #4b4d51; font-weight: bold;">suspendsessionconfig.cpp:82</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"> <span class="n">m_suspendThenHibernate<span class="bright">Enabled</span></span> <span style="color: #aa2211">=</span> <span style="color: #aa4000">new</span> <span class="n">QCheckBox</span><span class="p">(</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);"> <span class="n">i18nc</span><span class="p">(</span><span style="color: #766510">"Hibernate after <span class="bright">a delay</span> when suspended"</span><span class="p">,</span> <span style="color: #766510">"Hibernate after <span class="bright">a delay</span> when suspended"</span><span class="p">));</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span class="n">m_suspendThenHibernate<span class="bright">Check</span></span> <span style="color: #aa2211">=</span> <span style="color: #aa4000">new</span> <span class="n">QCheckBox</span><span class="p">(</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span class="n">i18nc</span><span class="p">(</span><span style="color: #766510">"Hibernate after <span class="bright">3 hours</span> when suspended"</span><span class="p">,</span> <span style="color: #766510">"Hibernate after <span class="bright">3 hours</span> when suspended"</span><span class="p">));</span>
</div><div style="padding: 0 8px; margin: 0 4px; "> <span class="n">QWidget</span> <span style="color: #aa2211">*</span><span class="n">tempWidget</span> <span style="color: #aa2211">=</span> <span style="color: #aa4000">new</span> <span class="n">QWidget</span><span class="p">;</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><ol class="remarkup-list">
<li class="remarkup-list-item">The first argument is the context, which doesn't need to be the same string as the actual user-displayed string itself. It's mainly used for very short strings where the string itself may not illustrate on its own what the context is and how it should be translated. Since this string is descriptive enough, I'd just use a regular old <tt style="background: #ebebeb; font-size: 13px;">i18n("blablabla")</tt></li>
</ol>
<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">We now use the word "Sleep" instead of "Suspend", and now that I look at this some more, I think a better string would be, "Sleep for 3 hours, then hibernate"</li>
</ol></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R122 Powerdevil</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16425">https://phabricator.kde.org/D16425</a></div></div><br /><div><strong>To: </strong>avaldes, broulik, ngraham<br /><strong>Cc: </strong>soriano, abalaji, graesslin, ngraham, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>