Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plasma 6 compatibility #73

Open
guiodic opened this issue Jan 22, 2024 · 40 comments
Open

Plasma 6 compatibility #73

guiodic opened this issue Jan 22, 2024 · 40 comments

Comments

@guiodic
Copy link

guiodic commented Jan 22, 2024

Hi, is an update planned for Plasma 6?

Thank you!

@guiodic
Copy link
Author

guiodic commented Feb 28, 2024

Plasma 6 was released today. I hope this decoration can be updated to the new DE. Thank you!

@guiodic
Copy link
Author

guiodic commented Mar 14, 2024

Sorry, a gentle bump :)

@AntarEspadas
Copy link

I am also curious if an update is planned

@abolast
Copy link

abolast commented Mar 21, 2024

I regret not having learned C++. C++ is too difficult.
Similarly, this tool can improve my work efficiency.

@guiodic
Copy link
Author

guiodic commented Apr 2, 2024

I am seeing that @Zren is prioritizing widgets, I hope he has time for this decoration, much of the work I think has already been done. I really can't get by without the LIM!

@nicman23
Copy link

nicman23 commented Apr 9, 2024

I am independently working on it, I just need to figure out kconfig but it is so boring as I do not use the kconfig menu, as I always just changed the source code

@guiodic
Copy link
Author

guiodic commented Apr 9, 2024 via email

@guiodic
Copy link
Author

guiodic commented Apr 11, 2024

@nicman23 do you have any news? Is it possible to test the decoration?

@nicman23
Copy link

sorry guys it is a tough month, i might just upload whatever i have already done but yeah i have not worked at it at all

@guiodic
Copy link
Author

guiodic commented Apr 11, 2024

Thank you very much! I dont care kconfig for now, default options are ok for me, I will eventually change source code. Or, if present, the values in kwinrc.

@guiodic
Copy link
Author

guiodic commented Apr 15, 2024

@nicman23 sorry to bother you but... any news?

@nicman23
Copy link

sorry here is the wip code https://github.com/nicman23/material-decoration but still no progress since my first message

@guiodic
Copy link
Author

guiodic commented Apr 19, 2024

thank you. I tried to build it. I receive an error on qtdbusmenu so I tried commenting the line in cmake but then I receive another error on "FindQt.cmake".
Well, I will wait...

@Anarcraft
Copy link

On my test machine I don't see a package named "Qt6" which this would parse to:
find_package(QT NAMES Qt${QT_VERSION_MAJOR})
It may be the package named "Qt6-base".

@guiodic
Copy link
Author

guiodic commented Apr 19, 2024

On my test machine I don't see a package named "Qt6" which this would parse to: find_package(QT NAMES Qt${QT_VERSION_MAJOR}) It may be the package named "Qt6-base".

I presume "package" in cmake refers to QT internal names, not to the packages from the distribution.

@Anarcraft
Copy link

On my test machine I don't see a package named "Qt6" which this would parse to: find_package(QT NAMES Qt${QT_VERSION_MAJOR}) It may be the package named "Qt6-base".

I presume "package" in cmake refers to QT internal names, not to the packages from the distribution.

Oh, OK, I don't know much about cmake.

@guiodic
Copy link
Author

guiodic commented Apr 23, 2024

@nicman23 sorry to bother you again ...

@nicman23
Copy link

i ll take a crack at it if i have no work today

@nicman23
Copy link

nicman23 commented Apr 24, 2024

i did make it compile but it does not show up in system config. if at all possible @Zren could you take a look?

it is not buy any way ready - i have removed the configuration for now - but it should had worked (?)

just for reference it compiles with qt6 and kf6

@guiodic
Copy link
Author

guiodic commented Apr 24, 2024

thank you. I still receive this error:

CMake Error at /usr/lib/cmake/Qt6DBus/Qt6DBusMacros.cmake:64 (qt5_add_dbus_interface):
Unknown CMake command "qt5_add_dbus_interface".
Call Stack (most recent call first):
src/libdbusmenuqt/CMakeLists.txt:24 (qt_add_dbus_interface)

