<table><tr><td style="">ochurlaud 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/D16508">View Revision</a></tr></table><br /><div><div><p>Just to be sure: you just factored both treatment in one?  Or do I miss something?</p>

<p>If it is just factoring, you can ship it. If not, I'd like to be sure not to miss anything in the change.</p>

<p>The idea was to always do:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">1) load resource
 1a) if color, use it
 1b) if invalid color,
   1b1) try to use the one from the dav
   1b2) if none in the dav, define one in resources and load resource</pre></div>

<p>It seems you do exactly that (which I thought I was doing, but with a clumsy way)</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16508">https://phabricator.kde.org/D16508</a></div></div><br /><div><strong>To: </strong>dfaure, ochurlaud, dkurz, dvratil<br /><strong>Cc: </strong>KDE PIM<br /></div>