Summary: | Meta+number does not work on Plasma 6 | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Thiago Sueto <herzenschein> |
Component: | Task Manager and Icons-Only Task Manager | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | VERIFIED FIXED | ||
Severity: | major | CC: | nate, qydwhotmail |
Priority: | NOR | Keywords: | qt6 |
Version: | master | ||
Target Milestone: | 1.0 | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
See Also: | https://bugs.kde.org/show_bug.cgi?id=474228 | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/-/commit/b9c139c441b1a10fa7d6433860cad01272e4d82f | Version Fixed In: | |
Sentry Crash Report: |
Description
Thiago Sueto
2023-08-29 23:05:03 UTC
Can reproduce. Found the problem. On this line of code: https://invent.kde.org/plasma/plasma-workspace/-/blob/master/shell/shellcorona.cpp?ref_type=heads#L2499 ....we're getting -1, instead of it correctly finding the method. My theory is that the method signature changed and maybe the argument for activateTaskAtIndex(index) isn't typed as a QVariant anymore. But I can't find any documentation on what it might be (or was) and naively changing it to look for a method with an int doesn't fix it. In this cases, JavaScript's dynamically typed nature is being annoying. Giving up for now; hopefully someone more experienced than me can fix this. Git commit b9c139c441b1a10fa7d6433860cad01272e4d82f by Marco Martin. Committed on 06/09/2023 at 17:59. Pushed by mart into branch 'master'. call activateTaskAtIndex on the proper object now the appletinterface is the plasmoid root object and is the one which provides the activateTaskAtIndex function, so don't iterate on its children anymore M +13 -20 shell/shellcorona.cpp https://invent.kde.org/plasma/plasma-workspace/-/commit/b9c139c441b1a10fa7d6433860cad01272e4d82f Confirming the fix! |