Re: [patch] Chapter top spacing - regression

2018-02-12 Thread Pavel Sanda
Scott Kostyshak wrote: > On Mon, Feb 12, 2018 at 09:27:49PM +, Richard Heck wrote: > > On 02/12/2018 03:17 PM, Pavel Sanda wrote: > > > > Scott, 2.3.0 or 2.3.1? > > > > FWIW, I'd suggest 2.3.0. It's totally cosmetic and safe, but annoying > > for people with small screens. > > Go ahead for

Re: [patch] Chapter top spacing - regression

2018-02-12 Thread Scott Kostyshak
On Mon, Feb 12, 2018 at 09:27:49PM +, Richard Heck wrote: > On 02/12/2018 03:17 PM, Pavel Sanda wrote: > > Scott, 2.3.0 or 2.3.1? > > FWIW, I'd suggest 2.3.0. It's totally cosmetic and safe, but annoying > for people with small screens. Go ahead for 2.3.0. Scott signature.asc Description

Re: [patch] Chapter top spacing - regression

2018-02-12 Thread Richard Heck
On 02/12/2018 03:17 PM, Pavel Sanda wrote: > Jean-Marc Lasgouttes wrote: >> Le 10/02/2018 ?? 04:04, Richard Heck a écrit : >>> Yes, so that is the one JMarc guessed, though the real culprit is >>> e7827264e7e. >>> I would guess that the simple layout solution originally proposed is >>> correct, the

Re: [patch] Chapter top spacing - regression

2018-02-12 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: > Le 10/02/2018 ?? 04:04, Richard Heck a écrit : >> Yes, so that is the one JMarc guessed, though the real culprit is >> e7827264e7e. >> I would guess that the simple layout solution originally proposed is >> correct, then. This commit just removed special handling for c

Re: [patch] Chapter top spacing - regression

2018-02-12 Thread Jean-Marc Lasgouttes
Le 10/02/2018 à 04:04, Richard Heck a écrit : Yes, so that is the one JMarc guessed, though the real culprit is e7827264e7e. I would guess that the simple layout solution originally proposed is correct, then. This commit just removed special handling for chapters, so what's happening now is kind

Re: [patch] Chapter top spacing - regression

2018-02-09 Thread Richard Heck
On 02/09/2018 05:25 PM, Pavel Sanda wrote: > Jean-Marc Lasgouttes wrote: >> Le 09/02/2018 ?? 19:48, Pavel Sanda a écrit : > Could it be the removal of chapter hardcoding at e7827264 ? If that is not the commit that changed behavior, I could do a bisect if it would help. >>> No this wo

Re: [patch] Chapter top spacing - regression

2018-02-09 Thread Pavel Sanda
Jean-Marc Lasgouttes wrote: > Le 09/02/2018 ?? 19:48, Pavel Sanda a écrit : Could it be the removal of chapter hardcoding at e7827264 ? >>> >>> If that is not the commit that changed behavior, I could do a bisect if >>> it would help. >> No this won't be the commit. If you can still compile 2.

Re: [patch] Chapter top spacing - regression

2018-02-09 Thread Jean-Marc Lasgouttes
Le 09/02/2018 à 19:48, Pavel Sanda a écrit : Could it be the removal of chapter hardcoding at e7827264 ? If that is not the commit that changed behavior, I could do a bisect if it would help. No this won't be the commit. If you can still compile 2.0-2.1 development on your computer then pleas

Re: [patch] Chapter top spacing - regression

2018-02-09 Thread Scott Kostyshak
On Fri, Feb 09, 2018 at 06:48:13PM +, Pavel Sanda wrote: > Scott Kostyshak wrote: > > On Fri, Feb 09, 2018 at 04:54:07PM +, Jean-Marc Lasgouttes wrote: > > > > > Could it be the removal of chapter hardcoding at e7827264 ? > > > > If that is not the commit that changed behavior, I could do

Re: [patch] Chapter top spacing - regression

2018-02-09 Thread Pavel Sanda
Scott Kostyshak wrote: > On Fri, Feb 09, 2018 at 04:54:07PM +, Jean-Marc Lasgouttes wrote: > > > Could it be the removal of chapter hardcoding at e7827264 ? > > If that is not the commit that changed behavior, I could do a bisect if > it would help. No this won't be the commit. If you can st

Re: [patch] Chapter top spacing - regression

2018-02-09 Thread Scott Kostyshak
On Fri, Feb 09, 2018 at 04:54:07PM +, Jean-Marc Lasgouttes wrote: > Could it be the removal of chapter hardcoding at e7827264 ? If that is not the commit that changed behavior, I could do a bisect if it would help. Scott signature.asc Description: PGP signature

Re: [patch] Chapter top spacing - regression

2018-02-09 Thread Jean-Marc Lasgouttes
Le 07/02/2018 à 23:44, Richard Heck a écrit : No objection, but I agree that there must be some underlying cause. I'll guess that maybe the size of the Chapter font is now being used to calculate what "4" means, whereas maybe it was the size of the base font previously? It looks like we already

Re: [patch] Chapter top spacing - regression

2018-02-07 Thread Richard Heck
On 02/07/2018 05:00 PM, Pavel Sanda wrote: > Hi, > > I already wrote about this some time ago, switching to 2.3 made going through > the documents with chapters somewhat annoying on small screens because third > of > my monitor is taken by vertical space. Eg. after loading tutorial on my laptop >

[patch] Chapter top spacing - regression

2018-02-07 Thread Pavel Sanda
Hi, I already wrote about this some time ago, switching to 2.3 made going through the documents with chapters somewhat annoying on small screens because third of my monitor is taken by vertical space. Eg. after loading tutorial on my laptop I can't read even the first paragraph because of the empt