alexankitty / FancyTasks

Make your plasma task manager widget pretty.
GNU General Public License v3.0
132 stars 7 forks source link

Indicator on top doesn't work anymore #36

Closed Vistaus closed 1 year ago

Vistaus commented 1 year ago

Thank you for the latest update. However, showing the indicator on top of a button is now broken for me (worked fine before the update). Left and right too, btw: it only displays the indicator on the bottom regardless of the setting.

alexankitty commented 1 year ago

image image image These seem to be working for me. Can you give me some more specifics? A print out of journalctl might help.

Vistaus commented 1 year ago

Hmm, journalctl doesn't show any errors:

~ journalctl -b | grep FancyTasks
Hint: You are currently not seeing messages from other users and the system.
      Users in the 'systemd-journal' group can see all messages. Pass -q to
      turn off this notice.
aug 23 11:30:06 localhost kwin_wayland[2563]: js: applicationswitcher: activated [FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:30:06 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:30:06 localhost kwin_wayland[2563]: js: applicationswitcher: getting app group [FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:30:06 localhost kwin_wayland[2563]: js: applicationswitcher: auto-raising FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:30:25 localhost kwin_wayland[2563]: js: applicationswitcher: activated FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:30:25 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:30:25 localhost kwin_wayland[2563]: js: applicationswitcher: getting app group [FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:30:25 localhost kwin_wayland[2563]: js: applicationswitcher: auto-raising FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:31:11 localhost kwin_wayland[2563]: js: applicationswitcher: activated FancyTasks : fish — Yakuake
aug 23 11:31:11 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,undefined,undefined,undefined,undefined,undefined,FancyTasks : fish — Yakuake]
aug 23 11:31:13 localhost kwin_wayland[2563]: js: applicationswitcher: activated FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:31:13 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:31:13 localhost kwin_wayland[2563]: js: applicationswitcher: getting app group [FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:31:13 localhost kwin_wayland[2563]: js: applicationswitcher: auto-raising FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:31:17 localhost kwin_wayland[2563]: js: applicationswitcher: activated FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:31:17 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:31:17 localhost kwin_wayland[2563]: js: applicationswitcher: getting app group [FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:31:17 localhost kwin_wayland[2563]: js: applicationswitcher: auto-raising FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:31:31 localhost kwin_wayland[2563]: js: applicationswitcher: activated FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:31:31 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:32:40 localhost kwin_wayland[2563]: js: applicationswitcher: activated FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:32:40 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 11:33:33 localhost kwin_wayland[2563]: js: applicationswitcher: activated FancyTasks: Make your plasma task manager widget pretty. - Falkon
aug 23 11:33:33 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,FancyTasks: Make your plasma task manager widget pretty. - Falkon]
aug 23 12:14:40 localhost kwin_wayland[2563]: js: applicationswitcher: activated FancyTasks : fish — Yakuake
aug 23 12:14:40 localhost kwin_wayland[2563]: js: applicationswitcher: updating app group [undefined,undefined,undefined,undefined,undefined,undefined,undefined,undefined,undefined,undefined,undefined,FancyTasks : fish — Yakuake]
aug 23 12:14:40 localhost kwin_wayland[2563]: js: applicationswitcher: getting app group [FancyTasks : fish — Yakuake]
aug 23 12:14:40 localhost kwin_wayland[2563]: js: applicationswitcher: auto-raising FancyTasks : fish — Yakuake
~ 

OS: openSUSE Tumbleweed Plasma version: 5.27.7 APU: AMD Ryzen 5900HX

It's so strange, because like I said: it worked fine before the update. So I have no idea what's changed.

alexankitty commented 1 year ago

There's no useful output in there unfortunately. Maybe doing journalctl /usr/bin/plasmashell -f will provide something more useful. I did change the indicator to not be directly apart of the frame svg, though I need to know why your setup is ignoring the setting.

alexankitty commented 1 year ago

In the middle of setting up translations at the moment, I noticed you do use the Dutch translation. I have changed some of the i18ns around, so that might be why you're having problems. Let me finish getting this built into the main repo and then we'll go from there.

alexankitty commented 1 year ago

Tested under nl_NL.UTF-8 and seems to work for me. Mind updating and reporting back?

Vistaus commented 1 year ago

journalctl /usr/bin/plasmashell -f

Thanks. Tried that, but there's no output related to FancyTasks in there.

alexankitty commented 1 year ago

Any luck with updating? May also want to remove and re-add the widget to force it to use default settings and then reconfigure. Might give me some hints on where the issue is.

Vistaus commented 1 year ago

Sorry for my late reply! I just re-added the widget and reconfigured everything, however, the issue is still present. :(

alexankitty commented 1 year ago

This makes no sense to me... Can you send me your config? Do you use it as a widget? Is it docked to the panel? Which direction is the panel? Still kinda think it might be a language thing. I'll take a look at the actual enum for it and see if there's a more sanitized solution.

alexankitty commented 1 year ago

Here's the list of variables responsible for determining the indicator location plasmoid.configuration.indicatorOverride plasmoid.configuration.indicatorLocation plasmoid.location plasmoid.configuration.indicatorReverse

                    console.log(`indicatorOverride: ${plasmoid.configuration.indicatorOverride}`)
                    console.log(`indicatorLocation: ${plasmoid.configuration.indicatorLocation}`)
                    console.log(`plasmoid.location: ${plasmoid.location}`)
                    console.log(`indicatorReverse: ${plasmoid.configuration.indicatorReverse}`)

If you could stick the above codeblock right after line 504 in $XDG_HOME/.local/share/plasma/plasmoids/alexankitty.fancytasks/contents/ui/Task.qml (from AUR it's /usr/share/plasma/plasmoids/alexankitty.fancytasks/contents/ui/Task.qml) It would give me an idea of why the Top state won't trigger. Once done run the following in your terminal

killall plasmashell
kstart5 plasmashell

That should directly log what plasma is doing to your terminal and you should be able to see the logging output look something like this.

qml: indicatorReverse: false
qml: indicatorOverride: false
qml: indicatorLocation: 3
qml: plasmoid.location: 4
alexankitty commented 1 year ago

Task.zip Or you could extract this to the directory I mentioned earlier.

alexankitty commented 1 year ago

Now that I think about it, you could also try rebuilding the Qt cache.

rm ~/.config/Trolltech.conf
$ kbuildsycoca5 --noincremental
Vistaus commented 1 year ago

I figured it out: it had to do with the desktop theme I was using. Changed it and now it works fine again. Thank you for all of your help - I really appreciate it! 🙂