<table><tr><td style="">alex added inline comments.
</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/D29123">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/D29123#inline-166472">View Inline</a><span style="color: #4b4d51; font-weight: bold;">leinir</span> wrote in <span style="color: #4b4d51; font-weight: bold;">installation.cpp:632</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This will want to be more... question like... "The thing failed" isn't really a question, not sure how the user's supposed to make an informed choice based on that... Perhaps something like "The uninstallation process failed to run the command %1. The output was:\n%2\nIf you think this is incorrect, you can continue, or you can cancel the process." (given how much this is an error situation, it feels like we can give the user a bit of technical information... cancelling in a panic would be the appropriate reaction to "I don't know" anyway for this sort of thing, so thinking we'd be ok with doing that).</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Yes I get what you mean :-).</p>

<p style="padding: 0; margin: 8px;">I was just not sure where this patch should land (I asked about this in  a previous comment) and wanted to know this before editing translations.<br />
But I guess it will go on master then?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R304 KNewStuff</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29123">https://phabricator.kde.org/D29123</a></div></div><br /><div><strong>To: </strong>alex, KNewStuff, meven, ngraham, leinir<br /><strong>Cc: </strong>leinir, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>