<table><tr><td style="">davidhurka marked 2 inline comments as done.<br />davidhurka 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/D21195">View Revision</a></tr></table><br /><div><div><p>I have added CheckableActionMenu, which cuts the default action connection between the KActionMenu itself and the toolbar button. Now it is possible to make the toolbar button checkable, but not the submenu.</p>

<p>But the toolbar button (QToolButton) pops up a menu which I don’t really understand. It is a menu consisting of the default action (ok so far) and the KActionMenu itself as submenu (not ok). <a href="https://phabricator.kde.org/F6824299" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6824299: Screenshot_2019-05-15_15:03:40.png</a></p>

<p>I have one idea how this submenu gets there: KActionMenu::createWidget sets the KActionMenu as default action for the QToolButton.<br />
Now the popup menu works fine. As soon as CheckableActionMenu::createWidget sets a new default action, this menu becomes a submenu (?).</p>

<p>QToolButton::setMenu() has no effect, and QToolButton::menu() always returns nullptr.</p>

<p>So, default action of the toolbar button works, menu works not.</p>

<p>Someone an idea how I can keep the correct menu?</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21195#inline-119107">View Inline</a><span style="color: #4b4d51; font-weight: bold;">pageview.cpp:761</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">a</span><span style="color: #aa2211">-></span><span class="n">setCheckable</span><span class="p">(</span> <span style="color: #304a96">true</span> <span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">a</span><span style="color: #aa2211">-></span><span class="n">setData</span><span class="p">(</span> <span style="color: #aa4000">int</span><span class="p">(</span> <span class="n">id</span> <span class="p">)</span> <span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span class="n">d</span><span style="color: #aa2211">-></span><span class="n">aColorModeMenu</span><span style="color: #aa2211">-></span><span class="n">addAction</span><span class="p">(</span> <span class="n">a</span> <span class="p">);</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Can I use Okular::SettingsCore::EnumRenderMode::type somehow for the action data, without casting it to int?</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D21195#inline-119105">View Inline</a><span style="color: #4b4d51; font-weight: bold;">GB_2</span> wrote in <span style="color: #4b4d51; font-weight: bold;">pageview.cpp:672</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">No, we always use three dots.<br />
This item should have the <tt style="background: #ebebeb; font-size: 13px;">configure</tt> icon BTW.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Ok I’ll add that.</p></div></div></div></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/D21195">https://phabricator.kde.org/D21195</a></div></div><br /><div><strong>To: </strong>davidhurka, Okular, VDG<br /><strong>Cc: </strong>GB_2, davidre, aacid, ngraham, okular-devel, joaonetto, tfella, darcyshen<br /></div>