simgunz added a comment.
It does not compile indeed. My fault in the review process, I missed the last commit. I have added the inline comments to fix the bug. INLINE COMMENTS > pageview.cpp:660 > + d->aMouseModeMenu = new ToggleActionMenu( QIcon(),QString(), this, > + QToolButton::MenuButtonPopup, > + > ToggleActionMenu::ImplicitDefaultAction ); Replace `QToolButton::MenuButtonPopup` with `ToggleActionMenu::MenuButtonPopup`. > toggleactionmenu.h:66 > + MenuButtonPopup > + } > + Missing semicolon after enum declaration REPOSITORY R223 Okular BRANCH create-configurable-toggleactionmenu REVISION DETAIL https://phabricator.kde.org/D21971 To: davidhurka, simgunz Cc: aacid, ngraham, simgunz, okular-devel, johnzh, andisa, siddharthmanthan, maguirre, fbampaloukas, joaonetto, kezik, tfella, darcyshen
