<table><tr><td style="">vitali added a comment.
</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/D5729" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>So, what do we want to do with this patch?<br />
I've been using it for a week now, and didn't notice any problems. Has anyone else been testing it?</p>

<p>As for the link, I am for keeping the http version, as it is the one used by NatGeo, so it would be less likely to be broken in the near future. If and when they will move to TLS it should be their responsibility to issue the right redirect.<br />
Anyway, any decision will be fine for me, as long as it works.</p>

<p>Lastly, any opinion on the regex?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R114 Plasma Addons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5729" rel="noreferrer">https://phabricator.kde.org/D5729</a></div></div><br /><div><strong>To: </strong>vitali, sebas<br /><strong>Cc: </strong>Zren, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas<br /></div>