Re: kio man + khelpcenter

2022-06-17 Thread Martin Koller
On Sonntag, 26. September 2021 21:53:12 CEST you wrote: > Martin Koller ha scritto: > > Hi, > > > > a recent update (openSuse TW, kio-extras5 Version: 21.08.1) had the effect > > that whenever I want to view a man page > > via Alt-F2 (Krunner) and e.g. "#l

Re: Aw: Re: kcm_bluetooth changed ?

2022-02-13 Thread Martin Koller
On Sonntag, 13. Februar 2022 19:27:22 CET alexander.loh...@gmx.de wrote: > Hello, > > Usually the KCMs are opened by starting kcmshell or systemsettings with the > module names as arguments, > meaning we resolve the location of the KCMs internally and don't require > changes for consumers.

Re: kcm_bluetooth changed ?

2022-02-13 Thread Martin Koller
On Sonntag, 13. Februar 2022 14:15:09 CET Fusion Future wrote: > On 2022/2/13 20:20, Martin Koller wrote: > > I already tried this but it did not solve the issue. > > This code snippet works on my system. Please have a try. > > ```cpp > dialog = new KCMultiDialog

Re: kcm_bluetooth changed ?

2022-02-13 Thread Martin Koller
On Sonntag, 13. Februar 2022 04:59:51 CET Fusion Future wrote: > On 2022/2/13 05:46, Martin Koller wrote: > > What has changed and how would I now use the new KCM in a KCMultiDialog ? > > It can be changed to "kcm_bluetooth". Please see [1] to see if that helps. >

kcm_bluetooth changed ?

2022-02-12 Thread Martin Koller
Hi, in liquidshell I start the bluetooth KCM via: dialog = new KCMultiDialog(this); KCModuleInfo module("bluetooth"); if ( module.service() ) dialog->addModule("bluetooth"); this worked until the latest update to 5.24.0 (working means: the KCM content appeared inside the

kipi-plugins question

2021-06-24 Thread Martin Koller
Hi, when using the KIPI mechanism in my application, I get log messages like: kipi.library: This should only be invoked if the host application supports KIPI::Features ( HostSupportsMetadataProcessing ). If host application do support that, then this function should have been overridden in the

duplicate notifications ?

2021-04-01 Thread Martin Koller
Hi, in liquidshell I'm using a NotificationServer implementation which listens on new notifications via dbus "org.freedesktop.Notifications". Some recent KF5 or kmail(?) changes however led now to duplicate notifications. E.g. In the Notify slot I receive the following notification when I send

changing icon sizes no longer emits signal

2020-04-25 Thread Martin Koller
Hi, in liquidshell I'm using connect(KIconLoader::global(), ::iconLoaderSettingsChanged, this, ::adjustIconSize); to be informed whenever the icon sizes get changed in systemsettings. I don't know since when but at least in version Operating System: openSUSE Tumbleweed 20200419 KDE Plasma

D24672: GIT_SILENT Run uncrustify-kf5 on the whole tree

2019-10-15 Thread Martin Koller
mkoller requested changes to this revision. mkoller added a comment. This revision now requires changes to proceed. why ? I don't like lots of these changes. REPOSITORY R374 KolourPaint REVISION DETAIL https://phabricator.kde.org/D24672 To: ahmadsamir, kde-frameworks-devel, mkoller

D7820: man ioslave: spurious numbers included in clang(1) man page

2019-07-18 Thread Martin Koller
mkoller requested changes to this revision. mkoller added a comment. This revision now requires changes to proceed. I think this was solved with bugfix to Bug Nr 335169 with commit 87d8ab4371149c84794f4e93d1f590f7bac5f327

D9033: man ioslave: asserts trying to display pam(8)

2019-07-18 Thread Martin Koller
mkoller accepted this revision. This revision is now accepted and ready to land. REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D9033 To: marten, #plasma, kfm-devel, mkoller Cc: ltoscano, kde-frameworks-devel, apol, plasma-devel, aprcela, fprice, LeGast00n,

D17741: avoid blocking kio_http_cache_cleaner and ensure exit with session

2019-01-07 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R241:a3fce6e0b7f2: avoid blocking kio_http_cache_cleaner and ensure exit with session (authored by mkoller). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D17741?vs=48084=48924#toc REPOSITORY

D17741: avoid blocking kio_http_cache_cleaner and ensure exit with session

2018-12-23 Thread Martin Koller
mkoller updated this revision to Diff 48084. mkoller added a comment. David is right, processEvents() seems not be needed at all - removed it. Actually while testing, I found some other bugs - mainly read/write of the scoreboard and the info from the slaves where one side wrote qint64

D17741: avoid blocking kio_http_cache_cleaner and ensure exit with session

2018-12-22 Thread Martin Koller
mkoller edited the summary of this revision. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D17741 To: mkoller, dfaure Cc: kde-frameworks-devel, michaelh, ngraham, bruns

D17741: avoid blocking kio_http_cache_cleaner and ensure exit with session

2018-12-22 Thread Martin Koller
mkoller created this revision. mkoller added a reviewer: dfaure. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mkoller requested review of this revision. REVISION SUMMARY Currently when logging out, kio_http_cache_cleaner stays alive and blocks in

D15859: allow non-local file:// redirect to a M$-WebDav URL

2018-10-08 Thread Martin Koller
mkoller added a comment. I was talking about the code I changed, which uses QUrl, which can not parse the @SSL syntax. I tried. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D15859 To: mkoller, dfaure, #frameworks Cc: bruns, ngraham, kde-frameworks-devel, michaelh

D15859: allow non-local file:// redirect to a M$-WebDav URL

2018-10-07 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R241:1c7b47cb677b: allow non-local file:// redirect to a Windows WebDav URL (authored by mkoller). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D15859?vs=42626=43070#toc REPOSITORY R241 KIO

D15859: allow non-local file:// redirect to a M$-WebDav URL

2018-10-07 Thread Martin Koller
mkoller added a comment. file://some.server:80/DavWWWRoot/pwa/some/path/to/a/document clearly works since it's standard URL syntax. file://some.server@80/DavWWWRoot/pwa/some/path/to/a/document does not, since it's not a valid URL syntax. 443 is not enabled, so I can't test it, but

D15859: allow non-local file:// redirect to a M$-WebDav URL

2018-10-06 Thread Martin Koller
mkoller added a comment. ping REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D15859 To: mkoller, dfaure, #frameworks Cc: bruns, ngraham, kde-frameworks-devel, michaelh

D15859: allow non-local file:// redirect to a M$-WebDav URL

2018-09-30 Thread Martin Koller
mkoller updated this revision to Diff 42626. mkoller added a comment. M$ -> windows About SSL: I have no idea how an SSL url from a sharepoint server would look like, since it seems the setup we use does not use ssl. The mentioned webdavsystem.com link just describes how it would look

D15859: allow non-local file:// redirect to a M$-WebDav URL

2018-09-30 Thread Martin Koller
mkoller created this revision. mkoller added a reviewer: dfaure. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. mkoller requested review of this revision. REVISION SUMMARY In our mainly windows company environment, I often receive links in emails like this

D8519: do not make the context menu a Window

2017-12-17 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R289:f2f65dee2962: do not make the context menu a Window (authored by mkoller). REPOSITORY R289 KNotifications CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8519?vs=21431=23999 REVISION

D8519: do not make the context menu a Window

2017-12-17 Thread Martin Koller
mkoller added a comment. Ping! Anything wrong with this 2 liner patch ? REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D8519 To: mkoller, davidedmundson, graesslin Cc: #frameworks

D9249: forward socket error string to KTcpSocket

2017-12-08 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R241:84f290a2659d: forward socket error string to KTcpSocket (authored by mkoller). REPOSITORY R241 KIO CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9249?vs=23638=23648 REVISION DETAIL

D8519: do not make the context menu a Window

2017-12-08 Thread Martin Koller
mkoller retitled this revision from "do not make the context menu a Window; do not force raise a window" to "do not make the context menu a Window". mkoller edited the summary of this revision. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D8519 To: mkoller,

D9249: forward socket error string to KTcpSocket

2017-12-08 Thread Martin Koller
mkoller created this revision. mkoller added a reviewer: dfaure. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY KTcpSocket is a QIODevice, internally using a Qt socket - however it does not set the error string of

D9182: return nullptr -> return {} for QFlags

2017-12-05 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R287:7f2c44add4aa: Use brace-initializer instead of nullptr in returning 0-QFlags (authored by mkoller). REPOSITORY R287 KImageFormats CHANGES SINCE LAST UPDATE

D9182: return nullptr -> return {} for QFlags

2017-12-04 Thread Martin Koller
mkoller created this revision. mkoller added reviewers: kfunk, kossebau. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Change the irritating nullptr return value to {} for the QImageIOPlugin::Capabilities QFlags

D3987: Use nullptr in all Frameworks (just diff in KIO shown here)

2017-12-03 Thread Martin Koller
mkoller added a comment. I just found out that this change introduced wrong replacements. The changes in frameworks/kimageformats/src/imageformats like the following are incorrect, since the return type is not a pointer but an enum flags type, so it should still be 0, not nullptr. @@

D8519: do not make the context menu a Window; do not force raise a window

2017-10-27 Thread Martin Koller
mkoller updated this revision to Diff 21431. mkoller added a comment. In https://phabricator.kde.org/D8519#160813, @davidedmundson wrote: > > The removal of the force activation is done since according to kwin maintainer it is wrong and must be done by the SysTray itself. > > But

D8519: do not make the context menu a Window; do not force raise a window

2017-10-27 Thread Martin Koller
mkoller created this revision. mkoller added reviewers: davidedmundson, graesslin. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY The setParent() call in this patch was changing the window flags from Popup to Window,

D8416: really raising window when shown from systray

2017-10-26 Thread Martin Koller
mkoller abandoned this revision. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D8416 To: mkoller, davidedmundson, graesslin Cc: #frameworks

D8401: fix URL detection with double urls like "http://www.foo.bar"

2017-10-23 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R244:095a75eeb212: fix URL detection with double urls like http://www.foo.barhttp://foo.bar/; (authored by mkoller). REPOSITORY R244 KCoreAddons CHANGES SINCE LAST UPDATE

D8416: really raising window when shown from systray

2017-10-23 Thread Martin Koller
mkoller added inline comments. INLINE COMMENTS > graesslin wrote in kstatusnotifieritem.cpp:653 > It only works on X11 because KWin has a bug. KWin allows the force (which is > in X11 Terms a "from tool" request) without checking who sends it. I will > look into hardening this in KWin. Then

D8416: really raising window when shown from systray

2017-10-22 Thread Martin Koller
mkoller added inline comments. INLINE COMMENTS > graesslin wrote in kstatusnotifieritem.cpp:653 > The correct approach would be sni telling the host to activate the window. > The host could then communicate with the X11 window manager/Wayland > compositor through a special interface that the

D8416: really raising window when shown from systray

2017-10-22 Thread Martin Koller
mkoller added inline comments. INLINE COMMENTS > graesslin wrote in kstatusnotifieritem.cpp:653 > The other code is also not correct and should be removed. Do you mind telling me the correct approach ? REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D8416 To:

D8416: really raising window when shown from systray

2017-10-22 Thread Martin Koller
mkoller added inline comments. INLINE COMMENTS > graesslin wrote in kstatusnotifieritem.cpp:653 > Please read the documentation of this API call. It is not meant for > applications to use this! > > Please also note that this won't work on Wayland at all. > > The proper way is to ask the

D8416: really raising window when shown from systray

2017-10-22 Thread Martin Koller
mkoller created this revision. mkoller added reviewers: davidedmundson, graesslin. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY I have akregator running and an icon is shown in the systray. When the main window is

D8401: fix URL detection with double urls like "http://www.foo.bar"

2017-10-21 Thread Martin Koller
mkoller created this revision. mkoller added a reviewer: mlaurent. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY This fixes URL detection for cases where one url with enclosing brackets follows one without, which

KCModuleContainer

2017-10-11 Thread Martin Koller
Hi, I'm trying to use KCModuleContainer to show 3 kcm modules, however this class constantly crashes when switching tabs. checking the sourcecode, I see the reason: void KCModuleContainer::tabSwitched(int index) ... setAboutData(mod->aboutData()); but since the base class KCModule always

D6941: fix plasmashell unsetting QSG_RENDER_LOOP

2017-07-27 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R296:886dd7aaa33d: fix plasmashell unsetting QSG_RENDER_LOOP (authored by mkoller). REPOSITORY R296 KDeclarative CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6941?vs=17258=17276 REVISION

D6941: fix plasmashell unsetting QSG_RENDER_LOOP

2017-07-26 Thread Martin Koller
mkoller created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY being hit by the problem of plasmashell using way too much CPU even when doing "nothing", I found the description of the QSG_RENDER_LOOP

D5774: speed up detail treeview display by avoiding too many column resizes

2017-05-13 Thread Martin Koller
This revision was automatically updated to reflect the committed changes. Closed by commit R241:10e53ca3b2a7: speed up detail treeview display by avoiding too many column resizes (authored by mkoller). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D5774?vs=14300=14461#toc REPOSITORY

D5774: speed up detail treeview display by avoiding too many column resizes

2017-05-13 Thread Martin Koller
mkoller added inline comments. INLINE COMMENTS > dfaure wrote in kdiroperatordetailview.cpp:156 > Is there any risk that this connect is done multiple times? To be honest, I don't know. I'll add Qt::UniqueConnection just to be safe REPOSITORY R241 KIO REVISION DETAIL

D5774: speed up detail treeview display by avoiding too many column resizes

2017-05-08 Thread Martin Koller
mkoller created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY This patch basically delays the connect to the layoutChanged model signal until it emits its completed() signal. Therefore the many calls

Re: KConfigDialogManager and QButtonGroup

2017-05-06 Thread Martin Koller
On Samstag, 6. Mai 2017 13:48:00 CEST David Faure wrote: > On samedi 6 mai 2017 11:41:41 CEST Martin Koller wrote: > > On Samstag, 6. Mai 2017 11:20:06 CEST David Faure wrote: > > > On lundi 1 mai 2017 20:31:28 CEST Martin Koller wrote: > > > > from Qt4 days, where

Re: KConfigDialogManager and QButtonGroup

2017-05-06 Thread Martin Koller
On Dienstag, 2. Mai 2017 11:30:04 CEST Albert Astals Cid wrote: > El dilluns, 1 de maig de 2017, a les 20:31:28 CEST, Martin Koller va escriure: > > Hi, > > > > trying to fix https://bugs.kde.org/show_bug.cgi?id=378930 > > > > I see that the mboxresource has

Re: KConfigDialogManager and QButtonGroup

2017-05-06 Thread Martin Koller
On Samstag, 6. Mai 2017 11:20:06 CEST David Faure wrote: > On lundi 1 mai 2017 20:31:28 CEST Martin Koller wrote: > > from Qt4 days, where the groupBox > > did also manage the buttons, whereby now you need a QButtonGroup class. > > That's a Qt3-Qt4 change, not a Qt4-Qt5 chang

KWallet::openWallet and 0-WinId ?

2017-05-01 Thread Martin Koller
Hi, the Wallet::openWallet() documentation says for the WinId argument: "The window id to associate any dialogs with. You can pass 0 if you don't have a window the password dialog should associate with." However passing 0 gives a runtime warning qDebug() << "Pass a valid window to

KConfigDialogManager and QButtonGroup

2017-05-01 Thread Martin Koller
Hi, trying to fix https://bugs.kde.org/show_bug.cgi?id=378930 I see that the mboxresource has a configuration dialog which uses KConfigDialogManager to handle the settings, but it has a QGroupBox widget holding QRadioButtons, which represent one setting (e.g. one of 2 possible compaction

KDirWatch issue

2017-04-25 Thread Martin Koller
Hi, just wondering if I'm doing something wrong or there really is a bug in KDirWatch (I'm on openSuse Leap 42.2, KF5 5.33.0) From the docs I understand that when I call KDirWatch::stopDirScan(dir); ... create files in the dir ... KDirWatch::restartDirScan(dir) it should not emit the dirty

Re: Qt5 version of qimageblitz

2016-03-06 Thread Martin Koller
On Sunday 06 March 2016 12:35:32 Albert Astals Cid wrote: > El Sunday 06 March 2016, a les 08:38:14, Boudhayan Gupta va escriure: > > On 6 March 2016 at 01:26, Martin Koller <kol...@aon.at> wrote: > > > Who is in charge of the old SVN repos ? > > > Who is in c

Re: What to do with KScanDialog

2013-09-01 Thread Martin Koller
On Saturday 31 August 2013 23:11:57 Kåre Särs wrote: I would vote to move it to kde4support and deprecate it. I can volunteer to port kolourpaint to libksane if needed. Hi all, I'm the current maintainer of kolourpaint. I have no problem with using anything else than what's currently