<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>Might it be a hint that there are too many toolbars? I think only the part should have the annotation toolbar, so it is clear which is affected by the hide action.</p></blockquote>

<p>Not defining the annotation toolbar also in <tt style="background: #ebebeb; font-size: 13px;">shell.rc</tt> makes the implementation of the action to toggle the toolbar more difficult (if not impossible) given the tricky way in which the toolbars are created.<br />
Currently I create the action <tt style="background: #ebebeb; font-size: 13px;">showAnnotationToolBar</tt> when the part is created, but at this point the UI of the part does not exist yet, so the annotation toolbar must already exist (and it is the one in <tt style="background: #ebebeb; font-size: 13px;">shell.rc</tt>). Creating the action later in the code creates other problems, so the current implementation is sanest I could think of.</p>

<hr class="remarkup-hr" />

<p>Now the annotation toolbar is toggled in the correct window.</p>

<p>I should have fixed all the bugs that were found by the testers.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>BRANCH</strong><div><div>new-annotation-toolbar_ToggleActionMenu</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, ngraham<br /><strong>Cc: </strong>kde-doc-english, andreashurka, yurchor, trickyricky26, simgunz, ltoscano, cfeck, aacid, davidhurka, knambiar, ngraham, tobiasdeiminger, okular-devel, johnzh, andisa, siddharthmanthan, maguirre, gennad, fbampaloukas, joaonetto, kezik, tfella, skadinna, darcyshen<br /></div>