-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103812/
-----------------------------------------------------------

Review request for kdelibs and David Faure.


Description
-------

KToolBar::Private::slotContextShowText() was assuming that the xmlgui file it 
had to write was
  KGlobal::mainComponent.componentName() + "ui.rc";
which is obviously wrong since we have a setXMLFile function for a reason.

I tried using 
  xmlguiClient->xmlFile()
directly but in Okular we use the same the same toolbar name defined in two xml 
files, so that still did not work because this means we end up with just one 
KToolbar (yes i know that might be a misuse of the API).

So i ended up going through the actioncollections to find the action and get 
the correct client from there.


This addresses bug 292574.
    http://bugs.kde.org/show_bug.cgi?id=292574


Diffs
-----

  kdeui/widgets/ktoolbar.cpp cce242b 

Diff: http://git.reviewboard.kde.org/r/103812/diff/diff


Testing
-------

Fixes the issue in Okular, i tested it does still work with Kate that is using 
the ui.rc scheme.


Thanks,

Albert Astals Cid

Reply via email to