davidhurka added inline comments.

INLINE COMMENTS

> pageview.cpp:5705
>  
> +void PageView::slotSetChangeColorsMode( QAction * action )
> +{

Just discovered that the existing slots (slotToggleChangeColors(), 
slotSetChangeColors(bool)) are exposed to D-Bus through Okular::Part.

Probably, there should be another slot slotSetChangeColorsMode(QString or 
similar) to set the color mode from D-Bus. Exposing this slot to D-Bus makes 
little sense, because of the QAction* parameter.

REPOSITORY
  R223 Okular

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

To: davidhurka, #okular, #vdg
Cc: GB_2, davidre, aacid, ngraham, okular-devel, joaonetto, tfella, darcyshen

Reply via email to