<table><tr><td style="">simgunz 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/D15580">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D15580#546492" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15580#546492</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Typically the color chooser includes a "transparent" item.</p></div>
</blockquote>

<p>The things are trickier here. In Okular the opacity of the annotation is common to the whole annotation, i.e. we cannot have an opacity value for the border color and one for the inner color. This means that the opacity needs to be managed by itself (e.g we cannot just show the alpha channel in the two color pickers) using the <tt style="background: #ebebeb; font-size: 13px;">QColorDialog</tt> option <tt style="background: #ebebeb; font-size: 13px;">ShowAlphaChannel</tt>.</p>

<p>If we show the alpha channel only for the inner color picker we still have an undefined situation because the user can either choose a color with opacity 100% or 0% everything in between is invalid. If he chooses 0% we can remove the fill color.</p>

<p>If we do not show the alpha channel I think we cannot display an option to select the transparent color.</p>

<p>That is why I came up with the solution of the submenu in the inner color picker action.</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/D15580">https://phabricator.kde.org/D15580</a></div></div><br /><div><strong>To: </strong>simgunz, Okular, VDG<br /><strong>Cc: </strong>simgunz, ltoscano, cfeck, aacid, davidhurka, knambiar, ngraham, tobiasdeiminger, okular-devel, andisa, siddharthmanthan, maguirre, fbampaloukas, joaonetto, kezik, tfella, darcyshen<br /></div>