@guiodic
Copy link
Author

guiodic commented Apr 24, 2024

and if I comment the qt dbusmenu including in cmake, thei I receive this other error:
CMake Error at /usr/lib/cmake/Qt6Qml/Qt6QmlMacros.cmake:2829 (message):
qt_generate_foreign_qml_types() is only available in Qt 6.
Call Stack (most recent call first):
/usr/lib/cmake/Qt6Qml/Qt6QmlConfig.cmake:145 (include)
/usr/share/cmake/Modules/CMakeFindDependencyMacro.cmake:76 (find_package)
/usr/lib/cmake/KF6Config/KF6ConfigConfig.cmake:53 (find_dependency)
/usr/share/ECM/find-modules/FindKF6.cmake:52 (find_package)
src/CMakeLists.txt:13 (find_package)

(I'm on Manjaro unstable, it's pretty the same of arch)

@guiodic
Copy link
Author

guiodic commented Apr 24, 2024

p.s. I use this command to build:
cmake -DCMAKE_INSTALL_PREFIX=/usr ..

@nicman23
Copy link

try cmake -DQT_MAJOR_VERSION=6 -DCMAKE_INSTALL_PREFIX=/usr ..

@guiodic
Copy link
Author

guiodic commented Apr 25, 2024

yes, it works! thank you! but it has not shadow

@guiodic
Copy link
Author

guiodic commented Apr 25, 2024

i tried to add this to kdecoration_materialrc

[Common]
MenuAlwaysShow=true
ShadowStrength=128

but the shadow doesn't show. Anyway the menu works

@nicman23
Copy link

nicman23 commented Apr 25, 2024

what? tf, it does not work on my machine :D

E: huh the .so is installed in the wrong folder. i do not care to look into it right now, i just did a

cp src/materialdecoration.so /usr/lib/qt6/plugins/org.kde.kdecoration2/

and it worked lol

@guiodic
Copy link
Author

guiodic commented Apr 26, 2024

@nicman23 I hope the shadows can be restored.

@nicman23
Copy link

i do not use them so probably eh maybe

@guiodic
Copy link
Author

guiodic commented Apr 27, 2024

i do not use them so probably eh maybe

Please 🙏🙏🙏

@guiodic
Copy link
Author

guiodic commented Apr 30, 2024

@nicman23 can you at least make some suggestions? I tried fiddling with the code but couldn't solve the shadow problem. Thank you!

@Zren some love for this bug? Thanks!

@guiodic
Copy link
Author

guiodic commented May 6, 2024

Sorry again @nicman23 ...

@guiodic
Copy link
Author

guiodic commented May 12, 2024

To restore shadows, in the Decoration.cc, line 567

if (!s_cachedShadow ---> if (s_cachedShadow

@guiodic
Copy link
Author

guiodic commented May 23, 2024

Is there any chance of this decoration being ported to Wayland?

@nicman23
Copy link

no

@guiodic
Copy link
Author

guiodic commented May 24, 2024

no

maybe @Zren ?

@nicman23
Copy link

no i mean it is currently not possible

@guiodic
Copy link
Author

guiodic commented May 24, 2024

no i mean it is currently not possible

Can you explain why? Thank you!

@HerrCraziDev
Copy link

no i mean it is currently not possible

Can you explain why? Thank you!

Because Wayland sucks. It's much more restrictive than X11.

Also, any progress on this ?

@guiodic
Copy link
Author

guiodic commented Jun 6, 2024

no i mean it is currently not possible

Can you explain why? Thank you!

Because Wayland sucks. It's much more restrictive than X11.

Also, any progress on this ?

I agree on Wayland sucks etc. but in this particular case I think there are no major problems, in fact the global menu and the menu button in the title bar work correctly (at least for QT apps, the GTK ones have a problem upstream)

@HerrCraziDev
Copy link

Damn I'll have to switch to Debian /s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants