causa-prima / TIL3R

TIL3R adds more manual tiling options to KWin. With this script you can divide you screen in thirds, using the full or half screen height.
GNU General Public License v2.0
10 stars 16 forks source link

Broken #8

Open mikketeus opened 2 years ago

mikketeus commented 2 years ago

Hi, I love this tiler, but it does not seem to work with kde plasma 5.23.1.

javiromero commented 2 years ago

I'm facing the same issue. I wonder how can I provide any meaningful log to help debugging it?

In my case when I try to use any key combination to move the windows to thirds, quarters, etc. the compositor gets disabled and the window doesn't move.

Operating System: KDE neon 5.23 KDE Plasma Version: 5.23.2 KDE Frameworks Version: 5.87.0 Qt Version: 5.15.3 Kernel Version: 5.11.0-38-generic (64-bit) Graphics Platform: X11 Processors: 12 × AMD Ryzen 5 2600 Six-Core Processor Memory: 15.5 GiB of RAM Graphics Processor: Radeon RX 580 Series

causa-prima commented 2 years ago

I'm sorry to hear you're having trouble, but I'm afraid I can't help, as I am currently not using TIL3R (or even a PC). Thus, I won't be able to debug etc. anything, but will merge any pull request that is reported to solve the issue and looks okay overall.

javiromero commented 2 years ago

Sorry to hear that but thank you for sharing this with us: I've been using this for years and it has really helped with my work flows, i really appreciate it!

mikketeus commented 2 years ago

Too bad my JS skills are none existing, but there might be some clues here?

Nov 2 21:04:43 XPS-15 kwin_x11[9952]: js: TIL3R called with wide: 2, choice 1 Nov 2 21:04:44 XPS-15 kded5[1705]: Registering ":1.109/StatusNotifierItem" to system tray Nov 2 21:04:44 XPS-15 plasmashell[1785]: QFont::setPointSizeF: Point size <= 0 (0.000000), must be greater than 0 Nov 2 21:04:44 XPS-15 kwin_x11[10071]: Warning: Setting a new default format with a different version or profile after the global shared context is created may cause issues with context sharing. Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5127:46: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5129:48: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5131:48: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5135:47: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5137:46: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5145:48: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5149:46: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5151:46: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5155:45: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: /usr/share/X11/locale/en_US.UTF-8/Compose:5158:46: this compose sequence is a duplicate of another; skipping line Nov 2 21:04:46 XPS-15 kwin_x11[10071]: kwin_xkbcommon: XKB: inet:224:50: unrecognized keysym "XF86FullScreen" Nov 2 21:04:46 XPS-15 kwin_x11[10071]: js: TIL3R active

mikketeus commented 2 years ago

Good news, after some research I found that the kde version you can install from "Get New Scripts" differs from the github version. Check out line 90 .local/share/kwin/scripts/TIL3R/contents/code/til3r.js

kde version: var workGeo = workspace.clientArea(KWin.WorkArea, 1, 1); github version: var workGeo = workspace.clientArea(KWin.PlacementArea, activeClient.screen, 1);

Regards mikkeTeus