Skip to content

Update html heading structure#280

Open
femmdi wants to merge 4 commits into
AntennaPod:masterfrom
femmdi:master
Open

Update html heading structure#280
femmdi wants to merge 4 commits into
AntennaPod:masterfrom
femmdi:master

Conversation

@femmdi

@femmdi femmdi commented Oct 15, 2023

Copy link
Copy Markdown
Contributor

Structure headings to have only one H1 per page.

Structure headings to have only one H1 per page.
@ByteHamster

Copy link
Copy Markdown
Member

I wonder if we can shift the headings one level down programmatically. Feels a bit strange to have a markdown file where # is not allowed because it is already used outside the markdown file

@keunes

keunes commented Oct 18, 2023

Copy link
Copy Markdown
Member

Yeah, was thinking the same (see #211). Haven't looked into a technical solution yet though.

@ByteHamster

Copy link
Copy Markdown
Member

Apparently it's okay to have multiple <h1> on the same page if one of them is in an <article> or similar. Then we just need to adapt the font size instead of actually shifting the levels. Given that we are discussing some website changes lately, should we maybe postpone this until the website changes are done?

@keunes

keunes commented Dec 11, 2023

Copy link
Copy Markdown
Member

Given that we are discussing some website changes lately, should we maybe postpone this until the website changes are done?

Yes, I think that's best.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants