
<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Mar 12, 2016, at 7:38 AM, Sebastian Kügler <<a href="mailto:sebas@kde.org" class="">sebas@kde.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">On Friday, March 11, 2016 03:00:26 Tomaz Canabrava wrote:<br class=""><blockquote type="cite" class="">error messages in red.<br class=""></blockquote><br class="">Brrrrr. Encapsulate that in a small method? So it can be more easily changed <br class="">without editing a lot of HTML in translated messages that would all have to be <br class="">re-checked if the HTML bits change? (Also less prone to errors by <br class="">translators.)<br class=""></div></blockquote><br class=""></div><div>Did you have a chance to look at the version of that patch that actually</div><div>made it upstream? I didn't use a function but a macro instead and kept</div><div>all the markup outside of the translated strings. Maybe not as elegant</div><div>as a function, but I think it reaches the same goals as you had in mind...</div><div><br class=""></div><div>commit <span style="color: rgb(175, 173, 36); font-family: Menlo; font-size: 11px;" class="">fcb308fdf2be</span></div><div><span style="color: rgb(175, 173, 36); font-family: Menlo; font-size: 11px;" class=""><br class=""></span></div><div>the diff in our git web (with shortened as the URL was >100 chars)</div><div><br class=""></div><div><a href="http://bit.ly/24WQWgR" class="">http://bit.ly/24WQWgR</a></div><div><br class=""></div><div>/D</div><br class=""></body></html>