Bo Peng wrote:
> The following patch has just been committed to fix the toolbar
> management problem. It
> 
> 1. change ToolbarMap to ToolbarList
> 2. sort the loaded toolbars according to their saved locations.
> Something like, top/line1/1st, top/line1/2nd, top/line2/1st,
> right/line1/1st ...
> 3. add toolbars in order and add toolbar breaks if necessary.

Great (also because it gets rid of some rather ugly code). I cannot test it
now, but I have one question:

You remove the session thing (which is good). However, how are toolbars read
in initially (when no session info is yet available)? Are all toolbar put
on single lines? (even if so, the patch is an improvement of course; just
asking).

Jürgen
 
> Please test.
> Bo


Reply via email to