<table><tr><td style="">dileepsankhla 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/D13203">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>How about changing ui/pagepainter.cpp L668 to</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);">acolor.setAlpha( a->style().opacity() * a->style().color().alpha() );</pre></div>

<p>It gives us transparent background for new typewriter annotations. It should be backwards/forwards compatible with *.okular documents from other Okular versions. And I believe it's more consistent to what happens in Poppler, because I assume in Poppler color[alpha] and opacity will also get multiplied to determine the final alpha of background color for FreeText, if both values are set. But haven't verified this assumption yet.</p></blockquote>

<p>I agree but why do we need to multiply alpha value and opacity to determine the final alpha of background color for FreeText in Poppler? It is already determined by alpha value solely. I mean isn't it enough to change the line to <tt style="background: #ebebeb; font-size: 13px;">acolor.setAlpha( a->style().opacity() * a->style().color().alpha() )</tt> as you suggested? Why do we need to change the Poppler's one?</p>

<p>Secondly, in my opinion, we should change L 628 to <tt style="background: #ebebeb; font-size: 13px;">unsigned int opacity = (unsigned int)( a->style().color().alpha() * a->style().opacity() )</tt> and here the variable <tt style="background: #ebebeb; font-size: 13px;">opacity</tt> will determine the background color of FreeText.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13203">https://phabricator.kde.org/D13203</a></div></div><br /><div><strong>To: </strong>dileepsankhla<br /><strong>Cc: </strong>ltoscano, ngraham, tobiasdeiminger, aacid, okular-devel<br /></div>