broulik added inline comments.

INLINE COMMENTS

> ktoolbar.cpp:908
>  {
> -#ifndef KXMLGUI_NO_DEPRECATED
> +#if KXMLGUI_BUILD_DEPRECATED_SINCE(5, 0)
>      if (mainWindow() && d->enableContext) {

I just realized this is false-y deprecated. The feature adds the configuration 
options for toolbar buttons.
What needs to be done here for proper deprecation is port that over to 
`customContextMenuRequested` or checking for policy being not `NoContextMenu` 
or so.

REPOSITORY
  R263 KXmlGui

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

To: kossebau, #frameworks, dfaure, mlaurent
Cc: broulik, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

Reply via email to