wf49670 / ppgen

Post-processing generator for DP
6 stars 4 forks source link

vertical spacing #64

Closed ghost closed 9 years ago

ghost commented 9 years ago

I've just pushed to develop my attempt at coding Nigel's solution to the vertical spacing problem Carolyn identified. I pushed it after sending a copy to her and getting a response that it seems to work. My testing is not exhaustive by any means. I feel like I've missed something but I'm not sure what. I will experiment with it some more and then announce perhaps in the morning that it's available for testing.

wf49670 commented 9 years ago

On 11/17/2014 1:01 AM, rfrank wrote:

I've just pushed to develop my attempt at coding Nigel's solution to the vertical spacing problem Carolyn identified. I pushed it after sending a copy to her and getting a response that it seems to work. My testing is not exhaustive by any means. I feel like I've missed something but I'm not sure what. I will experiment with it some more and then announce perhaps in the morning that it's available for testing.

To make sure I understand the theory: By changing the structure of the .h2 HTML and the .nf HTML ppgen should now allow the PPer's .sp 2 following a .h2 to be absorbed as needed by the bottom margin for the h2 or the top margin for the .nf, which should give just the 2em of space that the PPer wanted. And this makes the PPer's markup work for both text and HTML versions.

Right?

Thanks, Walt

ghost commented 9 years ago

Yes that's how it is coded. The chapter div is completed before the h2 and the spacing on the no-fill block is moved to the outermost level. The other adjustment was to eliminate the spacing before the first inner block since it wasn't at the same level as the outer div where it could be absorbed.

ghost commented 9 years ago

I'll close this as an issue. I've seen no negative reports.