Pipemenu and submenuShowDelay

[johanmalm]

Implement cat B+ items 7.1.4 and 2.11.3

https://github.com/labwc/labwc-scope/blob/main/incomplete-cat-B-items.md

The notes below are are based on an IRC discussion. Critique + review + ideas welcome, but it's mostly there as a note-to-self so that I remember what I'm doing.

• Split parse_xml() to take fp from fopen() or popen()
• Replace bespoke array with wl_list to make iteration easier when splitting menu.c
• Implement pipe menus Support pipemenus #1216
• Split menu.c
• Move handle_menu_keys() to menu/input.c
• Refactor menu_configure() to be called in real time rather just once on initialization (which is fine for a static menu, but not so good for a dynamic one).
• Re-consider what server->menu variables we want (currently just menu_current, but no selection)
• Should menu struct contain size/offset?
• Implement submenu delay
• Handle menus which are too big for usable_output

[Consolatis]

Move handle_menu_keys() to menu/input.c

Do we really want to do that? It uses various XKB_KEY_ defines and basically just maps keys to actions. I think src/keyboard.c is a good fit along with the other keybind handling.

Moving the handlers that are called by that function (and those that are called by src/cursor.c like menu_process_cursor_motion) somewhere else within src/menu/ sounds totally fine, just not sure about handle_menu_keys() itself.

Another thing: do we want a private and public header similar to ssd.h and ssd-internal.h?

[johanmalm]

Okay. I'll split without moving the key-handling thing into it.

Yes, good idea about a private header.

[Consolatis]

Marked the pipe menu entry as done. Is there anything you want to see in 0.7.2? Otherwise I'd suggest that we move this to the 0.7.3 milestone.

[johanmalm]

Thanks. Nothing else is imminent. I've moved it to 0.7.3