Proposal: Option to bind listener on backend that is handled without spawning a new thread about webui HOT 7 CLOSED

Implemented (4fb247e).

// Single window
webui_set_event_block(win, true);

// All windows
webui_set_config(ui_event_block, true);

from webui.

Actually, all old versions (2.2 and older) were single thread processing system, but I noticed if an event block for too long by the backend code, the user may click again, and again, which give a bad UX overall.

Your proposal is instead of completely removing the single thread processing, we can add a new API to let the programmer to choose which system he prefers. Honestly, this is a great idea. Thank you for the suggestion.

Making this new option at bind level may take time to correctly implement and test. I suggestion instead making it a window level, so the new API can be for example webui_set_bind_blocking(myWindow, true), so all binds in this window will be a one thread processing system.

@dgllghr, what do you think?

from webui.

I like that! I like it because it reduces complexity (you are only ever dealing with one system or the other). And I like it because my guess would be that most users would really only want one or the other for a single app: all events are ordered (or not), we have a separate runtime system (or we want webui to handle it).

I'm happy to give the implementation a shot. Or if you'd rather do it since it is a fundamental change, I understand.

from webui.

That's fine. I have already started working on it. I will push the commit soon.

I'm also thinking of adding it to webui_set_config() as you said, in case someone wants this system for the whole app (all windows). However, webui_set_bind_blocking() is for a single window if someone wants both systems.

from webui.

Oh I see. I misunderstood the distinction. Yes I think that makes sense, too.

from webui.

I just noticed the name, probably webui_set_event_block should be webui_set_event_blocking.

from webui.

webui_set_event_block renamed to webui_set_event_blocking.
d394a2f

from webui.