<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 #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I think this is ok. Two wishes:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The toolbar button shows a checked state, when the fill color is se</li>
</ul></blockquote>

<p>As it is now that button already displays a rectangle with the currect color (or nothing if no fill color). That should be enough to show the user the current state. Having the button as it is now (InstantPopup) seems more usable to me.<br />
Isn't this enough? (Maybe I never posted a screenshot with this displayed)</p>

<p><a href="https://phabricator.kde.org/F7592924" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7592924: Screenshot_20191014_095818.png</a></p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<li class="remarkup-list-item">The dropdown menu shows the last ~5 selected colors and/or the colors of the favourite annotations.</li>
</ul></blockquote>

<p>Well actually showing a list of colors is a good idea (I did put this in the future improvements list, but it is actually not that hard to do).</p>

<p>We can have the following action list:</p>

<p>Color picker: color1, color2, ..., colorN, Custom color...<br />
Inner color picker: color1, color2, ..., colorN, Transparent, Custom color...</p>

<p>We should decide which colors to choose. <br />
The basic option is to provide a palette of standard colors, which seems the solution used by most of the other readers if I am not wrong.<br />
The second option is to list the previously selected colors. How would you fill the list by default? With standard colors?<br />
I would discard the third option of having the colors of the favorite annotations.</p>

<p>I am more inclined towards the first option for the following reasons:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">it is logically simple</li>
<li class="remarkup-list-item">it is what most of the others readers do (so what users are used to)</li>
<li class="remarkup-list-item">If a user has to use custom colors a lot, I expect that it saves those as custom tools in the quick annotation list. E.g. I create a yellow, green, blue highlighters and save them to the quick annotations. Then I think an average user will use those three tools/colors most of the time. I think that situations where one has to use different tools with each different colors (so that the quick annotation list explodes) are quite uncommon. (This is also why I created the quick annotation list, to have the same tool with my custom colors ready to use)</li>
</ul></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>