On Sun, Sep 02, 2007 at 12:46:00AM +0200, Abdelrazak Younes wrote: > Alfredo Braunstein wrote: > > Abdelrazak Younes wrote: > >>> But what about deeply nested stuff, or stuff inside minipages etc. It > >>> seems difficult to come up with good heuristics to avoid that metrics > >>> call without using textwidth. > >> I think that by testing he dimension of the button and if we forbid > >> openinlined if the button is too large wrt the screen, this is OK. > > Sure, IIUC ideally we should test if the inset just fits inlined with the > > button or not. But the reference width here should be the textwidth_, not > > the screen width (think about deeply nested stuff). > > Previsouly we had (rev 14161): > > openinlined_ = textdim_.wid + 2 * dim.wid <= mi.base.textwidth; > > Probably the wide() thing was interfering badly with that. Might be a good > idea to come back to the old formula now. Martin, would you care to comment?
Sure... wide() is gone and (probably) so is the reason for this. Go for a better formula if it works. - Martin