>>>>> "Dekel" == Dekel Tsur <[EMAIL PROTECTED]> writes:

Dekel> I've tried using fl_setup_maxpup(0) with XForms0.88, but it
Dekel> always returns 32, even after changing the maximum number.

That's strange, but the Changes document for 0.88.1 says:

V0.88.1 Janurary 1998
  Compatible, source and binary, with V0.88.
  Bug fixed version:
    fixed a couple of menu problems.
[...]

So it may be that this was fixed in 0.88.1.

>>  What about a simple solution: at the beginning of
>> Menus:ShowTocMenu(), add a fl_setpup_maxpup(max_number_of_menus);
>> which ensures we have atleast this number of menus, and costs
>> virtually nothing in terms of time.

Dekel> But how do we know the number of submenus the ToC will use? We
Dekel> will need to write a function which compute it by scanning the
Dekel> toclist vector, working similarly to Add_to_toc_menu().

I was thinking to just set it to 128 anmd hope it will work :)

Dekel> I've taken my original patch and ported it to 1.1.6, and then
Dekel> change it a bit: I added the function get_new_submenu which is
Dekel> responsible for increasing the maximum popup number.

I'll have a look.

Dekel> The patch also allows showing all the TOC when there are more
Dekel> than 25 chapters. Let me know if you need a patch for 1.1.5.

We can maybe just apply it to 1.1.6. This is not a very big problem,
after all.

JMarc

Reply via email to