Johan Malm

This seems to fix the occasional duplicate menus / ids. ``` diff --git a/src/menu/menu.c b/src/menu/menu.c index 2069b654..6eb8cf81 100644 --- a/src/menu/menu.c +++ b/src/menu/menu.c @@ -988,6 +988,8 @@ handle_pipemenu_timeout(void *_ctx) return 0;...

Intend to merge in a day or so. Testers - if you want to try to break it, now is a good time :smile:

Just did an interactive-rebase HEAD~3 to fixup the last two.

> Should I write in #1226?(did not take the time to read it yet) No here is enough. Thanks for feedback.

I think it would be cleaner to implement that in a separate foreign-toplevel client and wtype.

Ah, yes, what I meant is that if you define keybinds in rc.xml (for window move/resize/snap-to-region/etc) then you can trigger them from a script with wtype

v2. Just tidied up some coding style.

If a test fails, just run it directly ./t1000-but.c to see what triggered the failure We could simply just create a separate repo to keep it clean and reduce commit...

> Actually, it makes more sense to me to do functional testing first. Complex actions, different config layouts, maybe some kind of moving windows back and forth, that sort of...

Have added a couple of paragraphs in `t/README.md`: https://github.com/labwc/labwc/pull/1099/files#diff-ea1cf576d6af5497c99c07e86980795b200126085e5bbb0a3e30cfc95a9d512f