Jean-Marc Lasgouttes wrote: > Le 03/10/2015 01:27, Pavel Sanda a écrit : >> Jean-Marc Lasgouttes wrote: >>> Why not do it in InsetText::Metrics directly? Is it a box-only problem? >> >> Hmm, is there another insettext which allows fixing to certain percentage >> of screen size width? Wrapping float came to my mind but apparently >> it does not try to visualize the width set. No other inset came to my >> mind. > > You should not have to care about it.
Why? We definitely don't want to do this change for one-liner insets like branch or insets without button like ERT. > The code in InsetText::metrics has > provision for too narrow inset, it can be improved for all cases IMO. Note > that all our explicit pixel counts should be updated to take zoom into > account, or even current font em size (think retina display). I thought that zoom is already taken into account in the patch sent, e.g. button size is already calculated wrt zoom. Pavel
