termoid icon indicating copy to clipboard operation
termoid copied to clipboard

Update for KDE Plasma 6 and Qt6

Open JonathanxD opened this issue 1 year ago • 10 comments

This depends on Tofee/qmltermwidget@1e7ab36e9894f9d7c5b7d82267faba1e60efd635, which has PR open on upstream: Swordfish90/qmltermwidget#41.

jq is a dependency for building because KDE Plasma 6 changed its metadata format to json.

X-KDE-PluginInfo-KdeStoreId was I removed because desktoptojson complained that it didn't know about this property, although it still work with the property regardless.

If you want to try building it, clone this branch of qmltermwidget and follow the regular instructions for building, but replace qmake with qmake6:

$ git clone -b tofe/qt6 https://github.com/Tofee/qmltermwidget/
$ cd qmltermwidget
$ qmake6 && make
$ sudo make install

Then build my changes, install and run:

$ plasmawindowed com.koldbyte.kde.termoid

JonathanxD avatar Feb 28 '24 23:02 JonathanxD

i got the updated version to build on Fedora40 but the general section looks broken image

and i get

qt.gui.imageio: libpng warning: iCCP: CRC error
qt.gui.imageio: libpng warning: iCCP: CRC error
qt.gui.imageio: libpng warning: iCCP: CRC error
qt.gui.imageio: libpng warning: iCCP: CRC error

not sure if this is a problem on my side

Yixuan2020 avatar May 02 '24 21:05 Yixuan2020

i got the updated version to build on Fedora40 but the general section looks broken image

and i get

qt.gui.imageio: libpng warning: iCCP: CRC error
qt.gui.imageio: libpng warning: iCCP: CRC error
qt.gui.imageio: libpng warning: iCCP: CRC error
qt.gui.imageio: libpng warning: iCCP: CRC error

not sure if this is a problem on my side

@Yixuan2020 The properties of SpinBox has been changed in Qt6, I will do a patch base on @JonathanxD 's PR.

dhay3 avatar May 20 '24 09:05 dhay3

i also tired to update it but couldn't figure the spinbox out

Yixuan2020 avatar May 20 '24 11:05 Yixuan2020

i also tired to update it but couldn't figure the spinbox out

You could try my PR dhay3/termoid.

dhay3 avatar May 20 '24 11:05 dhay3

I didn't notice that the configuration was broken, I still use it daily but never had to change the configuration.

@dhay3 thanks for the patch, I'll try it out and merge into my branch if it works.

JonathanxD avatar May 20 '24 14:05 JonathanxD

Thank you, this works!

dri3sp avatar Jun 02 '24 12:06 dri3sp

Safe to say this is abandoned. Might as well fork it and call it "termoid6" or soemthing

YamiYukiSenpai avatar Jul 07 '24 03:07 YamiYukiSenpai

Safe to say this is abandoned. Might as well fork it and call it "termoid6" or soemthing

Sadly. And the upstream patch needed was never merged as well.

Unfortunately, I have no desire to maintain a fork of Termoid, there are some bugs and missing features that I would love to implement, but I don't feel like doing it anytime soon (if ever, but who knows).

The only thing I can say is that I'll be probably updating this PR for as long as I keep using Termoid.

JonathanxD avatar Jul 07 '24 17:07 JonathanxD

Apologies for being inactive. Unfortunately, I am currently unable to provide ample time to add new fixes or feature but I will definitely accept community support. I will try taking out some time to test PRs and ensure this doesn't get abandoned.

Thanks @JonathanxD for your support. +1 I can merge this PR this weekend and update the new releases. Let me know if you have any other changes you want to add.

koldbyte avatar Jul 12 '24 14:07 koldbyte

@koldbyte No worries. I have no new changes to push, feel free to merge it as it is.

JonathanxD avatar Jul 12 '24 21:07 JonathanxD