D6573: Babelfish plugin: Allow secure URLs to be translated
David Faure
noreply at phabricator.kde.org
Sat Jul 8 13:38:35 UTC 2017
dfaure added a comment.
Looks good, just some details.
INLINE COMMENTS
> plugin_babelfish.cpp:225
> + part->url().scheme().compare(QLatin1String("https"), Qt::CaseInsensitive) == 0 ) &&
> + (part->inherits("KHTMLPart") || part->inherits("KWebKitPart"))) {
> m_menu->setEnabled(true);
This is missing WebEngine btw, but instead this should be generalized with something like
"the part has a browser extension", like this:
&& KParts::BrowserExtension::childObject(part)
> plugin_babelfish.cpp:273
> + if (url.scheme() == "https") {
> + if (KMessageBox::warningContinueCancel(0L,
> + xi18nc("@info", "\
the 0L should be part->widget() instead.
REPOSITORY
R226 Konqueror
REVISION DETAIL
https://phabricator.kde.org/D6573
To: marten, #konqueror, #plasma, dfaure
Cc: plasma-devel, #dolphin, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170708/c0a4bc0c/attachment-0001.html>
More information about the Plasma-devel
mailing list