By just looking at the code, it makes me think it won't work properly if I were to actually use it. The library treats windows as any other control, with a window_id and a control_id. It should just take an 'id'.
Also, other controls could take a /Window object as a window_id. It might look nicer even though it'll still be converted to a text string in the end.