phpBB • Free and Open Source Forum Software

johnd0e · by **johnd0e** » Fri Oct 04, 2019 3:18 pm

Currently all content (generated with this extension enabled) is wrapped into additional aragraph.
Is this really necessary?

In fact, it conflicts with [spoiler] bbcode, and I suspect that it can be caused by that excessive wrapping.

AbaddonOrmuz · by **AbaddonOrmuz** » Sun Oct 06, 2019 2:52 am

johnd0e wrote: Currently all content (generated with this extension enabled) is wrapped into additional aragraph.
Is this really necessary?

Well, that's the most adopted implementation for Markdown. Removing them is not an option since Litedown, the plugin used for that, works that way.

The problem is not that it adds extra tags, the problem is that, phpBB changes the styling for some tags like the one for paragraph.

I would guess it's because it wasn't expected that someone would add those tags in the content (post, messages, etc).

I'm treating those kind of issues as CSS bugs, so if you have issues you could report them here.

johnd0e wrote: In fact, it conflicts with [spoiler] bbcode, and I suspect that it can be caused by that excessive wrapping.

Well, I don't think it adds excessive paragraph wrapping, that's how (in my opinion) should behave.

In the particular case for the spoiler, you could easily fix it with CSS. For example (untested):

Code: Select all

.spoiler p:last-child {
	margin-bottom: 0;
}

johnd0e · by **johnd0e** » Wed Oct 09, 2019 3:02 pm

Removing them is not an option since Litedown, the plugin used for that, works that way.

In fact it's possible https://github.com/s9e/TextFormatter/is ... -539991514

In the particular case for the spoiler, you could easily fix it with CSS.

Perhaps you mean https://github.com/AlfredoRamos/phpbb-e ... le-spoiler

Unfortunately our board implements spoiler functionality via such bbcode: https://pastebin.com/aizmGTJP

AbaddonOrmuz · by **AbaddonOrmuz** » Wed Oct 09, 2019 7:23 pm

johnd0e wrote:In fact it's possible https://github.com/s9e/TextFormatter/is ... -539991514

Thanks but I'm already aware of that.

I didn't say it was not possible, I said that it's not an option to remove that feature from this extension.

johnd0e wrote: Unfortunately our board implements spoiler functionality via such bbcode: https://pastebin.com/aizmGTJP

Then please, for future references, provide as much information as you can when reporting an issue.

You can try the following:

Code: Select all

.spoilercontent  p:last-child {
	margin-bottom: 0;
}

Markdown

remove <p>content</p> wrapping - Markdown

advertisements