tmedwards / sugarcube-2

SugarCube is a free (gratis and libre) story format for Twine/Twee.
https://www.motoslave.net/sugarcube/2/
BSD 2-Clause "Simplified" License
185 stars 42 forks source link

"Config.cleanupWikifierOutput = true;" is ineffective withing a "div" #208

Open selden opened 1 year ago

selden commented 1 year ago

"Config.cleanupWikifierOutput = true;" results in an inappropriate increase in the vertical spacing between paragraphs and between lists within a "div". Paragraph indentation also changes within that "div". Apparently the cleanup doesn't happen within a "div". I see no mention of this cleanup limitation in SugarCube's documentation

To Reproduce: Steps to reproduce the behavior:

  1. specify Config.cleanupWikifierOutput = true; within a script passage
  2. specify .passage p { text-indent: 3.0em; } within a stylesheet passage
  3. surround paragraphs and lists by div tags in a normal passage

Expected behavior. Paragraphs should be indented lists should be separated by only a single blank line.

Screenshots.

test_div

Project details.

Desktop details.

Additional context. The zip below includes the twee and html necessary to reproduce this problem.

test_div.zip

selden commented 1 year ago

This looks like it might be related to issue #189.

Are there still plans for a v3 of SugarCube? I'm sure there are quite a few people who'd be glad to alpha-test it for you :)