-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120595/
-----------------------------------------------------------
(Updated Oct. 15, 2014, 10:42 a.m.)
Review request for KDE Frameworks and David Faure.
Changes
-------
added david. it looks OK to me, but he knows more foo. esp. considering the
comment above which says docPath could be a full URL might not work now
anymore...
Repository: kconfigwidgets
Description
-------
During the port of KAppTemplate to KF5 I encountered a problem with the help
invokation by hand - KAppTemplate has a help button in the main window/widget.
These methods were used:
KDE4: KToolInvocation::invokeHelp("kapptemplate-init", "kapptemplate");
KF5: KHelpClient::invokeHelp("kapptemplate-init", "kapptemplate"); +
Application name is set
In KDE 4 it worked without a problem, but as soon it was ported to KF5 it
stopped working. So I debugged the method and saw, that the URL for calling the
help page has this (wrong) format:
http://kapptemplate/index.html?anchor=kapptemplate-index
So, because the help address is an http-Url, I wrote a fix and the final URL
now looks like this:
help:/kapptemplate/index.html?anchor=kapptemplate-index
This URL is KHelpCenter-valid and will call KHelpCenter. You can also take a
look at the else part of that line: If no application is given, _appname is
used and inserted in the correct url.
Diffs
-----
src/khelpclient.cpp 09a3167
Diff: https://git.reviewboard.kde.org/r/120595/diff/
Testing
-------
Testing was done under a project neon 5 system and with the current master
branch of KConfigWidgets from git. Before installing the patched library, the
help menu was not invokable from the KAppTemplate application. After installing
the patching library it was.
Also tested it with Kate help menu item from the "Settings" menu. It is also
callable from "Settings" --> "Configure Kate" --> "Help" (Bottom left").
Thanks,
Simon Wächter
_______________________________________________
Kde-frameworks-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
