Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Styles for headers inside user markdown #64

Merged
merged 2 commits into from
Jan 26, 2020

Conversation

koraa
Copy link
Contributor

@koraa koraa commented Jun 20, 2019

When markdown is used inside jsdoc, any headers are just emitted
as is; in practice this leads to bigger headers inside smaller
headers (h1 inside h2 in the readme), which does not look very good.

This commit introduces a special set of header styles for use inside
documentation, which makes the case above look great.

When markdown is used inside jsdoc, any headers are just emitted
as is; in practice this leads to bigger headers inside smaller
headers (h1 inside h2 in the readme), which does not look very good.

This commit introduces a special set of header styles for use inside
documentation, which makes the case above look great.
@ar2rsawseen
Copy link
Collaborator

ok, I see the problem, but don't think Source Pro Code font fits there stylistically.
Maybe just decreasing inner .usertext heading size is ok?

@koraa
Copy link
Contributor Author

koraa commented Sep 30, 2019

Hmm, I tried this and I do not think that's enough. Expressing five levels of headers accurately using just font size, style & margin is already very hard. Either your h1 and h2 become to big, or your h5 are barely visible any more. Double the amount would be extremely difficult to accurately represent…

I think using a distinct font is the best way to achieve this; personally I do like source code pro, but that is completely a matter of taste… which font would you like?

@ar2rsawseen ar2rsawseen merged commit b049fba into clenemt:master Jan 26, 2020
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.

2 participants