aesophor / wmderland

🌳 X11 tiling window manager using space partitioning trees
https://www.reddit.com/r/unixporn/comments/fb8ve1/wmderland_104_time_to_move_on/
MIT License
409 stars 15 forks source link

toggle_floating not working #24

Open nikotidar opened 4 years ago

nikotidar commented 4 years ago

Hi again! :dancer: I'm trying to use Wmderland on Kali Linux, it's works perfectly! But, toggle_floating is not working. Here's the log (maybe)

Nov 27 05:26:41 MacBook-Pro kernel: [41269.431278] traps: Wmderland[256837] trap divide error ip:56531cf4db41 sp:7fff96e586e0 error:0 in Wmderland[56531cf4a000+b000]

Os: Kali 2019.4 Wmderland Build: Latest

aesophor commented 4 years ago

Hi there!

I've tried running Wmderland on Kali 2019.4 and floating seems to be working for me.

May I ask how you started the wm? Did you start it via lightdm, or simply via startx? I started it with startx.

nikotidar commented 4 years ago

Hi there!

I've tried running Wmderland on Kali 2019.4 and floating seems to be working for me.

May I ask how you started the wm? Did you start it via lightdm, or simply via startx? I started it with startx.

Hi, Marco!

For login manager, i'm using GDM3. Thanks before.

-- Confirmed, using xinit toggle_floating is working. For now, i've disabled Gdm3 for a while.

aesophor commented 4 years ago

I've successfully reproduced the bug by these steps:

  1. spawn a window
  2. press Mod+g (or Mod+v) multiple times
  3. press toggle_floating

The problem is in Workspace::SetTilingDirection, setting tiling direction multiple times will mess up the internal data structures. I'll try to redesign the algorithm here. Thanks!


I tried running grep -r ' / ' to find all statements that could potentially lead to divide-by-zero errors:

int child_width = (dir == TilingDirection::HORIZONTAL) ? w / children.size() : w;
int child_height = (dir == TilingDirection::VERTICAL) ? h / children.size() : h;
nikotidar commented 4 years ago

I'm looking for the next update. Thanks, Marco!

aesophor commented 4 years ago

Hello friend, I've fixed this bug in PR #28 , could you please check if it works on your machine?

Thanks! :smile:

nikotidar commented 4 years ago

toggle_floating still not working with gdm3 and Wmderland latest build, i'm back using xinit (again) :joy: