Re: r35377 - lyx-devel/trunk/src/insets

2010-09-15 Thread Abdelrazak Younes
On 09/15/2010 01:42 PM, Abdelrazak Younes wrote: On 09/15/2010 01:39 PM, Richard Heck wrote: On 09/15/2010 07:28 AM, Abdelrazak Younes wrote: void InsetLine::draw(PainterInfo& pi, int x, int y) const { -frontend::FontMetrics const& fm = theFontMetrics(pi.base.font); +// FIXME: We

Re: r35377 - lyx-devel/trunk/src/insets

2010-09-15 Thread Abdelrazak Younes
On 09/15/2010 01:39 PM, Richard Heck wrote: On 09/15/2010 07:28 AM, Abdelrazak Younes wrote: void InsetLine::draw(PainterInfo& pi, int x, int y) const { -frontend::FontMetrics const& fm = theFontMetrics(pi.base.font); +// FIXME: We cannot use InsetCommand::dimension() as this retu

Re: r35377 - lyx-devel/trunk/src/insets

2010-09-15 Thread Richard Heck
On 09/15/2010 07:28 AM, Abdelrazak Younes wrote: void InsetLine::draw(PainterInfo& pi, int x, int y) const { -frontend::FontMetrics const& fm = theFontMetrics(pi.base.font); +// FIXME: We cannot use InsetCommand::dimension() as this returns the dimension +// of the button, whi

Re: r35377 - lyx-devel/trunk/src/insets

2010-09-15 Thread Abdelrazak Younes
Uwe, As a final cutesy I just committed a final fix. As I said, if you want to further improve things, starts from there; but I don't really know what needs to be improved. Drawing works well, now. Otherwise, if you insist on putting your own code, we should just remove this new inset. Abdel