Re: [LyX/master] Adapt caret height to context in mathed.

2018-05-02 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: >>> Pavel, please test this new code. It should get the cursor size right >>> without crashing of changing bizarrely the current font. >>> >>> Riki, if this works, I will probably propose to backport to 2.3.2. >> Any chance you would provide 2.3 patch? Anything in master

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-27 Thread Jean-Marc Lasgouttes
Le 27/04/2018 à 11:35, Pavel Sanda a écrit : Jean-Marc Lasgouttes wrote: Le 27/04/2018 ?? 00:13, Jean-Marc Lasgouttes a écrit : commit 90cfe4ec3b4ff22ef798a63e98ca70d0d33a1656 Author: Jean-Marc Lasgouttes Date: Fri Apr 27 00:03:48 2018 +0200 Adapt caret height to context in mathed.

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-27 Thread Jean-Marc Lasgouttes
Le 27/04/2018 à 13:17, Pavel Sanda a écrit : Clearly you prefer to think in a systematic way. That might be good for the code, but what will remain for all the procrastinating hackers in the future who will be left without the mystery where the b came from? The good side is that we have plent

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-27 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: >> I was refering merely to caret height; but we are getting to nitpicking, >> meaning the situation is likely good enough for now :) > > Currently, the caret is never higher than the cell it is in. This is why I > argue that the cell is too small. Clearly you prefer t

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-27 Thread Jean-Marc Lasgouttes
Le 27/04/2018 à 12:38, Pavel Sanda a écrit : Jean-Marc Lasgouttes wrote: Le 27/04/2018 ?? 11:47, Pavel Sanda a écrit : Limited testing shows no crash. As said previously cursor is much better now, though I would suggest use of 'b' instead of 'x' min height, e.g. try "\dot u" in mathed and compa

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-27 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: > Le 27/04/2018 ?? 11:47, Pavel Sanda a écrit : >> Limited testing shows no crash. As said previously cursor is much better >> now, >> though I would suggest use of 'b' instead of 'x' min height, e.g. try >> "\dot u" >> in mathed and compare both versions. > > Note tha

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-27 Thread Jean-Marc Lasgouttes
Le 27/04/2018 à 11:47, Pavel Sanda a écrit : Limited testing shows no crash. As said previously cursor is much better now, though I would suggest use of 'b' instead of 'x' min height, e.g. try "\dot u" in mathed and compare both versions. Note that, while height of x is a proper font measuremen

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-27 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: > Le 27/04/2018 ?? 00:13, Jean-Marc Lasgouttes a écrit : >> commit 90cfe4ec3b4ff22ef798a63e98ca70d0d33a1656 >> Author: Jean-Marc Lasgouttes >> Date: Fri Apr 27 00:03:48 2018 +0200 >> Adapt caret height to context in mathed. >> Compute a height from curr

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-27 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: > Le 27/04/2018 ?? 00:13, Jean-Marc Lasgouttes a écrit : >> commit 90cfe4ec3b4ff22ef798a63e98ca70d0d33a1656 >> Author: Jean-Marc Lasgouttes >> Date: Fri Apr 27 00:03:48 2018 +0200 >> Adapt caret height to context in mathed. >> Compute a height from curr

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-26 Thread Richard Kimberly Heck
On 04/26/2018 06:17 PM, Jean-Marc Lasgouttes wrote: > Le 27/04/2018 à 00:13, Jean-Marc Lasgouttes a écrit : >> commit 90cfe4ec3b4ff22ef798a63e98ca70d0d33a1656 >> Author: Jean-Marc Lasgouttes >> Date:   Fri Apr 27 00:03:48 2018 +0200 >> >> Adapt caret height to context in mathed. >>

Re: [LyX/master] Adapt caret height to context in mathed.

2018-04-26 Thread Jean-Marc Lasgouttes
Le 27/04/2018 à 00:13, Jean-Marc Lasgouttes a écrit : commit 90cfe4ec3b4ff22ef798a63e98ca70d0d33a1656 Author: Jean-Marc Lasgouttes Date: Fri Apr 27 00:03:48 2018 +0200 Adapt caret height to context in mathed. Compute a height from current font and current cell vertical d