simgunz added inline comments.

INLINE COMMENTS

> davidhurka wrote in pageview.h:75
> In the meantime I learned more about this.
> 
> The Color Mode menu will stay here in PageView::setupViewerActions(), because 
> it is needed only in viewer modes, and is page view related.
> 
> m_embedMode is not needed, that’s handled by Part.
> 
> But I still don’t like the parameter part, any comments?

You can add an action in `Part::setupActions` similar to 
`options_configure_annotations`. Then in you can access it with something 
similar to

  QAction * aConfigColors = actionCollection()->action( 
QStringLiteral("options_configure_colors") );
  d->aColorModeMenu->addAction( aConfigColors );

REPOSITORY
  R223 Okular

REVISION DETAIL
  https://phabricator.kde.org/D21195

To: davidhurka, #okular, #vdg
Cc: simgunz, GB_2, aacid, ngraham, okular-devel, johnzh, andisa, 
siddharthmanthan, maguirre, fbampaloukas, joaonetto, kezik, tfella, darcyshen

Reply via email to