alshedivat / al-folio

A beautiful, simple, clean, and responsive Jekyll theme for academics
https://alshedivat.github.io/al-folio/
MIT License
10.43k stars 10.89k forks source link

Block quote formatting in distill-style posts #2458

Open albertfgu opened 3 months ago

albertfgu commented 3 months ago

Have you checked that your issue isn't already filed?

Bug description

Block quotes aren't formatted properly in distill-style blog posts

How to reproduce the bug

The example blog post with block quotes gives this example

> ##### TIP
>
> A tip can be used when you want to give advice
> related to a certain content.
{: .block-tip }

which is displayed as: image

However, the same example in a distill-style blog post looks like: image

The former looks much better: Ideally at least, block quotes would support headers that are non-italicized and colored.

(Personally, I think that having the entire quote in non-italics is nicer, but at least the header should be formatted)

Error messages and logs

No response

What operating system are you using?

Mac

Where are you seeing the problem on?

Running locally without Docker

More info

No response

scottleechua commented 1 month ago

@george-gca hello! 👋

as a long-time al-folio user I'm looking to start contributing back and want to help with this issue.

I do agree with having formatting consistency between distill and non-distill posts, but this would mean hand-tweaking the CSS in template.v2.js and transforms.v2.js.

Is this advisable? Are those autogenerated files? Is formatting parity between distill and non-distill a goal for this theme?

george-gca commented 1 month ago

I believe at least for the main parts (like code theme, block quote, maybe others) this is advisable. As for hand-tweaking their CSS, maybe we could add a new scss file that will be imported only in the distill layout overriding the specific rules. This way we don't have to mess inside these files, just look at which overrides will be made.

The distill template works as in this guide, simply by importing these files. These files were probably auto generated a long time ago, since their repository doesn't see an update in years and neither their site. They even posted about taking a long hiatus. So I think it is safe to say that they won't change for the foreseeable future too.

scottleechua commented 1 month ago

Got it! How have you and the maintainers thought about feature parity between the two kinds of post layouts, standard and distill?

For instance, creating tabs in posts works in standard but not distill.

I've had a quick skim of the post features, and the two that make sense to me to support in distill posts are 1) custom blockquotes, and 2) posts that can be cited.

What do you think? I'm happy to open a PR for the two listed above. I just realize it might be good to establish what features are supported where.

george-gca commented 1 month ago

I believe it would be useful to support all (if not most) features already implemented for regular posts in the distill template, unless it conflicts with it (for example, the table of contents, or we can have another look at it). At least I see the distill template as a basic template that we shoould customize as we wish.

What do you guys thin @alshedivat, @pourmand1376?