[lattedock] [Bug 418559] QRegion is not registered to DBus
https://bugs.kde.org/show_bug.cgi?id=418559 Michail Vourlakos changed: What|Removed |Added Latest Commit|https://commits.kde.org/lat |https://commits.kde.org/lat |te-dock/a7f786d87f1df4fd5a0 |te-dock/f31cc660b4dfdec4c64 |83175839e73c2ab519624 |d0f269f43366a0ea8b2b2 --- Comment #5 from Michail Vourlakos --- Git commit f31cc660b4dfdec4c64d0f269f43366a0ea8b2b2 by Michail Vourlakos. Committed on 07/03/2020 at 08:20. Pushed by mvourlakos into branch 'master'. send QRegion through dbus as QList M +10 -1app/plasma/extended/screengeometries.cpp https://commits.kde.org/latte-dock/f31cc660b4dfdec4c64d0f269f43366a0ea8b2b2 -- You are receiving this mail because: You are watching all bug changes.
[lattedock] [Bug 418559] QRegion is not registered to DBus
https://bugs.kde.org/show_bug.cgi?id=418559
--- Comment #4 from tr...@yandex.com ---
(In reply to Michail Vourlakos from comment #3)
> (In reply to trmdi from comment #1)
> > That method wants a QList argument, not a QRegion one. You have to
> > convert QRegion to QList.
> > See:
> > https://phabricator.kde.org/source/plasma-workspace/browse/master/shell/
> > strutmanager.cpp;38757fb52d81ff4bb3a923ff1d3dc4f512095a0c$87
>
> Can you please check that everything is fine with
> https://phabricator.kde.org/source/latte-dock/browse/master/app/plasma/
> extended/screengeometries.cpp$95 , how it sends the dbus data to plasma ?
With this commit, plasmashell doesn't get the data.
I think you could revert this commit, then do like this:
1, add qDBusRegisterMetaType>() in the init method.
2, Modify that part like this:
> m_lastAvailableRegion[scrName] = availableRegion;
> QList rects;
> foreach (const QRect &rect, availableRegion) {
> rects << rect;
> }
> plasmaStrutsIface.call("setAvailableScreenRegion",
> LATTESERVICE, scrName, QVariant::fromValue(rects));
> qDebug() << " PLASMA SCREEN GEOMETRIES AVAILABLE REGION ::
> " << screen->name() << " : " << availableRegion;
--
You are receiving this mail because:
You are watching all bug changes.
[lattedock] [Bug 418559] QRegion is not registered to DBus
https://bugs.kde.org/show_bug.cgi?id=418559 --- Comment #3 from Michail Vourlakos --- (In reply to trmdi from comment #1) > That method wants a QList argument, not a QRegion one. You have to > convert QRegion to QList. > See: > https://phabricator.kde.org/source/plasma-workspace/browse/master/shell/ > strutmanager.cpp;38757fb52d81ff4bb3a923ff1d3dc4f512095a0c$87 Can you please check that everything is fine with https://phabricator.kde.org/source/latte-dock/browse/master/app/plasma/extended/screengeometries.cpp$95 , how it sends the dbus data to plasma ? -- You are receiving this mail because: You are watching all bug changes.
[lattedock] [Bug 418559] QRegion is not registered to DBus
https://bugs.kde.org/show_bug.cgi?id=418559 Michail Vourlakos changed: What|Removed |Added Latest Commit||https://commits.kde.org/lat ||te-dock/a7f786d87f1df4fd5a0 ||83175839e73c2ab519624 Status|REPORTED|RESOLVED Resolution|--- |FIXED --- Comment #2 from Michail Vourlakos --- Git commit a7f786d87f1df4fd5a083175839e73c2ab519624 by Michail Vourlakos. Committed on 07/03/2020 at 05:47. Pushed by mvourlakos into branch 'master'. change QRegion to QList for dbus plasma call --change how dbus arguments are sent to Plasma and a QList arguments for this M +18 -2app/plasma/extended/screengeometries.cpp https://commits.kde.org/latte-dock/a7f786d87f1df4fd5a083175839e73c2ab519624 -- You are receiving this mail because: You are watching all bug changes.
[lattedock] [Bug 418559] QRegion is not registered to DBus
https://bugs.kde.org/show_bug.cgi?id=418559 tr...@yandex.com changed: What|Removed |Added CC||tr...@yandex.com --- Comment #1 from tr...@yandex.com --- That method wants a QList argument, not a QRegion one. You have to convert QRegion to QList. See: https://phabricator.kde.org/source/plasma-workspace/browse/master/shell/strutmanager.cpp;38757fb52d81ff4bb3a923ff1d3dc4f512095a0c$87 -- You are receiving this mail because: You are watching all bug changes.
